* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419
@ 2017-02-18 0:32 Ben Hutchings
2017-02-20 12:59 ` Thomas Petazzoni
0 siblings, 1 reply; 7+ messages in thread
From: Ben Hutchings @ 2017-02-18 0:32 UTC (permalink / raw)
To: linux-arm-kernel
The old board code for the TS419 assigns MPP pins 15 and 16 as SATA
activity signals (and none as SATA presence signals). Currently the
device tree assigns the SoC's default pinmux groups for SATA, which
conflict with the second Ethernet port.
Reported-by: gmbh at gazeta.pl
Tested-by: gmbh at gazeta.pl
References: https://bugs.debian.org/855017
Cc: stable at vger.kernel.org # 3.15+
Fixes: 934b524b3f49 ("ARM: Kirkwood: Add DT description of QNAP 419")
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
---
arch/arm/boot/dts/kirkwood-ts419.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/kirkwood-ts419.dtsi b/arch/arm/boot/dts/kirkwood-ts419.dtsi
index 02bd53762705..532506cb0f4a 100644
--- a/arch/arm/boot/dts/kirkwood-ts419.dtsi
+++ b/arch/arm/boot/dts/kirkwood-ts419.dtsi
@@ -73,3 +73,11 @@
phy-handle = <ðphy1>;
};
};
+
+&pmx_sata0 {
+ marvell,pins = "mpp15";
+};
+
+&pmx_sata1 {
+ marvell,pins = "mpp16";
+};
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 811 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170218/7d56af58/attachment.sig>
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 2017-02-18 0:32 [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 Ben Hutchings @ 2017-02-20 12:59 ` Thomas Petazzoni 2017-02-20 16:40 ` Ben Hutchings 0 siblings, 1 reply; 7+ messages in thread From: Thomas Petazzoni @ 2017-02-20 12:59 UTC (permalink / raw) To: linux-arm-kernel Hello, Adding in Cc all the maintainers of the Kirkwood platforms. On Sat, 18 Feb 2017 00:32:51 +0000, Ben Hutchings wrote: > +&pmx_sata0 { > + marvell,pins = "mpp15"; > +}; > + > +&pmx_sata1 { > + marvell,pins = "mpp16"; > +}; This is not only muxing mpp15 as sata0 and mpp16 as sata1, but also removes the muxing of sata0/sata1 pins described in kirkwood-6282.dtsi: pmx_sata0: pmx-sata0 { marvell,pins = "mpp5", "mpp21", "mpp23"; marvell,function = "sata0"; }; pmx_sata1: pmx-sata1 { marvell,pins = "mpp4", "mpp20", "mpp22"; marvell,function = "sata1"; }; So it means that MPP 4, 5, 20, 21, 22 and 23 will no longer be muxed as sata0/sata1. Is this really what you want? Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 2017-02-20 12:59 ` Thomas Petazzoni @ 2017-02-20 16:40 ` Ben Hutchings 2017-02-20 16:50 ` Thomas Petazzoni 0 siblings, 1 reply; 7+ messages in thread From: Ben Hutchings @ 2017-02-20 16:40 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2017-02-20 at 13:59 +0100, Thomas Petazzoni wrote: > Hello, > > Adding in Cc all the maintainers of the Kirkwood platforms. > > On Sat, 18 Feb 2017 00:32:51 +0000, Ben Hutchings wrote: > > > +&pmx_sata0 { > > + marvell,pins = "mpp15"; > > +}; > > + > > +&pmx_sata1 { > > + marvell,pins = "mpp16"; > > +}; > > This is not only muxing mpp15 as sata0 and mpp16 as sata1, but also > removes the muxing of sata0/sata1 pins described in kirkwood-6282.dtsi: > > ????????????????????????pmx_sata0: pmx-sata0 { > ????????????????????????????????marvell,pins = "mpp5", "mpp21", "mpp23"; > ????????????????????????????????marvell,function = "sata0"; > ????????????????????????}; > ????????????????????????pmx_sata1: pmx-sata1 { > ????????????????????????????????marvell,pins = "mpp4", "mpp20", "mpp22"; > ????????????????????????????????marvell,function = "sata1"; > ????????????????????????}; > > So it means that MPP 4, 5, 20, 21, 22 and 23 will no longer be muxed as > sata0/sata1. Is this really what you want? That is precisely what I intended. 20-23 are used by the second Ethernet port. The old board code doesn't assign 4 or 5 at all. Ben. -- Ben Hutchings If at first you don't succeed, you're doing about average. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: This is a digitally signed message part URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170220/13dad357/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 2017-02-20 16:40 ` Ben Hutchings @ 2017-02-20 16:50 ` Thomas Petazzoni 2017-02-20 17:22 ` Ben Hutchings 0 siblings, 1 reply; 7+ messages in thread From: Thomas Petazzoni @ 2017-02-20 16:50 UTC (permalink / raw) To: linux-arm-kernel Hello, On Mon, 20 Feb 2017 16:40:25 +0000, Ben Hutchings wrote: > That is precisely what I intended. 20-23 are used by the second > Ethernet port. The old board code doesn't assign 4 or 5 at all. Then I believe it would be more explicit to have separate pin muxing configurations for SATA on this board. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 2017-02-20 16:50 ` Thomas Petazzoni @ 2017-02-20 17:22 ` Ben Hutchings 2017-02-20 20:31 ` Thomas Petazzoni 2017-03-07 17:16 ` Gregory CLEMENT 0 siblings, 2 replies; 7+ messages in thread From: Ben Hutchings @ 2017-02-20 17:22 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2017-02-20 at 17:50 +0100, Thomas Petazzoni wrote: > Hello, > > On Mon, 20 Feb 2017 16:40:25 +0000, Ben Hutchings wrote: > > > That is precisely what I intended.??20-23 are used by the second > > Ethernet port.??The old board code doesn't assign 4 or 5 at all. > > Then I believe it would be more explicit to have separate pin muxing > configurations for SATA on this board. You mean, define additional pinmux nodes and override the pinctrl-0 property of &sata? More like this: --- a/arch/arm/boot/dts/kirkwood-ts419.dtsi +++ b/arch/arm/boot/dts/kirkwood-ts419.dtsi @@ -73,3 +73,19 @@ phy-handle = <ðphy1>; }; }; + +&pinctrl { + pmx_sata0_ts419: pmx-sata0-ts419 { + marvell,pins = "mpp15"; + marvell,function = "sata0"; + }; + + pmx_sata1_ts419: pmx-sata1-ts419 { + marvell,pins = "mpp16"; + marvell,function = "sata1"; + }; +}; + +&sata { + pinctrl-0 = <&pmx_sata0_ts419 &pmx_sata1_ts419>; +}; --- END --- Ben. -- Ben Hutchings If at first you don't succeed, you're doing about average. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: This is a digitally signed message part URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170220/f795a5b5/attachment-0001.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 2017-02-20 17:22 ` Ben Hutchings @ 2017-02-20 20:31 ` Thomas Petazzoni 2017-03-07 17:16 ` Gregory CLEMENT 1 sibling, 0 replies; 7+ messages in thread From: Thomas Petazzoni @ 2017-02-20 20:31 UTC (permalink / raw) To: linux-arm-kernel Hello, On Mon, 20 Feb 2017 17:22:38 +0000, Ben Hutchings wrote: > You mean, define additional pinmux nodes and override the pinctrl-0 > property of &sata? More like this: Yes, exactly. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 2017-02-20 17:22 ` Ben Hutchings 2017-02-20 20:31 ` Thomas Petazzoni @ 2017-03-07 17:16 ` Gregory CLEMENT 1 sibling, 0 replies; 7+ messages in thread From: Gregory CLEMENT @ 2017-03-07 17:16 UTC (permalink / raw) To: linux-arm-kernel Hi Ben, On lun., f?vr. 20 2017, Ben Hutchings <ben@decadent.org.uk> wrote: > On Mon, 2017-02-20 at 17:50 +0100, Thomas Petazzoni wrote: >> Hello, >> >> On Mon, 20 Feb 2017 16:40:25 +0000, Ben Hutchings wrote: >> >> > That is precisely what I intended.??20-23 are used by the second >> > Ethernet port.??The old board code doesn't assign 4 or 5 at all. >> >> Then I believe it would be more explicit to have separate pin muxing >> configurations for SATA on this board. > > You mean, define additional pinmux nodes and override the pinctrl-0 > property of &sata? More like this: > > --- a/arch/arm/boot/dts/kirkwood-ts419.dtsi > +++ b/arch/arm/boot/dts/kirkwood-ts419.dtsi > @@ -73,3 +73,19 @@ > phy-handle = <ðphy1>; > }; > }; > + > +&pinctrl { > + pmx_sata0_ts419: pmx-sata0-ts419 { > + marvell,pins = "mpp15"; > + marvell,function = "sata0"; > + }; > + > + pmx_sata1_ts419: pmx-sata1-ts419 { > + marvell,pins = "mpp16"; > + marvell,function = "sata1"; > + }; > +}; > + > +&sata { > + pinctrl-0 = <&pmx_sata0_ts419 &pmx_sata1_ts419>; > +}; > --- END --- If you send a new version of your patch, then I will be able to apply it on mvebu/dt. Thanks, Gregory > > Ben. > > -- > Ben Hutchings > If at first you don't succeed, you're doing about average. > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-03-07 17:16 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-02-18 0:32 [PATCH] ARM: dts: kirkwood: Fix SATA pinmux-ing for TS419 Ben Hutchings 2017-02-20 12:59 ` Thomas Petazzoni 2017-02-20 16:40 ` Ben Hutchings 2017-02-20 16:50 ` Thomas Petazzoni 2017-02-20 17:22 ` Ben Hutchings 2017-02-20 20:31 ` Thomas Petazzoni 2017-03-07 17:16 ` Gregory CLEMENT
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.