| Message ID | 1435673759-11996-1-git-send-email-digetx@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Am Dienstag 30 Juni 2015, 17:15:50 schrieb Dmitry Osipenko: > Commit 72daceb9a10a ("net: rfkill: gpio: Add default GPIO driver mappings > for ACPI") removed possibility to request GPIO by table index for non-ACPI > platforms without changing it users. As result "shutdown" GPIO request > will fail if request for "reset" GPIO succeeded or "reset" will be > requested instead of "shutdown" if "reset" wasn't defined. Fix it by > making gpiod_lookup_table use con_id's instead of indexes. > > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > Fixes: 72daceb (net: rfkill: gpio: Add default GPIO driver mappings for > ACPI) Cc: <stable@vger.kernel.org> # v3.19+ > Acked-by: Alexandre Courbot <acourbot@nvidia.com> beside a small typo in the commit message (is users -> its users) this looks good now. I can confirm that the gpio's are all available now. Reviewed-by: Marc Dietrich <marvin24@gmx.de> > --- > > v2: s/GPIO_LOOKUP_IDX/GPIO_LOOKUP/, otherwise "shutdown" GPIO won't be > requested > > arch/arm/mach-tegra/board-paz00.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/mach-tegra/board-paz00.c > b/arch/arm/mach-tegra/board-paz00.c index fbe74c6..49d1110 100644 > --- a/arch/arm/mach-tegra/board-paz00.c > +++ b/arch/arm/mach-tegra/board-paz00.c > @@ -39,8 +39,8 @@ static struct platform_device wifi_rfkill_device = { > static struct gpiod_lookup_table wifi_gpio_lookup = { > .dev_id = "rfkill_gpio", > .table = { > - GPIO_LOOKUP_IDX("tegra-gpio", 25, NULL, 0, 0), > - GPIO_LOOKUP_IDX("tegra-gpio", 85, NULL, 1, 0), > + GPIO_LOOKUP("tegra-gpio", 25, "reset", 0), > + GPIO_LOOKUP("tegra-gpio", 85, "shutdown", 0), > { }, > }, > };
30.06.2015 17:15, Dmitry Osipenko ?????: > Commit 72daceb9a10a ("net: rfkill: gpio: Add default GPIO driver mappings > for ACPI") removed possibility to request GPIO by table index for non-ACPI > platforms without changing it users. As result "shutdown" GPIO request > will fail if request for "reset" GPIO succeeded or "reset" will be > requested instead of "shutdown" if "reset" wasn't defined. Fix it by > making gpiod_lookup_table use con_id's instead of indexes. > > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > Fixes: 72daceb (net: rfkill: gpio: Add default GPIO driver mappings for ACPI) > Cc: <stable@vger.kernel.org> # v3.19+ > Acked-by: Alexandre Courbot <acourbot@nvidia.com> > --- > > v2: s/GPIO_LOOKUP_IDX/GPIO_LOOKUP/, otherwise "shutdown" GPIO won't be requested > > arch/arm/mach-tegra/board-paz00.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/mach-tegra/board-paz00.c b/arch/arm/mach-tegra/board-paz00.c > index fbe74c6..49d1110 100644 > --- a/arch/arm/mach-tegra/board-paz00.c > +++ b/arch/arm/mach-tegra/board-paz00.c > @@ -39,8 +39,8 @@ static struct platform_device wifi_rfkill_device = { > static struct gpiod_lookup_table wifi_gpio_lookup = { > .dev_id = "rfkill_gpio", > .table = { > - GPIO_LOOKUP_IDX("tegra-gpio", 25, NULL, 0, 0), > - GPIO_LOOKUP_IDX("tegra-gpio", 85, NULL, 1, 0), > + GPIO_LOOKUP("tegra-gpio", 25, "reset", 0), > + GPIO_LOOKUP("tegra-gpio", 85, "shutdown", 0), > { }, > }, > }; > Ping
Am Montag, 7. September 2015, 12:21:37 schrieb Dmitry Osipenko: > 30.06.2015 17:15, Dmitry Osipenko ?????: > > Commit 72daceb9a10a ("net: rfkill: gpio: Add default GPIO driver mappings > > for ACPI") removed possibility to request GPIO by table index for non-ACPI > > platforms without changing it users. As result "shutdown" GPIO request > > will fail if request for "reset" GPIO succeeded or "reset" will be > > requested instead of "shutdown" if "reset" wasn't defined. Fix it by > > making gpiod_lookup_table use con_id's instead of indexes. > > > > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > > Fixes: 72daceb (net: rfkill: gpio: Add default GPIO driver mappings for > > ACPI) Cc: <stable@vger.kernel.org> # v3.19+ > > Acked-by: Alexandre Courbot <acourbot@nvidia.com> > > --- > > > > v2: s/GPIO_LOOKUP_IDX/GPIO_LOOKUP/, otherwise "shutdown" GPIO won't be > > requested> > > arch/arm/mach-tegra/board-paz00.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm/mach-tegra/board-paz00.c > > b/arch/arm/mach-tegra/board-paz00.c index fbe74c6..49d1110 100644 > > --- a/arch/arm/mach-tegra/board-paz00.c > > +++ b/arch/arm/mach-tegra/board-paz00.c > > @@ -39,8 +39,8 @@ static struct platform_device wifi_rfkill_device = { > > > > static struct gpiod_lookup_table wifi_gpio_lookup = { > > > > .dev_id = "rfkill_gpio", > > .table = { > > > > - GPIO_LOOKUP_IDX("tegra-gpio", 25, NULL, 0, 0), > > - GPIO_LOOKUP_IDX("tegra-gpio", 85, NULL, 1, 0), > > + GPIO_LOOKUP("tegra-gpio", 25, "reset", 0), > > + GPIO_LOOKUP("tegra-gpio", 85, "shutdown", 0), > > > > { }, > > > > }, > > > > }; > > Ping also Tested-by: Marc Dietrich <marvin24@gmx.de> I thing this should go through the tegra-soc tree. Thierry? Stephen? Marc
diff --git a/arch/arm/mach-tegra/board-paz00.c b/arch/arm/mach-tegra/board-paz00.c index fbe74c6..49d1110 100644 --- a/arch/arm/mach-tegra/board-paz00.c +++ b/arch/arm/mach-tegra/board-paz00.c @@ -39,8 +39,8 @@ static struct platform_device wifi_rfkill_device = { static struct gpiod_lookup_table wifi_gpio_lookup = { .dev_id = "rfkill_gpio", .table = { - GPIO_LOOKUP_IDX("tegra-gpio", 25, NULL, 0, 0), - GPIO_LOOKUP_IDX("tegra-gpio", 85, NULL, 1, 0), + GPIO_LOOKUP("tegra-gpio", 25, "reset", 0), + GPIO_LOOKUP("tegra-gpio", 85, "shutdown", 0), { }, }, };
Commit 72daceb9a10a ("net: rfkill: gpio: Add default GPIO driver mappings for ACPI") removed possibility to request GPIO by table index for non-ACPI platforms without changing it users. As result "shutdown" GPIO request will fail if request for "reset" GPIO succeeded or "reset" will be requested instead of "shutdown" if "reset" wasn't defined. Fix it by making gpiod_lookup_table use con_id's instead of indexes. Signed-off-by: Dmitry Osipenko <digetx@gmail.com> Fixes: 72daceb (net: rfkill: gpio: Add default GPIO driver mappings for ACPI) Cc: <stable@vger.kernel.org> # v3.19+ Acked-by: Alexandre Courbot <acourbot@nvidia.com> --- v2: s/GPIO_LOOKUP_IDX/GPIO_LOOKUP/, otherwise "shutdown" GPIO won't be requested arch/arm/mach-tegra/board-paz00.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)