| Message ID | 1456150130-2668-11-git-send-email-ldewangan@nvidia.com |
|---|---|
| State | New |
| Headers | show |
On Mon, 22 Feb 2016 19:37:59 +0530 Laxman Dewangan <ldewangan@nvidia.com> wrote: > Use devm_gpiochip_add_data() for GPIO registration. > > Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com> > Cc: Alban Bedel <albeu@free.fr> > --- > drivers/gpio/gpio-ath79.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpio/gpio-ath79.c b/drivers/gpio/gpio-ath79.c > index d13dd13..a6aad59 100644 > --- a/drivers/gpio/gpio-ath79.c > +++ b/drivers/gpio/gpio-ath79.c > @@ -182,7 +182,7 @@ static int ath79_gpio_probe(struct platform_device *pdev) > ctrl->chip.direction_output = ar934x_gpio_direction_output; > } > > - err = gpiochip_add_data(&ctrl->chip, ctrl); > + err = devm_gpiochip_add_data(&pdev->dev, &ctrl->chip, ctrl); > if (err) { > dev_err(&pdev->dev, > "cannot add AR71xx GPIO chip, error=%d", err); With the patches already applied to the devel branch a few more changes will be needed to remove the calls to gpiochip_remove() in the probe error path and the remove callback. Alban -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tuesday 23 February 2016 01:43 AM, Alban wrote: > On Mon, 22 Feb 2016 19:37:59 +0530 > Laxman Dewangan <ldewangan@nvidia.com> wrote: > >> Use devm_gpiochip_add_data() for GPIO registration. >> >> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com> >> Cc: Alban Bedel <albeu@free.fr> >> --- >> drivers/gpio/gpio-ath79.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpio/gpio-ath79.c b/drivers/gpio/gpio-ath79.c >> index d13dd13..a6aad59 100644 >> --- a/drivers/gpio/gpio-ath79.c >> +++ b/drivers/gpio/gpio-ath79.c >> @@ -182,7 +182,7 @@ static int ath79_gpio_probe(struct platform_device *pdev) >> ctrl->chip.direction_output = ar934x_gpio_direction_output; >> } >> >> - err = gpiochip_add_data(&ctrl->chip, ctrl); >> + err = devm_gpiochip_add_data(&pdev->dev, &ctrl->chip, ctrl); >> if (err) { >> dev_err(&pdev->dev, >> "cannot add AR71xx GPIO chip, error=%d", err); > With the patches already applied to the devel branch a few more changes > will be needed to remove the calls to gpiochip_remove() in the probe > error path and the remove callback. > OK, I will drop this patch when provide the git pointer to Linus. We can work on this later. -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/gpio/gpio-ath79.c b/drivers/gpio/gpio-ath79.c index d13dd13..a6aad59 100644 --- a/drivers/gpio/gpio-ath79.c +++ b/drivers/gpio/gpio-ath79.c @@ -182,7 +182,7 @@ static int ath79_gpio_probe(struct platform_device *pdev) ctrl->chip.direction_output = ar934x_gpio_direction_output; } - err = gpiochip_add_data(&ctrl->chip, ctrl); + err = devm_gpiochip_add_data(&pdev->dev, &ctrl->chip, ctrl); if (err) { dev_err(&pdev->dev, "cannot add AR71xx GPIO chip, error=%d", err);
Use devm_gpiochip_add_data() for GPIO registration. Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com> Cc: Alban Bedel <albeu@free.fr> --- drivers/gpio/gpio-ath79.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)