* [PATCH] watchdog: mena21_wdt: Convert to GPIO descriptors
@ 2018-12-02 11:04 Linus Walleij
2018-12-03 8:12 ` Johannes Thumshirn
2018-12-15 16:28 ` Guenter Roeck
0 siblings, 2 replies; 4+ messages in thread
From: Linus Walleij @ 2018-12-02 11:04 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck
Cc: linux-watchdog, Linus Walleij, Johannes Thumshirn,
Johannes Thumshirn, Johannes Thumshirn
This drops the old OF API use to look up global GPIO
numbers and replace it with the GPIO descriptor API.
Cc: Johannes Thumshirn <jthumshirn@suse.de>
Cc: Johannes Thumshirn <morbidrsa@gmail.com>
Cc: Johannes Thumshirn <jth@kernel.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
drivers/watchdog/mena21_wdt.c | 73 +++++++++++++++++------------------
1 file changed, 36 insertions(+), 37 deletions(-)
diff --git a/drivers/watchdog/mena21_wdt.c b/drivers/watchdog/mena21_wdt.c
index 0be7f50e8ff9..6db69883ece6 100644
--- a/drivers/watchdog/mena21_wdt.c
+++ b/drivers/watchdog/mena21_wdt.c
@@ -13,10 +13,10 @@
#include <linux/platform_device.h>
#include <linux/watchdog.h>
#include <linux/uaccess.h>
-#include <linux/gpio.h>
-#include <linux/of_gpio.h>
+#include <linux/gpio/consumer.h>
#include <linux/delay.h>
#include <linux/bitops.h>
+#include <linux/of.h>
#define NUM_GPIOS 6
@@ -31,7 +31,7 @@ enum a21_wdt_gpios {
struct a21_wdt_drv {
struct watchdog_device wdt;
- unsigned gpios[NUM_GPIOS];
+ struct gpio_desc *gpios[NUM_GPIOS];
};
static bool nowayout = WATCHDOG_NOWAYOUT;
@@ -43,9 +43,9 @@ static unsigned int a21_wdt_get_bootstatus(struct a21_wdt_drv *drv)
{
int reset = 0;
- reset |= gpio_get_value(drv->gpios[GPIO_WD_RST0]) ? (1 << 0) : 0;
- reset |= gpio_get_value(drv->gpios[GPIO_WD_RST1]) ? (1 << 1) : 0;
- reset |= gpio_get_value(drv->gpios[GPIO_WD_RST2]) ? (1 << 2) : 0;
+ reset |= gpiod_get_value(drv->gpios[GPIO_WD_RST0]) ? (1 << 0) : 0;
+ reset |= gpiod_get_value(drv->gpios[GPIO_WD_RST1]) ? (1 << 1) : 0;
+ reset |= gpiod_get_value(drv->gpios[GPIO_WD_RST2]) ? (1 << 2) : 0;
return reset;
}
@@ -54,7 +54,7 @@ static int a21_wdt_start(struct watchdog_device *wdt)
{
struct a21_wdt_drv *drv = watchdog_get_drvdata(wdt);
- gpio_set_value(drv->gpios[GPIO_WD_ENAB], 1);
+ gpiod_set_value(drv->gpios[GPIO_WD_ENAB], 1);
return 0;
}
@@ -63,7 +63,7 @@ static int a21_wdt_stop(struct watchdog_device *wdt)
{
struct a21_wdt_drv *drv = watchdog_get_drvdata(wdt);
- gpio_set_value(drv->gpios[GPIO_WD_ENAB], 0);
+ gpiod_set_value(drv->gpios[GPIO_WD_ENAB], 0);
return 0;
}
@@ -72,9 +72,9 @@ static int a21_wdt_ping(struct watchdog_device *wdt)
{
struct a21_wdt_drv *drv = watchdog_get_drvdata(wdt);
- gpio_set_value(drv->gpios[GPIO_WD_TRIG], 0);
+ gpiod_set_value(drv->gpios[GPIO_WD_TRIG], 0);
ndelay(10);
- gpio_set_value(drv->gpios[GPIO_WD_TRIG], 1);
+ gpiod_set_value(drv->gpios[GPIO_WD_TRIG], 1);
return 0;
}
@@ -96,9 +96,9 @@ static int a21_wdt_set_timeout(struct watchdog_device *wdt,
}
if (timeout == 1)
- gpio_set_value(drv->gpios[GPIO_WD_FAST], 1);
+ gpiod_set_value(drv->gpios[GPIO_WD_FAST], 1);
else
- gpio_set_value(drv->gpios[GPIO_WD_FAST], 0);
+ gpiod_set_value(drv->gpios[GPIO_WD_FAST], 0);
wdt->timeout = timeout;
@@ -127,7 +127,6 @@ static struct watchdog_device a21_wdt = {
static int a21_wdt_probe(struct platform_device *pdev)
{
- struct device_node *node;
struct a21_wdt_drv *drv;
unsigned int reset = 0;
int num_gpios;
@@ -138,40 +137,40 @@ static int a21_wdt_probe(struct platform_device *pdev)
if (!drv)
return -ENOMEM;
- /* Fill GPIO pin array */
- node = pdev->dev.of_node;
-
- num_gpios = of_gpio_count(node);
+ num_gpios = gpiod_count(&pdev->dev, NULL);
if (num_gpios != NUM_GPIOS) {
dev_err(&pdev->dev, "gpios DT property wrong, got %d want %d",
num_gpios, NUM_GPIOS);
return -ENODEV;
}
- for (i = 0; i < num_gpios; i++) {
- int val;
-
- val = of_get_gpio(node, i);
- if (val < 0)
- return val;
-
- drv->gpios[i] = val;
- }
-
/* Request the used GPIOs */
for (i = 0; i < num_gpios; i++) {
- ret = devm_gpio_request(&pdev->dev, drv->gpios[i],
- "MEN A21 Watchdog");
- if (ret)
- return ret;
+ enum gpiod_flags gflags;
if (i < GPIO_WD_RST0)
- ret = gpio_direction_output(drv->gpios[i],
- gpio_get_value(drv->gpios[i]));
- else /* GPIO_WD_RST[0..2] are inputs */
- ret = gpio_direction_input(drv->gpios[i]);
- if (ret)
+ gflags = GPIOD_ASIS;
+ else
+ gflags = GPIOD_IN;
+ drv->gpios[i] = devm_gpiod_get_index(&pdev->dev, NULL, i,
+ gflags);
+ if (IS_ERR(drv->gpios[i])) {
+ ret = PTR_ERR(drv->gpios[i]);
return ret;
+ }
+
+ gpiod_set_consumer_name(drv->gpios[i], "MEN A21 Watchdog");
+
+ /*
+ * Retrieve the initial value from the GPIOs that should be
+ * output, then set up the line as output with that value.
+ */
+ if (i < GPIO_WD_RST0) {
+ int val;
+
+ val = gpiod_get_value(drv->gpios[i]);
+ gpiod_direction_output(drv->gpios[i], val);
+ }
}
watchdog_init_timeout(&a21_wdt, 30, &pdev->dev);
@@ -207,7 +206,7 @@ static void a21_wdt_shutdown(struct platform_device *pdev)
{
struct a21_wdt_drv *drv = dev_get_drvdata(&pdev->dev);
- gpio_set_value(drv->gpios[GPIO_WD_ENAB], 0);
+ gpiod_set_value(drv->gpios[GPIO_WD_ENAB], 0);
}
static const struct of_device_id a21_wdt_ids[] = {
--
2.19.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] watchdog: mena21_wdt: Convert to GPIO descriptors
2018-12-02 11:04 [PATCH] watchdog: mena21_wdt: Convert to GPIO descriptors Linus Walleij
@ 2018-12-03 8:12 ` Johannes Thumshirn
2018-12-05 12:21 ` Linus Walleij
2018-12-15 16:28 ` Guenter Roeck
1 sibling, 1 reply; 4+ messages in thread
From: Johannes Thumshirn @ 2018-12-03 8:12 UTC (permalink / raw)
To: Linus Walleij, Wim Van Sebroeck, Guenter Roeck
Cc: linux-watchdog, Johannes Thumshirn, Johannes Thumshirn
On 02/12/2018 12:04, Linus Walleij wrote:
> This drops the old OF API use to look up global GPIO
> numbers and replace it with the GPIO descriptor API.
>
> Cc: Johannes Thumshirn <jthumshirn@suse.de>
> Cc: Johannes Thumshirn <morbidrsa@gmail.com>
> Cc: Johannes Thumshirn <jth@kernel.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
I guess I have to many E-Mail addresses...
Anyways,
Acked-by: Johannes Thumshirn <jthumshirn@suse.de>
--
Johannes Thumshirn SUSE Labs Filesystems
jthumshirn@suse.de +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] watchdog: mena21_wdt: Convert to GPIO descriptors
2018-12-03 8:12 ` Johannes Thumshirn
@ 2018-12-05 12:21 ` Linus Walleij
0 siblings, 0 replies; 4+ messages in thread
From: Linus Walleij @ 2018-12-05 12:21 UTC (permalink / raw)
To: Johannes Thumshirn
Cc: Wim Van Sebroeck, Guenter Roeck, LINUXWATCHDOG,
Johannes Thumshirn, Johannes Thumshirn
On Mon, Dec 3, 2018 at 9:12 AM Johannes Thumshirn <jthumshirn@suse.de> wrote:
> On 02/12/2018 12:04, Linus Walleij wrote:
> > This drops the old OF API use to look up global GPIO
> > numbers and replace it with the GPIO descriptor API.
> >
> > Cc: Johannes Thumshirn <jthumshirn@suse.de>
> > Cc: Johannes Thumshirn <morbidrsa@gmail.com>
> > Cc: Johannes Thumshirn <jth@kernel.org>
> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> > ---
>
> I guess I have to many E-Mail addresses...
Haha :D
I just found several in the MAINTAINERS file so I got
confused and just added all of them.
Thanks!
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: watchdog: mena21_wdt: Convert to GPIO descriptors
2018-12-02 11:04 [PATCH] watchdog: mena21_wdt: Convert to GPIO descriptors Linus Walleij
2018-12-03 8:12 ` Johannes Thumshirn
@ 2018-12-15 16:28 ` Guenter Roeck
1 sibling, 0 replies; 4+ messages in thread
From: Guenter Roeck @ 2018-12-15 16:28 UTC (permalink / raw)
To: Linus Walleij
Cc: Wim Van Sebroeck, linux-watchdog, Johannes Thumshirn,
Johannes Thumshirn, Johannes Thumshirn
On Sun, Dec 02, 2018 at 12:04:10PM +0100, Linus Walleij wrote:
> This drops the old OF API use to look up global GPIO
> numbers and replace it with the GPIO descriptor API.
>
> Cc: Johannes Thumshirn <jthumshirn@suse.de>
> Cc: Johannes Thumshirn <morbidrsa@gmail.com>
> Cc: Johannes Thumshirn <jth@kernel.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> Acked-by: Johannes Thumshirn <jthumshirn@suse.de>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
... though I do wonder if anyone tested this. Guess we'll see.
Guenter
> ---
> drivers/watchdog/mena21_wdt.c | 73 +++++++++++++++++------------------
> 1 file changed, 36 insertions(+), 37 deletions(-)
>
> diff --git a/drivers/watchdog/mena21_wdt.c b/drivers/watchdog/mena21_wdt.c
> index 0be7f50e8ff9..6db69883ece6 100644
> --- a/drivers/watchdog/mena21_wdt.c
> +++ b/drivers/watchdog/mena21_wdt.c
> @@ -13,10 +13,10 @@
> #include <linux/platform_device.h>
> #include <linux/watchdog.h>
> #include <linux/uaccess.h>
> -#include <linux/gpio.h>
> -#include <linux/of_gpio.h>
> +#include <linux/gpio/consumer.h>
> #include <linux/delay.h>
> #include <linux/bitops.h>
> +#include <linux/of.h>
>
> #define NUM_GPIOS 6
>
> @@ -31,7 +31,7 @@ enum a21_wdt_gpios {
>
> struct a21_wdt_drv {
> struct watchdog_device wdt;
> - unsigned gpios[NUM_GPIOS];
> + struct gpio_desc *gpios[NUM_GPIOS];
> };
>
> static bool nowayout = WATCHDOG_NOWAYOUT;
> @@ -43,9 +43,9 @@ static unsigned int a21_wdt_get_bootstatus(struct a21_wdt_drv *drv)
> {
> int reset = 0;
>
> - reset |= gpio_get_value(drv->gpios[GPIO_WD_RST0]) ? (1 << 0) : 0;
> - reset |= gpio_get_value(drv->gpios[GPIO_WD_RST1]) ? (1 << 1) : 0;
> - reset |= gpio_get_value(drv->gpios[GPIO_WD_RST2]) ? (1 << 2) : 0;
> + reset |= gpiod_get_value(drv->gpios[GPIO_WD_RST0]) ? (1 << 0) : 0;
> + reset |= gpiod_get_value(drv->gpios[GPIO_WD_RST1]) ? (1 << 1) : 0;
> + reset |= gpiod_get_value(drv->gpios[GPIO_WD_RST2]) ? (1 << 2) : 0;
>
> return reset;
> }
> @@ -54,7 +54,7 @@ static int a21_wdt_start(struct watchdog_device *wdt)
> {
> struct a21_wdt_drv *drv = watchdog_get_drvdata(wdt);
>
> - gpio_set_value(drv->gpios[GPIO_WD_ENAB], 1);
> + gpiod_set_value(drv->gpios[GPIO_WD_ENAB], 1);
>
> return 0;
> }
> @@ -63,7 +63,7 @@ static int a21_wdt_stop(struct watchdog_device *wdt)
> {
> struct a21_wdt_drv *drv = watchdog_get_drvdata(wdt);
>
> - gpio_set_value(drv->gpios[GPIO_WD_ENAB], 0);
> + gpiod_set_value(drv->gpios[GPIO_WD_ENAB], 0);
>
> return 0;
> }
> @@ -72,9 +72,9 @@ static int a21_wdt_ping(struct watchdog_device *wdt)
> {
> struct a21_wdt_drv *drv = watchdog_get_drvdata(wdt);
>
> - gpio_set_value(drv->gpios[GPIO_WD_TRIG], 0);
> + gpiod_set_value(drv->gpios[GPIO_WD_TRIG], 0);
> ndelay(10);
> - gpio_set_value(drv->gpios[GPIO_WD_TRIG], 1);
> + gpiod_set_value(drv->gpios[GPIO_WD_TRIG], 1);
>
> return 0;
> }
> @@ -96,9 +96,9 @@ static int a21_wdt_set_timeout(struct watchdog_device *wdt,
> }
>
> if (timeout == 1)
> - gpio_set_value(drv->gpios[GPIO_WD_FAST], 1);
> + gpiod_set_value(drv->gpios[GPIO_WD_FAST], 1);
> else
> - gpio_set_value(drv->gpios[GPIO_WD_FAST], 0);
> + gpiod_set_value(drv->gpios[GPIO_WD_FAST], 0);
>
> wdt->timeout = timeout;
>
> @@ -127,7 +127,6 @@ static struct watchdog_device a21_wdt = {
>
> static int a21_wdt_probe(struct platform_device *pdev)
> {
> - struct device_node *node;
> struct a21_wdt_drv *drv;
> unsigned int reset = 0;
> int num_gpios;
> @@ -138,40 +137,40 @@ static int a21_wdt_probe(struct platform_device *pdev)
> if (!drv)
> return -ENOMEM;
>
> - /* Fill GPIO pin array */
> - node = pdev->dev.of_node;
> -
> - num_gpios = of_gpio_count(node);
> + num_gpios = gpiod_count(&pdev->dev, NULL);
> if (num_gpios != NUM_GPIOS) {
> dev_err(&pdev->dev, "gpios DT property wrong, got %d want %d",
> num_gpios, NUM_GPIOS);
> return -ENODEV;
> }
>
> - for (i = 0; i < num_gpios; i++) {
> - int val;
> -
> - val = of_get_gpio(node, i);
> - if (val < 0)
> - return val;
> -
> - drv->gpios[i] = val;
> - }
> -
> /* Request the used GPIOs */
> for (i = 0; i < num_gpios; i++) {
> - ret = devm_gpio_request(&pdev->dev, drv->gpios[i],
> - "MEN A21 Watchdog");
> - if (ret)
> - return ret;
> + enum gpiod_flags gflags;
>
> if (i < GPIO_WD_RST0)
> - ret = gpio_direction_output(drv->gpios[i],
> - gpio_get_value(drv->gpios[i]));
> - else /* GPIO_WD_RST[0..2] are inputs */
> - ret = gpio_direction_input(drv->gpios[i]);
> - if (ret)
> + gflags = GPIOD_ASIS;
> + else
> + gflags = GPIOD_IN;
> + drv->gpios[i] = devm_gpiod_get_index(&pdev->dev, NULL, i,
> + gflags);
> + if (IS_ERR(drv->gpios[i])) {
> + ret = PTR_ERR(drv->gpios[i]);
> return ret;
> + }
> +
> + gpiod_set_consumer_name(drv->gpios[i], "MEN A21 Watchdog");
> +
> + /*
> + * Retrieve the initial value from the GPIOs that should be
> + * output, then set up the line as output with that value.
> + */
> + if (i < GPIO_WD_RST0) {
> + int val;
> +
> + val = gpiod_get_value(drv->gpios[i]);
> + gpiod_direction_output(drv->gpios[i], val);
> + }
> }
>
> watchdog_init_timeout(&a21_wdt, 30, &pdev->dev);
> @@ -207,7 +206,7 @@ static void a21_wdt_shutdown(struct platform_device *pdev)
> {
> struct a21_wdt_drv *drv = dev_get_drvdata(&pdev->dev);
>
> - gpio_set_value(drv->gpios[GPIO_WD_ENAB], 0);
> + gpiod_set_value(drv->gpios[GPIO_WD_ENAB], 0);
> }
>
> static const struct of_device_id a21_wdt_ids[] = {
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-12-15 16:28 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-02 11:04 [PATCH] watchdog: mena21_wdt: Convert to GPIO descriptors Linus Walleij
2018-12-03 8:12 ` Johannes Thumshirn
2018-12-05 12:21 ` Linus Walleij
2018-12-15 16:28 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).