]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
gpio: gpio-dwapb: add optional reset
authorAlan Tull <atull@kernel.org>
Wed, 11 Oct 2017 16:34:44 +0000 (11:34 -0500)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 19 Oct 2017 20:32:38 +0000 (22:32 +0200)
Some platforms require reset to be released to allow register
access.

Signed-off-by: Alan Tull <atull@kernel.org>
Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
[Added DT bindings oneliner for standard reset binding]
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt
drivers/gpio/gpio-dwapb.c

index 4d6c8cdc85860567a09a1f1a2be269c3781bbee6..4a75da7051bd218c12ef8ff3bf49e555e2e0991c 100644 (file)
@@ -29,6 +29,7 @@ controller.
 - interrupts : The interrupt to the parent controller raised when GPIOs
   generate the interrupts.
 - snps,nr-gpios : The number of pins in the port, a single cell.
+- resets : Reset line for the controller.
 
 Example:
 
index 5cdb7bc3ad99b6d0f2f1605b4c876c917d63520d..d782ad195c891f9a367630f0af4235b7b3a424c6 100644 (file)
@@ -25,6 +25,7 @@
 #include <linux/of_irq.h>
 #include <linux/platform_device.h>
 #include <linux/property.h>
+#include <linux/reset.h>
 #include <linux/spinlock.h>
 #include <linux/platform_data/gpio-dwapb.h>
 #include <linux/slab.h>
@@ -98,6 +99,7 @@ struct dwapb_gpio {
        unsigned int            nr_ports;
        struct irq_domain       *domain;
        unsigned int            flags;
+       struct reset_control    *rst;
 };
 
 static inline u32 gpio_reg_v2_convert(unsigned int offset)
@@ -629,6 +631,12 @@ static int dwapb_gpio_probe(struct platform_device *pdev)
        gpio->dev = &pdev->dev;
        gpio->nr_ports = pdata->nports;
 
+       gpio->rst = devm_reset_control_get_optional_shared(dev, NULL);
+       if (IS_ERR(gpio->rst))
+               return PTR_ERR(gpio->rst);
+
+       reset_control_deassert(gpio->rst);
+
        gpio->ports = devm_kcalloc(&pdev->dev, gpio->nr_ports,
                                   sizeof(*gpio->ports), GFP_KERNEL);
        if (!gpio->ports)
@@ -680,6 +688,7 @@ static int dwapb_gpio_remove(struct platform_device *pdev)
 
        dwapb_gpio_unregister(gpio);
        dwapb_irq_teardown(gpio);
+       reset_control_assert(gpio->rst);
 
        return 0;
 }