LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
@ 2020-03-10 18:25 nick.hudson
2020-03-11 11:28 ` Nicolas Saenz Julienne
2020-03-11 21:24 ` Florian Fainelli
0 siblings, 2 replies; 8+ messages in thread
From: nick.hudson @ 2020-03-10 18:25 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Florian Fainelli, Ray Jui,
Scott Branden, bcm-kernel-feedback-list, Nicolas Saenz Julienne,
devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel
Cc: Nick Hudson, Nick Hudson
From: Nick Hudson <nick.hudson@gmx.co.uk>
Define the sdhci pinctrl state as "default" so it gets applied
correctly and to match all other RPis.
Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
Signed-off-by: Nick Hudson <skrll@netbsd.org>
---
arch/arm/boot/dts/bcm2835-rpi-zero-w.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts b/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts
index b75af21069f9..4c3f606e5b8d 100644
--- a/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts
+++ b/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts
@@ -112,6 +112,7 @@
&sdhci {
#address-cells = <1>;
#size-cells = <0>;
+ pinctrl-names = "default";
pinctrl-0 = <&emmc_gpio34 &gpclk2_gpio43>;
bus-width = <4>;
mmc-pwrseq = <&wifi_pwrseq>;
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
2020-03-10 18:25 [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name nick.hudson
@ 2020-03-11 11:28 ` Nicolas Saenz Julienne
2020-03-11 16:37 ` Florian Fainelli
2020-03-11 21:24 ` Florian Fainelli
1 sibling, 1 reply; 8+ messages in thread
From: Nicolas Saenz Julienne @ 2020-03-11 11:28 UTC (permalink / raw)
To: nick.hudson, Florian Fainelli
Cc: Nick Hudson, Rob Herring, Mark Rutland, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, devicetree, linux-rpi-kernel,
linux-arm-kernel, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 599 bytes --]
On Tue, 2020-03-10 at 18:25 +0000, nick.hudson@gmx.co.uk wrote:
> From: Nick Hudson <nick.hudson@gmx.co.uk>
>
> Define the sdhci pinctrl state as "default" so it gets applied
> correctly and to match all other RPis.
>
> Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
>
> Signed-off-by: Nick Hudson <skrll@netbsd.org>
I think this one has everything right. As a nitpick, there is no need to add a
space between the Fixes tag and the Signed-off-by tag, but it's OK as is.
Florian, can we channel this as a fix for v5.6 or are we too late?
Regards,
Nicolas
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
2020-03-11 11:28 ` Nicolas Saenz Julienne
@ 2020-03-11 16:37 ` Florian Fainelli
2020-03-11 16:39 ` Nicolas Saenz Julienne
0 siblings, 1 reply; 8+ messages in thread
From: Florian Fainelli @ 2020-03-11 16:37 UTC (permalink / raw)
To: Nicolas Saenz Julienne, nick.hudson
Cc: Nick Hudson, Rob Herring, Mark Rutland, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, devicetree, linux-rpi-kernel,
linux-arm-kernel, linux-kernel
On 3/11/20 4:28 AM, Nicolas Saenz Julienne wrote:
> On Tue, 2020-03-10 at 18:25 +0000, nick.hudson@gmx.co.uk wrote:
>> From: Nick Hudson <nick.hudson@gmx.co.uk>
>>
>> Define the sdhci pinctrl state as "default" so it gets applied
>> correctly and to match all other RPis.
>>
>> Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
>>
>> Signed-off-by: Nick Hudson <skrll@netbsd.org>
>
> I think this one has everything right. As a nitpick, there is no need to add a
> space between the Fixes tag and the Signed-off-by tag, but it's OK as is.
>
> Florian, can we channel this as a fix for v5.6 or are we too late?
We can try, let me queue this today.
--
Florian
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
2020-03-11 16:37 ` Florian Fainelli
@ 2020-03-11 16:39 ` Nicolas Saenz Julienne
0 siblings, 0 replies; 8+ messages in thread
From: Nicolas Saenz Julienne @ 2020-03-11 16:39 UTC (permalink / raw)
To: Florian Fainelli, nick.hudson
Cc: Nick Hudson, Mark Rutland, Scott Branden, devicetree, Ray Jui,
linux-kernel, Rob Herring, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel
[-- Attachment #1: Type: text/plain, Size: 916 bytes --]
On Wed, 2020-03-11 at 09:37 -0700, Florian Fainelli wrote:
> On 3/11/20 4:28 AM, Nicolas Saenz Julienne wrote:
> > On Tue, 2020-03-10 at 18:25 +0000, nick.hudson@gmx.co.uk wrote:
> > > From: Nick Hudson <nick.hudson@gmx.co.uk>
> > >
> > > Define the sdhci pinctrl state as "default" so it gets applied
> > > correctly and to match all other RPis.
> > >
> > > Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
> > >
> > > Signed-off-by: Nick Hudson <skrll@netbsd.org>
> >
> > I think this one has everything right. As a nitpick, there is no need to add
> > a
> > space between the Fixes tag and the Signed-off-by tag, but it's OK as is.
> >
> > Florian, can we channel this as a fix for v5.6 or are we too late?
>
> We can try, let me queue this today.
Cool, in that case you can add my:
Acked-by: Nicolas Saenz Julienne <nsaenzjulienne@suse.de>
Regards,
Nicolas
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
2020-03-10 18:25 [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name nick.hudson
2020-03-11 11:28 ` Nicolas Saenz Julienne
@ 2020-03-11 21:24 ` Florian Fainelli
2020-03-11 21:26 ` Florian Fainelli
1 sibling, 1 reply; 8+ messages in thread
From: Florian Fainelli @ 2020-03-11 21:24 UTC (permalink / raw)
To: bcm-kernel-feedback-list, nick.hudson, Rob Herring, Mark Rutland,
Ray Jui, Scott Branden, Nicolas Saenz Julienne, devicetree,
linux-rpi-kernel, linux-arm-kernel, linux-kernel
Cc: Nick Hudson
On Tue, 10 Mar 2020 18:25:37 +0000, nick.hudson@gmx.co.uk wrote:
> From: Nick Hudson <nick.hudson@gmx.co.uk>
>
> Define the sdhci pinctrl state as "default" so it gets applied
> correctly and to match all other RPis.
>
> Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
>
> Signed-off-by: Nick Hudson <skrll@netbsd.org>
> ---
Applied to devicetree/fixes, thanks!
--
Florian
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
2020-03-11 21:24 ` Florian Fainelli
@ 2020-03-11 21:26 ` Florian Fainelli
0 siblings, 0 replies; 8+ messages in thread
From: Florian Fainelli @ 2020-03-11 21:26 UTC (permalink / raw)
To: bcm-kernel-feedback-list, nick.hudson, Rob Herring, Mark Rutland,
Ray Jui, Scott Branden, Nicolas Saenz Julienne, devicetree,
linux-rpi-kernel, linux-arm-kernel, linux-kernel
Cc: Nick Hudson
On 3/11/20 2:24 PM, Florian Fainelli wrote:
> On Tue, 10 Mar 2020 18:25:37 +0000, nick.hudson@gmx.co.uk wrote:
>> From: Nick Hudson <nick.hudson@gmx.co.uk>
>>
>> Define the sdhci pinctrl state as "default" so it gets applied
>> correctly and to match all other RPis.
>>
>> Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
>>
>> Signed-off-by: Nick Hudson <skrll@netbsd.org>
>> ---
>
> Applied to devicetree/fixes, thanks!
Actually, no, taken back, my pre-commit checks failed with:
Missing Signed-off-by from author.
Your From and Signed-off-by are different, can you resubmit?
--
Florian
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
2020-03-12 9:03 Nick Hudson
@ 2020-03-12 20:06 ` Florian Fainelli
0 siblings, 0 replies; 8+ messages in thread
From: Florian Fainelli @ 2020-03-12 20:06 UTC (permalink / raw)
To: bcm-kernel-feedback-list, Nick Hudson, Rob Herring, Mark Rutland,
Nicolas Saenz Julienne, Ray Jui, Scott Branden, devicetree,
linux-rpi-kernel, linux-arm-kernel, linux-kernel
On Thu, 12 Mar 2020 09:03:45 +0000, Nick Hudson <skrll@netbsd.org> wrote:
> Define the sdhci pinctrl state as "default" so it gets applied
> correctly and to match all other RPis.
>
> Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
> Signed-off-by: Nick Hudson <skrll@netbsd.org>
> ---
Applied to devicetree/fixes, thanks!
--
Florian
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name
@ 2020-03-12 9:03 Nick Hudson
2020-03-12 20:06 ` Florian Fainelli
0 siblings, 1 reply; 8+ messages in thread
From: Nick Hudson @ 2020-03-12 9:03 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Nicolas Saenz Julienne,
Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, devicetree, linux-rpi-kernel,
linux-arm-kernel, linux-kernel
Cc: Nick Hudson
Define the sdhci pinctrl state as "default" so it gets applied
correctly and to match all other RPis.
Fixes: 2c7c040c73e9 ("ARM: dts: bcm2835: Add Raspberry Pi Zero W")
Signed-off-by: Nick Hudson <skrll@netbsd.org>
---
arch/arm/boot/dts/bcm2835-rpi-zero-w.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts b/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts
index b75af21069f9..4c3f606e5b8d 100644
--- a/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts
+++ b/arch/arm/boot/dts/bcm2835-rpi-zero-w.dts
@@ -112,6 +112,7 @@
&sdhci {
#address-cells = <1>;
#size-cells = <0>;
+ pinctrl-names = "default";
pinctrl-0 = <&emmc_gpio34 &gpclk2_gpio43>;
bus-width = <4>;
mmc-pwrseq = <&wifi_pwrseq>;
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-03-12 20:07 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-10 18:25 [PATCH] ARM: bcm2835-rpi-zero-w: Add missing pinctrl name nick.hudson
2020-03-11 11:28 ` Nicolas Saenz Julienne
2020-03-11 16:37 ` Florian Fainelli
2020-03-11 16:39 ` Nicolas Saenz Julienne
2020-03-11 21:24 ` Florian Fainelli
2020-03-11 21:26 ` Florian Fainelli
2020-03-12 9:03 Nick Hudson
2020-03-12 20:06 ` Florian Fainelli
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).