diff mbox series

[net] arm64: dts: fsl-ls1028a-kontron-sl28: specify in-band mode for ENETC

Message ID 20201101131905.8316-1-michael@walle.cc (mailing list archive)
State New, archived
Headers show
Series [net] arm64: dts: fsl-ls1028a-kontron-sl28: specify in-band mode for ENETC | expand

Commit Message

Michael Walle Nov. 1, 2020, 1:19 p.m. UTC
Since commit 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") the
network port of the Kontron sl28 board is broken. After the migration to
phylink the device tree has to specify the in-band-mode property. Add
it.

Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX")
Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Michael Walle <michael@walle.cc>
---
 arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Michael Walle Nov. 1, 2020, 1:43 p.m. UTC | #1
[+Ioana; the reference commit has a typo in her email address]

Am 2020-11-01 14:19, schrieb Michael Walle:
> Since commit 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") 
> the
> network port of the Kontron sl28 board is broken. After the migration 
> to
> phylink the device tree has to specify the in-band-mode property. Add
> it.
> 
> Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX")
> Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> Signed-off-by: Michael Walle <michael@walle.cc>
> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git
> a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> index f46eb47cfa4d..8161dd237971 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> @@ -75,6 +75,7 @@
>  &enetc_port0 {
>  	phy-handle = <&phy0>;
>  	phy-connection-type = "sgmii";
> +	managed = "in-band-status";
>  	status = "okay";
> 
>  	mdio {
Vladimir Oltean Nov. 1, 2020, 4:32 p.m. UTC | #2
On Sun, Nov 01, 2020 at 02:19:05PM +0100, Michael Walle wrote:
> Since commit 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") the
> network port of the Kontron sl28 board is broken. After the migration to
> phylink the device tree has to specify the in-band-mode property. Add
> it.
> 
> Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX")
> Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> Signed-off-by: Michael Walle <michael@walle.cc>
> ---

Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>

>  arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> index f46eb47cfa4d..8161dd237971 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
> @@ -75,6 +75,7 @@
>  &enetc_port0 {
>  	phy-handle = <&phy0>;
>  	phy-connection-type = "sgmii";
> +	managed = "in-band-status";
>  	status = "okay";
>  
>  	mdio {
> -- 
> 2.20.1
>
Michael Walle Nov. 8, 2020, 6 p.m. UTC | #3
Hi,

Am 2020-11-01 14:19, schrieb Michael Walle:
> Since commit 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") 
> the
> network port of the Kontron sl28 board is broken. After the migration 
> to
> phylink the device tree has to specify the in-band-mode property. Add
> it.
> 
> Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX")
> Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> Signed-off-by: Michael Walle <michael@walle.cc>

Ping. will this go through the net queue or Shawn's queue. In any case,
it should make it into a fixes queue, because this board is currently
broken in 5.10-rc2.

Thanks,
-michael
Vladimir Oltean Nov. 9, 2020, 10:46 a.m. UTC | #4
On Sun, Nov 08, 2020 at 07:00:46PM +0100, Michael Walle wrote:
> Hi,
> 
> Am 2020-11-01 14:19, schrieb Michael Walle:
> > Since commit 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") the
> > network port of the Kontron sl28 board is broken. After the migration to
> > phylink the device tree has to specify the in-band-mode property. Add
> > it.
> > 
> > Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX")
> > Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> > Signed-off-by: Michael Walle <michael@walle.cc>
> 
> Ping. will this go through the net queue or Shawn's queue. In any case,
> it should make it into a fixes queue, because this board is currently
> broken in 5.10-rc2.

Michael, this cannot go through the net queue, since you didn't copy
netdev@vger.kernel.org to the email. It's not in Jakub's patchwork. My
advice would be to resend it.
Michael Walle Nov. 9, 2020, 10:48 a.m. UTC | #5
Am 2020-11-09 11:46, schrieb Vladimir Oltean:
> On Sun, Nov 08, 2020 at 07:00:46PM +0100, Michael Walle wrote:
>> Hi,
>> 
>> Am 2020-11-01 14:19, schrieb Michael Walle:
>> > Since commit 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX") the
>> > network port of the Kontron sl28 board is broken. After the migration to
>> > phylink the device tree has to specify the in-band-mode property. Add
>> > it.
>> >
>> > Fixes: 71b77a7a27a3 ("enetc: Migrate to PHYLINK and PCS_LYNX")
>> > Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
>> > Signed-off-by: Michael Walle <michael@walle.cc>
>> 
>> Ping. will this go through the net queue or Shawn's queue. In any 
>> case,
>> it should make it into a fixes queue, because this board is currently
>> broken in 5.10-rc2.
> 
> Michael, this cannot go through the net queue, since you didn't copy
> netdev@vger.kernel.org to the email. It's not in Jakub's patchwork. My
> advice would be to resend it.

Oh damn. Thanks!

-michael
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
index f46eb47cfa4d..8161dd237971 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28.dts
@@ -75,6 +75,7 @@ 
 &enetc_port0 {
 	phy-handle = <&phy0>;
 	phy-connection-type = "sgmii";
+	managed = "in-band-status";
 	status = "okay";
 
 	mdio {