* [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count
@ 2022-06-30 18:27 Kris Bahnsen
2022-06-30 18:55 ` Krzysztof Kozlowski
2022-06-30 18:56 ` Krzysztof Kozlowski
0 siblings, 2 replies; 5+ messages in thread
From: Kris Bahnsen @ 2022-06-30 18:27 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree, linux-kernel
Cc: Mark Featherston, Kris Bahnsen
Device-tree incorrectly used "ngpio" which caused the driver to
fallback to 32 ngpios.
This platform has 62 GPIO registers.
Signed-off-by: Kris Bahnsen <kris@embeddedTS.com>
---
arch/arm/boot/dts/imx6qdl-ts7970.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx6qdl-ts7970.dtsi b/arch/arm/boot/dts/imx6qdl-ts7970.dtsi
index fded07f370b3..d6ba4b2a60f6 100644
--- a/arch/arm/boot/dts/imx6qdl-ts7970.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-ts7970.dtsi
@@ -226,7 +226,7 @@
reg = <0x28>;
#gpio-cells = <2>;
gpio-controller;
- ngpio = <32>;
+ ngpios = <62>;
};
sgtl5000: codec@a {
base-commit: 03c765b0e3b4cb5063276b086c76f7a612856a9a
--
2.11.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count
2022-06-30 18:27 [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count Kris Bahnsen
@ 2022-06-30 18:55 ` Krzysztof Kozlowski
2022-06-30 18:56 ` Krzysztof Kozlowski
1 sibling, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-30 18:55 UTC (permalink / raw)
To: Kris Bahnsen, Rob Herring, Krzysztof Kozlowski, devicetree, linux-kernel
Cc: Mark Featherston
On 30/06/2022 20:27, Kris Bahnsen wrote:
> Device-tree incorrectly used "ngpio" which caused the driver to
> fallback to 32 ngpios.
>
> This platform has 62 GPIO registers.
>
> Signed-off-by: Kris Bahnsen <kris@embeddedTS.com>
You should add here Fixes tag.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count
2022-06-30 18:27 [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count Kris Bahnsen
2022-06-30 18:55 ` Krzysztof Kozlowski
@ 2022-06-30 18:56 ` Krzysztof Kozlowski
2022-06-30 19:11 ` Kris Bahnsen
1 sibling, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-30 18:56 UTC (permalink / raw)
To: Kris Bahnsen, Rob Herring, Krzysztof Kozlowski, devicetree, linux-kernel
Cc: Mark Featherston
On 30/06/2022 20:27, Kris Bahnsen wrote:
> Device-tree incorrectly used "ngpio" which caused the driver to
> fallback to 32 ngpios.
>
> This platform has 62 GPIO registers.
>
> Signed-off-by: Kris Bahnsen <kris@embeddedTS.com>
> ---
Ah and one more - for some reason you did not send it to maintainers, so
patch will be most likely ignored. Please use scripts/get_maintainer.pl
to CC relevant people.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count
2022-06-30 18:56 ` Krzysztof Kozlowski
@ 2022-06-30 19:11 ` Kris Bahnsen
2022-06-30 19:27 ` Krzysztof Kozlowski
0 siblings, 1 reply; 5+ messages in thread
From: Kris Bahnsen @ 2022-06-30 19:11 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
devicetree, linux-kernel
Cc: Mark Featherston
On Thu, 2022-06-30 at 20:56 +0200, Krzysztof Kozlowski wrote:
> On 30/06/2022 20:27, Kris Bahnsen wrote:
> > Device-tree incorrectly used "ngpio" which caused the driver to
> > fallback to 32 ngpios.
> >
> > This platform has 62 GPIO registers.
> >
> > Signed-off-by: Kris Bahnsen <kris@embeddedTS.com>
> > ---
>
>
> Ah and one more - for some reason you did not send it to maintainers, so
> patch will be most likely ignored. Please use scripts/get_maintainer.pl
> to CC relevant people.
I reviewed that output and erred on the side of not generating
too much noise and skipped folks/lists marked as general ARM/NXP
maintainers. I'll be sure to send to all in the future.
Forgot about Fixes: tag, I'll add that in a v2.
Thanks for the feedback.
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count
2022-06-30 19:11 ` Kris Bahnsen
@ 2022-06-30 19:27 ` Krzysztof Kozlowski
0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-30 19:27 UTC (permalink / raw)
To: kris, Rob Herring, Krzysztof Kozlowski, devicetree, linux-kernel
Cc: Mark Featherston
On 30/06/2022 21:11, Kris Bahnsen wrote:
> On Thu, 2022-06-30 at 20:56 +0200, Krzysztof Kozlowski wrote:
>> On 30/06/2022 20:27, Kris Bahnsen wrote:
>>> Device-tree incorrectly used "ngpio" which caused the driver to
>>> fallback to 32 ngpios.
>>>
>>> This platform has 62 GPIO registers.
>>>
>>> Signed-off-by: Kris Bahnsen <kris@embeddedTS.com>
>>> ---
>>
>>
>> Ah and one more - for some reason you did not send it to maintainers, so
>> patch will be most likely ignored. Please use scripts/get_maintainer.pl
>> to CC relevant people.
>
> I reviewed that output and erred on the side of not generating
> too much noise and skipped folks/lists marked as general ARM/NXP
> maintainers. I'll be sure to send to all in the future.
For small patchsets usually entire output of get_maintainers.pl is
correct. What's worth to skip are the entries coming from history
(--no-git-fallback).
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-06-30 19:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-30 18:27 [PATCH] ARM: dts: imx6qdl-ts7970: Fix ngpio typo and count Kris Bahnsen
2022-06-30 18:55 ` Krzysztof Kozlowski
2022-06-30 18:56 ` Krzysztof Kozlowski
2022-06-30 19:11 ` Kris Bahnsen
2022-06-30 19:27 ` Krzysztof Kozlowski
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).