* [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node
@ 2015-02-10 8:48 Hans Holmberg
2015-02-10 8:50 ` Alexandre Courbot
2015-02-18 17:15 ` Linus Walleij
0 siblings, 2 replies; 6+ messages in thread
From: Hans Holmberg @ 2015-02-10 8:48 UTC (permalink / raw)
To: Linus Walleij, Alexandre Courbot
Cc: linux-gpio, linux-kernel, robert.jarzmik, cochran, Daniel Mack,
tylerwhall, Hans Holmberg
The change:
7b8792bbdffdff3abda704f89c6a45ea97afdc62
gpiolib: of: Correct error handling in of_get_named_gpiod_flags
assumed that only one gpio-chip is registred per of-node.
Some drivers register more than one chip per of-node, so
adjust the matching function of_gpiochip_find_and_xlate to
not stop looking for chips if a node-match is found and
the translation fails.
Signed-off-by: Hans Holmberg <hans.holmberg@intel.com>
---
drivers/gpio/gpiolib-of.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/gpio/gpiolib-of.c b/drivers/gpio/gpiolib-of.c
index 08261f2..26645a8 100644
--- a/drivers/gpio/gpiolib-of.c
+++ b/drivers/gpio/gpiolib-of.c
@@ -46,12 +46,13 @@ static int of_gpiochip_find_and_xlate(struct gpio_chip *gc, void *data)
ret = gc->of_xlate(gc, &gg_data->gpiospec, gg_data->flags);
if (ret < 0) {
- /* We've found the gpio chip, but the translation failed.
- * Return true to stop looking and return the translation
- * error via out_gpio
+ /* We've found a gpio chip, but the translation failed.
+ * Store translation error in out_gpio.
+ * Return false to keep looking, as more than one gpio chip
+ * could be registered per of-node.
*/
gg_data->out_gpio = ERR_PTR(ret);
- return true;
+ return false;
}
gg_data->out_gpio = gpiochip_get_desc(gc, ret);
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node
2015-02-10 8:48 [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node Hans Holmberg
@ 2015-02-10 8:50 ` Alexandre Courbot
2015-02-10 17:08 ` Robert Jarzmik
2015-02-18 17:15 ` Linus Walleij
1 sibling, 1 reply; 6+ messages in thread
From: Alexandre Courbot @ 2015-02-10 8:50 UTC (permalink / raw)
To: Hans Holmberg
Cc: Linus Walleij, linux-gpio, Linux Kernel Mailing List,
Robert Jarzmik, Howard Cochran, Daniel Mack, Tyler Hall
On Tue, Feb 10, 2015 at 5:48 PM, Hans Holmberg <hans.holmberg@intel.com> wrote:
> The change:
>
> 7b8792bbdffdff3abda704f89c6a45ea97afdc62
> gpiolib: of: Correct error handling in of_get_named_gpiod_flags
>
> assumed that only one gpio-chip is registred per of-node.
> Some drivers register more than one chip per of-node, so
> adjust the matching function of_gpiochip_find_and_xlate to
> not stop looking for chips if a node-match is found and
> the translation fails.
>
> Signed-off-by: Hans Holmberg <hans.holmberg@intel.com>
Acked-by: Alexandre Courbot <acourbot@nvidia.com>
Thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node
2015-02-10 8:50 ` Alexandre Courbot
@ 2015-02-10 17:08 ` Robert Jarzmik
0 siblings, 0 replies; 6+ messages in thread
From: Robert Jarzmik @ 2015-02-10 17:08 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Hans Holmberg, Linus Walleij, linux-gpio,
Linux Kernel Mailing List, Howard Cochran, Daniel Mack,
Tyler Hall
Alexandre Courbot <gnurou@gmail.com> writes:
> On Tue, Feb 10, 2015 at 5:48 PM, Hans Holmberg <hans.holmberg@intel.com> wrote:
>> The change:
>>
>> 7b8792bbdffdff3abda704f89c6a45ea97afdc62
>> gpiolib: of: Correct error handling in of_get_named_gpiod_flags
>>
>> assumed that only one gpio-chip is registred per of-node.
>> Some drivers register more than one chip per of-node, so
>> adjust the matching function of_gpiochip_find_and_xlate to
>> not stop looking for chips if a node-match is found and
>> the translation fails.
>>
>> Signed-off-by: Hans Holmberg <hans.holmberg@intel.com>
>
> Acked-by: Alexandre Courbot <acourbot@nvidia.com>
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
(on arm pxa architecture)
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node
@ 2015-02-10 17:08 ` Robert Jarzmik
0 siblings, 0 replies; 6+ messages in thread
From: Robert Jarzmik @ 2015-02-10 17:08 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Hans Holmberg, Linus Walleij, linux-gpio,
Linux Kernel Mailing List, Howard Cochran, Daniel Mack,
Tyler Hall
Alexandre Courbot <gnurou@gmail.com> writes:
> On Tue, Feb 10, 2015 at 5:48 PM, Hans Holmberg <hans.holmberg@intel.com> wrote:
>> The change:
>>
>> 7b8792bbdffdff3abda704f89c6a45ea97afdc62
>> gpiolib: of: Correct error handling in of_get_named_gpiod_flags
>>
>> assumed that only one gpio-chip is registred per of-node.
>> Some drivers register more than one chip per of-node, so
>> adjust the matching function of_gpiochip_find_and_xlate to
>> not stop looking for chips if a node-match is found and
>> the translation fails.
>>
>> Signed-off-by: Hans Holmberg <hans.holmberg@intel.com>
>
> Acked-by: Alexandre Courbot <acourbot@nvidia.com>
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
(on arm pxa architecture)
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node
2015-02-10 17:08 ` Robert Jarzmik
(?)
@ 2015-02-10 20:30 ` Tyler Hall
-1 siblings, 0 replies; 6+ messages in thread
From: Tyler Hall @ 2015-02-10 20:30 UTC (permalink / raw)
To: Robert Jarzmik
Cc: Alexandre Courbot, Hans Holmberg, Linus Walleij, linux-gpio,
Linux Kernel Mailing List, Howard Cochran, Daniel Mack
On Tue, Feb 10, 2015 at 12:08 PM, Robert Jarzmik <robert.jarzmik@free.fr> wrote:
> Alexandre Courbot <gnurou@gmail.com> writes:
>
>> On Tue, Feb 10, 2015 at 5:48 PM, Hans Holmberg <hans.holmberg@intel.com> wrote:
>>> The change:
>>>
>>> 7b8792bbdffdff3abda704f89c6a45ea97afdc62
>>> gpiolib: of: Correct error handling in of_get_named_gpiod_flags
>>>
>>> assumed that only one gpio-chip is registred per of-node.
>>> Some drivers register more than one chip per of-node, so
>>> adjust the matching function of_gpiochip_find_and_xlate to
>>> not stop looking for chips if a node-match is found and
>>> the translation fails.
>>>
>>> Signed-off-by: Hans Holmberg <hans.holmberg@intel.com>
>>
>> Acked-by: Alexandre Courbot <acourbot@nvidia.com>
> Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
> (on arm pxa architecture)
Tested-by: Tyler Hall <tylerwhall@gmail.com>
With gpio-pxa (multiple chips per of node) and on arm-mvebu with
mvebu-gpio (one chip per of-node).
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node
2015-02-10 8:48 [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node Hans Holmberg
2015-02-10 8:50 ` Alexandre Courbot
@ 2015-02-18 17:15 ` Linus Walleij
1 sibling, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2015-02-18 17:15 UTC (permalink / raw)
To: Hans Holmberg
Cc: Alexandre Courbot, linux-gpio, linux-kernel, Robert Jarzmik,
cochran, Daniel Mack, tylerwhall
On Tue, Feb 10, 2015 at 9:48 AM, Hans Holmberg <hans.holmberg@intel.com> wrote:
> The change:
>
> 7b8792bbdffdff3abda704f89c6a45ea97afdc62
> gpiolib: of: Correct error handling in of_get_named_gpiod_flags
>
> assumed that only one gpio-chip is registred per of-node.
> Some drivers register more than one chip per of-node, so
> adjust the matching function of_gpiochip_find_and_xlate to
> not stop looking for chips if a node-match is found and
> the translation fails.
>
> Signed-off-by: Hans Holmberg <hans.holmberg@intel.com>
OK this patch applied for stable and pushed for testing
in -next.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-02-18 17:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-10 8:48 [PATCH] gpiolib: of: allow of_gpiochip_find_and_xlate to find more than one chip per node Hans Holmberg
2015-02-10 8:50 ` Alexandre Courbot
2015-02-10 17:08 ` Robert Jarzmik
2015-02-10 17:08 ` Robert Jarzmik
2015-02-10 20:30 ` Tyler Hall
2015-02-18 17:15 ` Linus Walleij
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.