* [PATCH RESEND] gpio: generic: Use platform_device_id->driver_data field for driver flags
@ 2014-03-16 5:10 Alexander Shiyan
2014-03-17 6:52 ` Alexandre Courbot
2014-03-17 13:58 ` Linus Walleij
0 siblings, 2 replies; 3+ messages in thread
From: Alexander Shiyan @ 2014-03-16 5:10 UTC (permalink / raw)
To: linux-gpio
Cc: Linus Walleij, Alexandre Courbot, Anton Vorontsov,
Alexander Shiyan
Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
drivers/gpio/gpio-generic.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/drivers/gpio/gpio-generic.c b/drivers/gpio/gpio-generic.c
index d815dd2..b5dff9e 100644
--- a/drivers/gpio/gpio-generic.c
+++ b/drivers/gpio/gpio-generic.c
@@ -488,7 +488,7 @@ static int bgpio_pdev_probe(struct platform_device *pdev)
void __iomem *dirout;
void __iomem *dirin;
unsigned long sz;
- unsigned long flags = 0;
+ unsigned long flags = pdev->id_entry->driver_data;
int err;
struct bgpio_chip *bgc;
struct bgpio_pdata *pdata = dev_get_platdata(dev);
@@ -519,9 +519,6 @@ static int bgpio_pdev_probe(struct platform_device *pdev)
if (err)
return err;
- if (!strcmp(platform_get_device_id(pdev)->name, "basic-mmio-gpio-be"))
- flags |= BGPIOF_BIG_ENDIAN;
-
bgc = devm_kzalloc(&pdev->dev, sizeof(*bgc), GFP_KERNEL);
if (!bgc)
return -ENOMEM;
@@ -551,9 +548,14 @@ static int bgpio_pdev_remove(struct platform_device *pdev)
}
static const struct platform_device_id bgpio_id_table[] = {
- { "basic-mmio-gpio", },
- { "basic-mmio-gpio-be", },
- {},
+ {
+ .name = "basic-mmio-gpio",
+ .driver_data = 0,
+ }, {
+ .name = "basic-mmio-gpio-be",
+ .driver_data = BGPIOF_BIG_ENDIAN,
+ },
+ { }
};
MODULE_DEVICE_TABLE(platform, bgpio_id_table);
--
1.8.3.2
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH RESEND] gpio: generic: Use platform_device_id->driver_data field for driver flags
2014-03-16 5:10 [PATCH RESEND] gpio: generic: Use platform_device_id->driver_data field for driver flags Alexander Shiyan
@ 2014-03-17 6:52 ` Alexandre Courbot
2014-03-17 13:58 ` Linus Walleij
1 sibling, 0 replies; 3+ messages in thread
From: Alexandre Courbot @ 2014-03-17 6:52 UTC (permalink / raw)
To: Alexander Shiyan
Cc: linux-gpio@vger.kernel.org, Linus Walleij, Anton Vorontsov
On Sun, Mar 16, 2014 at 2:10 PM, Alexander Shiyan <shc_work@mail.ru> wrote:
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
> drivers/gpio/gpio-generic.c | 16 +++++++++-------
> 1 file changed, 9 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpio/gpio-generic.c b/drivers/gpio/gpio-generic.c
> index d815dd2..b5dff9e 100644
> --- a/drivers/gpio/gpio-generic.c
> +++ b/drivers/gpio/gpio-generic.c
> @@ -488,7 +488,7 @@ static int bgpio_pdev_probe(struct platform_device *pdev)
> void __iomem *dirout;
> void __iomem *dirin;
> unsigned long sz;
> - unsigned long flags = 0;
> + unsigned long flags = pdev->id_entry->driver_data;
> int err;
> struct bgpio_chip *bgc;
> struct bgpio_pdata *pdata = dev_get_platdata(dev);
> @@ -519,9 +519,6 @@ static int bgpio_pdev_probe(struct platform_device *pdev)
> if (err)
> return err;
>
> - if (!strcmp(platform_get_device_id(pdev)->name, "basic-mmio-gpio-be"))
> - flags |= BGPIOF_BIG_ENDIAN;
> -
> bgc = devm_kzalloc(&pdev->dev, sizeof(*bgc), GFP_KERNEL);
> if (!bgc)
> return -ENOMEM;
> @@ -551,9 +548,14 @@ static int bgpio_pdev_remove(struct platform_device *pdev)
> }
>
> static const struct platform_device_id bgpio_id_table[] = {
> - { "basic-mmio-gpio", },
> - { "basic-mmio-gpio-be", },
> - {},
> + {
> + .name = "basic-mmio-gpio",
> + .driver_data = 0,
> + }, {
> + .name = "basic-mmio-gpio-be",
> + .driver_data = BGPIOF_BIG_ENDIAN,
> + },
> + { }
> };
> MODULE_DEVICE_TABLE(platform, bgpio_id_table);
Agreed that this kind of setup ought to be data-oriented.
Reviewed-by: Alexandre Courbot <acourbot@nvidia.com>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH RESEND] gpio: generic: Use platform_device_id->driver_data field for driver flags
2014-03-16 5:10 [PATCH RESEND] gpio: generic: Use platform_device_id->driver_data field for driver flags Alexander Shiyan
2014-03-17 6:52 ` Alexandre Courbot
@ 2014-03-17 13:58 ` Linus Walleij
1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2014-03-17 13:58 UTC (permalink / raw)
To: Alexander Shiyan
Cc: linux-gpio@vger.kernel.org, Alexandre Courbot, Anton Vorontsov
On Sun, Mar 16, 2014 at 6:10 AM, Alexander Shiyan <shc_work@mail.ru> wrote:
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Thanks! Patch applied with Alexandre's ACK, sorry for missing
this patch in the past then dunno what has happened :-/
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-03-17 13:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-16 5:10 [PATCH RESEND] gpio: generic: Use platform_device_id->driver_data field for driver flags Alexander Shiyan
2014-03-17 6:52 ` Alexandre Courbot
2014-03-17 13:58 ` Linus Walleij
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).