* [PATCH] ARM: dts: imx: Fix the AR803X phy-mode @ 2019-04-01 12:47 Fabio Estevam 2019-04-01 17:55 ` Michael Grzeschik 2019-04-02 15:48 ` Sasha Levin 0 siblings, 2 replies; 10+ messages in thread From: Fabio Estevam @ 2019-04-01 12:47 UTC (permalink / raw) To: shawnguo Cc: andrew, baruch, linux, stable, smoch, stwiss.opensource, linux-imx, kernel, Marc Kleine-Budde, Fabio Estevam, linux-arm-kernel Commit 6d4cd041f0af ("net: phy: at803x: disable delay only for RGMII mode") exposed an issue on imx DTS files using AR8031/AR8035 PHYs. The end result is that the boards can no longer obtain an IP address via UDHCP, for example. Quoting Andrew Lunn: "The problem here is, all the DTs were broken since day 0. However, because the PHY driver was also broken, nobody noticed and it worked. Now that the PHY driver has been fixed, all the bugs in the DTs now become an issue" To fix this problem, the phy-mode property needs to be "rgmii-id", which has the following meaning as per Documentation/devicetree/bindings/net/ethernet.txt: "RGMII with internal RX and TX delays provided by the PHY, the MAC should not add the RX or TX delays in this case)" Tested on imx6-sabresd, imx6sx-sdb and imx7d-pico boards with successfully restored networking. Based on the initial submission from Steve Twiss for the imx6qdl-sabresd. Cc: <stable@vger.kernel.org> # 4.9+ Signed-off-by: Fabio Estevam <festevam@gmail.com> Tested-by: Baruch Siach <baruch@tkos.co.il> Tested-by: Soeren Moch <smoch@web.de> Tested-by: Steve Twiss <stwiss.opensource@diasemi.com> Tested-by: Adam Thomson <Adam.Thomson@diasemi.com> Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> Tested-by: Marc Kleine-Budde <mkl@pengutronix.de> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de> --- arch/arm/boot/dts/imx6dl-riotboard.dts | 2 +- arch/arm/boot/dts/imx6q-tbs2910.dts | 2 +- arch/arm/boot/dts/imx6qdl-apf6.dtsi | 2 +- arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 2 +- arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 2 +- arch/arm/boot/dts/imx6qdl-sr-som.dtsi | 2 +- arch/arm/boot/dts/imx6qdl-wandboard.dtsi | 2 +- arch/arm/boot/dts/imx6sx-sabreauto.dts | 2 +- arch/arm/boot/dts/imx6sx-sdb.dtsi | 2 +- arch/arm/boot/dts/imx7d-pico.dtsi | 2 +- 10 files changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/arm/boot/dts/imx6dl-riotboard.dts b/arch/arm/boot/dts/imx6dl-riotboard.dts index 65c184bb8fb0..d9de49efa802 100644 --- a/arch/arm/boot/dts/imx6dl-riotboard.dts +++ b/arch/arm/boot/dts/imx6dl-riotboard.dts @@ -92,7 +92,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-gpios = <&gpio3 31 GPIO_ACTIVE_LOW>; interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; diff --git a/arch/arm/boot/dts/imx6q-tbs2910.dts b/arch/arm/boot/dts/imx6q-tbs2910.dts index 2ce8399a10ba..bfff87ce2e1f 100644 --- a/arch/arm/boot/dts/imx6q-tbs2910.dts +++ b/arch/arm/boot/dts/imx6q-tbs2910.dts @@ -98,7 +98,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-gpios = <&gpio1 25 GPIO_ACTIVE_LOW>; status = "okay"; }; diff --git a/arch/arm/boot/dts/imx6qdl-apf6.dtsi b/arch/arm/boot/dts/imx6qdl-apf6.dtsi index 1ebf29f43a24..4738c3c1ab50 100644 --- a/arch/arm/boot/dts/imx6qdl-apf6.dtsi +++ b/arch/arm/boot/dts/imx6qdl-apf6.dtsi @@ -51,7 +51,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-duration = <10>; phy-reset-gpios = <&gpio1 24 GPIO_ACTIVE_LOW>; status = "okay"; diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi index 1280de50a984..f3404dd10537 100644 --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi @@ -292,7 +292,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; fsl,err006687-workaround-present; diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi index a0705066ccba..185fb17a3500 100644 --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi @@ -202,7 +202,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-gpios = <&gpio1 25 GPIO_ACTIVE_LOW>; status = "okay"; }; diff --git a/arch/arm/boot/dts/imx6qdl-sr-som.dtsi b/arch/arm/boot/dts/imx6qdl-sr-som.dtsi index 4ccb7afc4b35..6d7f6b9035bc 100644 --- a/arch/arm/boot/dts/imx6qdl-sr-som.dtsi +++ b/arch/arm/boot/dts/imx6qdl-sr-som.dtsi @@ -53,7 +53,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_microsom_enet_ar8035>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-duration = <2>; phy-reset-gpios = <&gpio4 15 GPIO_ACTIVE_LOW>; status = "okay"; diff --git a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi index b7d5fb421404..50d9a989e06a 100644 --- a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi +++ b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi @@ -224,7 +224,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-gpios = <&gpio3 29 GPIO_ACTIVE_LOW>; interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; diff --git a/arch/arm/boot/dts/imx6sx-sabreauto.dts b/arch/arm/boot/dts/imx6sx-sabreauto.dts index b0ee324afe58..315044ccd65f 100644 --- a/arch/arm/boot/dts/imx6sx-sabreauto.dts +++ b/arch/arm/boot/dts/imx6sx-sabreauto.dts @@ -75,7 +75,7 @@ &fec1 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet1>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-handle = <ðphy1>; fsl,magic-packet; status = "okay"; diff --git a/arch/arm/boot/dts/imx6sx-sdb.dtsi b/arch/arm/boot/dts/imx6sx-sdb.dtsi index 08ede56c3f10..f6972deb5e39 100644 --- a/arch/arm/boot/dts/imx6sx-sdb.dtsi +++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi @@ -191,7 +191,7 @@ pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet1>; phy-supply = <®_enet_3v3>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-handle = <ðphy1>; phy-reset-gpios = <&gpio2 7 GPIO_ACTIVE_LOW>; status = "okay"; diff --git a/arch/arm/boot/dts/imx7d-pico.dtsi b/arch/arm/boot/dts/imx7d-pico.dtsi index 3fd595a71202..6f50ebf31a0a 100644 --- a/arch/arm/boot/dts/imx7d-pico.dtsi +++ b/arch/arm/boot/dts/imx7d-pico.dtsi @@ -92,7 +92,7 @@ <&clks IMX7D_ENET1_TIME_ROOT_CLK>; assigned-clock-parents = <&clks IMX7D_PLL_ENET_MAIN_100M_CLK>; assigned-clock-rates = <0>, <100000000>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-handle = <ðphy0>; fsl,magic-packet; phy-reset-gpios = <&gpio6 11 GPIO_ACTIVE_LOW>; -- 2.17.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-01 12:47 [PATCH] ARM: dts: imx: Fix the AR803X phy-mode Fabio Estevam @ 2019-04-01 17:55 ` Michael Grzeschik 2019-04-01 19:47 ` Fabio Estevam 2019-04-02 15:48 ` Sasha Levin 1 sibling, 1 reply; 10+ messages in thread From: Michael Grzeschik @ 2019-04-01 17:55 UTC (permalink / raw) To: Fabio Estevam Cc: andrew, baruch, linux, stable, smoch, stwiss.opensource, linux-imx, kernel, Marc Kleine-Budde, shawnguo, linux-arm-kernel [-- Attachment #1.1: Type: text/plain, Size: 9527 bytes --] Hi, On Mon, Apr 01, 2019 at 09:47:30AM -0300, Fabio Estevam wrote: > Commit 6d4cd041f0af ("net: phy: at803x: disable delay only for RGMII mode") > exposed an issue on imx DTS files using AR8031/AR8035 PHYs. > > The end result is that the boards can no longer obtain an IP address > via UDHCP, for example. > > Quoting Andrew Lunn: > > "The problem here is, all the DTs were broken since day 0. However, > because the PHY driver was also broken, nobody noticed and it > worked. Now that the PHY driver has been fixed, all the bugs in the > DTs now become an issue" > > To fix this problem, the phy-mode property needs to be "rgmii-id", which > has the following meaning as per > Documentation/devicetree/bindings/net/ethernet.txt: > > "RGMII with internal RX and TX delays provided by the PHY, the MAC should > not add the RX or TX delays in this case)" > > Tested on imx6-sabresd, imx6sx-sdb and imx7d-pico boards with > successfully restored networking. > > Based on the initial submission from Steve Twiss for the > imx6qdl-sabresd. > > Cc: <stable@vger.kernel.org> # 4.9+ > Signed-off-by: Fabio Estevam <festevam@gmail.com> > Tested-by: Baruch Siach <baruch@tkos.co.il> > Tested-by: Soeren Moch <smoch@web.de> > Tested-by: Steve Twiss <stwiss.opensource@diasemi.com> > Tested-by: Adam Thomson <Adam.Thomson@diasemi.com> > Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> > Tested-by: Marc Kleine-Budde <mkl@pengutronix.de> > Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de> > --- > arch/arm/boot/dts/imx6dl-riotboard.dts | 2 +- > arch/arm/boot/dts/imx6q-tbs2910.dts | 2 +- > arch/arm/boot/dts/imx6qdl-apf6.dtsi | 2 +- > arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 2 +- > arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 2 +- > arch/arm/boot/dts/imx6qdl-sr-som.dtsi | 2 +- > arch/arm/boot/dts/imx6qdl-wandboard.dtsi | 2 +- > arch/arm/boot/dts/imx6sx-sabreauto.dts | 2 +- > arch/arm/boot/dts/imx6sx-sdb.dtsi | 2 +- > arch/arm/boot/dts/imx7d-pico.dtsi | 2 +- > 10 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6dl-riotboard.dts b/arch/arm/boot/dts/imx6dl-riotboard.dts > index 65c184bb8fb0..d9de49efa802 100644 > --- a/arch/arm/boot/dts/imx6dl-riotboard.dts > +++ b/arch/arm/boot/dts/imx6dl-riotboard.dts > @@ -92,7 +92,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-reset-gpios = <&gpio3 31 GPIO_ACTIVE_LOW>; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > diff --git a/arch/arm/boot/dts/imx6q-tbs2910.dts b/arch/arm/boot/dts/imx6q-tbs2910.dts > index 2ce8399a10ba..bfff87ce2e1f 100644 > --- a/arch/arm/boot/dts/imx6q-tbs2910.dts > +++ b/arch/arm/boot/dts/imx6q-tbs2910.dts > @@ -98,7 +98,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-reset-gpios = <&gpio1 25 GPIO_ACTIVE_LOW>; > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/imx6qdl-apf6.dtsi b/arch/arm/boot/dts/imx6qdl-apf6.dtsi > index 1ebf29f43a24..4738c3c1ab50 100644 > --- a/arch/arm/boot/dts/imx6qdl-apf6.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-apf6.dtsi > @@ -51,7 +51,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-reset-duration = <10>; > phy-reset-gpios = <&gpio1 24 GPIO_ACTIVE_LOW>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > index 1280de50a984..f3404dd10537 100644 > --- a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi > @@ -292,7 +292,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > fsl,err006687-workaround-present; > diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > index a0705066ccba..185fb17a3500 100644 > --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi > @@ -202,7 +202,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-reset-gpios = <&gpio1 25 GPIO_ACTIVE_LOW>; > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/imx6qdl-sr-som.dtsi b/arch/arm/boot/dts/imx6qdl-sr-som.dtsi > index 4ccb7afc4b35..6d7f6b9035bc 100644 > --- a/arch/arm/boot/dts/imx6qdl-sr-som.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-sr-som.dtsi > @@ -53,7 +53,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_microsom_enet_ar8035>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-reset-duration = <2>; > phy-reset-gpios = <&gpio4 15 GPIO_ACTIVE_LOW>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi > index b7d5fb421404..50d9a989e06a 100644 > --- a/arch/arm/boot/dts/imx6qdl-wandboard.dtsi > +++ b/arch/arm/boot/dts/imx6qdl-wandboard.dtsi > @@ -224,7 +224,7 @@ > &fec { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-reset-gpios = <&gpio3 29 GPIO_ACTIVE_LOW>; > interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, > <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; > diff --git a/arch/arm/boot/dts/imx6sx-sabreauto.dts b/arch/arm/boot/dts/imx6sx-sabreauto.dts > index b0ee324afe58..315044ccd65f 100644 > --- a/arch/arm/boot/dts/imx6sx-sabreauto.dts > +++ b/arch/arm/boot/dts/imx6sx-sabreauto.dts > @@ -75,7 +75,7 @@ > &fec1 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet1>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-handle = <ðphy1>; > fsl,magic-packet; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx6sx-sdb.dtsi b/arch/arm/boot/dts/imx6sx-sdb.dtsi > index 08ede56c3f10..f6972deb5e39 100644 > --- a/arch/arm/boot/dts/imx6sx-sdb.dtsi > +++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi > @@ -191,7 +191,7 @@ > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_enet1>; > phy-supply = <®_enet_3v3>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-handle = <ðphy1>; > phy-reset-gpios = <&gpio2 7 GPIO_ACTIVE_LOW>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx7d-pico.dtsi b/arch/arm/boot/dts/imx7d-pico.dtsi > index 3fd595a71202..6f50ebf31a0a 100644 > --- a/arch/arm/boot/dts/imx7d-pico.dtsi > +++ b/arch/arm/boot/dts/imx7d-pico.dtsi > @@ -92,7 +92,7 @@ > <&clks IMX7D_ENET1_TIME_ROOT_CLK>; > assigned-clock-parents = <&clks IMX7D_PLL_ENET_MAIN_100M_CLK>; > assigned-clock-rates = <0>, <100000000>; > - phy-mode = "rgmii"; > + phy-mode = "rgmii-id"; > phy-handle = <ðphy0>; > fsl,magic-packet; > phy-reset-gpios = <&gpio6 11 GPIO_ACTIVE_LOW>; > -- > 2.17.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > Please add: diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi index c3415aa348a2c..153002e632f43 100644 --- a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi @@ -331,7 +331,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>; txen-skew-ps = <0>; txc-skew-ps = <3000>; diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi index ed53f07c6b7b5..4ca653b467c76 100644 --- a/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6_som2.dtsi @@ -262,7 +262,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; interrupts-extended = <&gpio1 6 IRQ_TYPE_LEVEL_HIGH>, <&intc 0 119 IRQ_TYPE_LEVEL_HIGH>; fsl,err006687-workaround-present; diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi index 8b0e432099b52..f08428820a10e 100644 --- a/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi +++ b/arch/arm/boot/dts/imx6qdl-nitrogen6x.dtsi @@ -264,7 +264,7 @@ &fec { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>; - phy-mode = "rgmii"; + phy-mode = "rgmii-id"; phy-reset-gpios = <&gpio1 27 GPIO_ACTIVE_LOW>; txen-skew-ps = <0>; txc-skew-ps = <3000>; Tested-by: Michael Grzeschik <m.grzeschik@pengutronix.de> Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de> Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-01 17:55 ` Michael Grzeschik @ 2019-04-01 19:47 ` Fabio Estevam 2019-04-02 5:52 ` Michael Grzeschik 0 siblings, 1 reply; 10+ messages in thread From: Fabio Estevam @ 2019-04-01 19:47 UTC (permalink / raw) To: Michael Grzeschik Cc: Andrew Lunn, Baruch Siach, Russell King - ARM Linux, stable, Soeren Moch, Steve Twiss, NXP Linux Team, Sascha Hauer, Marc Kleine-Budde, Shawn Guo, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE Hi Michael, On Mon, Apr 1, 2019 at 2:55 PM Michael Grzeschik <mgr@pengutronix.de> wrote: > Please add: > > diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi Are these boars currently broken too? As far as I can see they do not use AR8031/AR8035. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-01 19:47 ` Fabio Estevam @ 2019-04-02 5:52 ` Michael Grzeschik 2019-04-02 11:06 ` Fabio Estevam 0 siblings, 1 reply; 10+ messages in thread From: Michael Grzeschik @ 2019-04-02 5:52 UTC (permalink / raw) To: Fabio Estevam Cc: Andrew Lunn, Baruch Siach, Russell King - ARM Linux, stable, Soeren Moch, Steve Twiss, NXP Linux Team, Sascha Hauer, Marc Kleine-Budde, Shawn Guo, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE [-- Attachment #1.1: Type: text/plain, Size: 925 bytes --] Hi Fabio, On Mon, Apr 01, 2019 at 04:47:21PM -0300, Fabio Estevam wrote: > Hi Michael, > > On Mon, Apr 1, 2019 at 2:55 PM Michael Grzeschik <mgr@pengutronix.de> wrote: > > > Please add: > > > > diff --git a/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi b/arch/arm/boot/dts/imx6qdl-nitrogen6_max.dtsi > > Are these boars currently broken too? > > As far as I can see they do not use AR8031/AR8035. These use the KSZ9021. But the fix was obviously the right one after v5.0 making the network stable again. I did not dig deeper into the cause, after finding this diff solving the issue. mgr -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-02 5:52 ` Michael Grzeschik @ 2019-04-02 11:06 ` Fabio Estevam 2019-04-02 13:21 ` Andrew Lunn 0 siblings, 1 reply; 10+ messages in thread From: Fabio Estevam @ 2019-04-02 11:06 UTC (permalink / raw) To: Michael Grzeschik Cc: Andrew Lunn, Baruch Siach, Russell King - ARM Linux, stable, Soeren Moch, Steve Twiss, NXP Linux Team, Sascha Hauer, Marc Kleine-Budde, Shawn Guo, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE Hi Michael, On Tue, Apr 2, 2019 at 2:52 AM Michael Grzeschik <mgr@pengutronix.de> wrote: > These use the KSZ9021. But the fix was obviously the right one > after v5.0 making the network stable again. I did not dig deeper > into the cause, after finding this diff solving the issue. I was not aware that KSZ9021 PHYs are currently broken too. This patch aims to fixing the i.MX boards with AR8031/AR8035 PHY. The fix for KSZ9021 would require some explanation as to what commit exposed or caused the issue and would be better to be on a separate patch. Could you please work on that? Thanks _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-02 11:06 ` Fabio Estevam @ 2019-04-02 13:21 ` Andrew Lunn 2019-04-03 6:40 ` Michael Grzeschik 0 siblings, 1 reply; 10+ messages in thread From: Andrew Lunn @ 2019-04-02 13:21 UTC (permalink / raw) To: Fabio Estevam Cc: moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE, Baruch Siach, Russell King - ARM Linux, stable, Soeren Moch, Steve Twiss, NXP Linux Team, Sascha Hauer, Marc Kleine-Budde, Shawn Guo, Michael Grzeschik > The fix for KSZ9021 would require some explanation as to what commit > exposed or caused the issue and would be better to be on a separate > patch. +1 Looking at the commit log for that driver, i don't see a patch which would cause issues like this. So a clear explanation is required. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-02 13:21 ` Andrew Lunn @ 2019-04-03 6:40 ` Michael Grzeschik 2019-04-08 10:44 ` Michael Grzeschik 0 siblings, 1 reply; 10+ messages in thread From: Michael Grzeschik @ 2019-04-03 6:40 UTC (permalink / raw) To: Andrew Lunn Cc: Baruch Siach, Fabio Estevam, Russell King - ARM Linux, stable, Soeren Moch, Steve Twiss, NXP Linux Team, Sascha Hauer, Marc Kleine-Budde, Shawn Guo, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE [-- Attachment #1.1: Type: text/plain, Size: 788 bytes --] On Tue, Apr 02, 2019 at 03:21:22PM +0200, Andrew Lunn wrote: > > The fix for KSZ9021 would require some explanation as to what commit > > exposed or caused the issue and would be better to be on a separate > > patch. > > +1 > > Looking at the commit log for that driver, i don't see a patch which > would cause issues like this. > > So a clear explanation is required. I will bisect the prbolem and come back after I found the patch causing the issue. mgr -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-03 6:40 ` Michael Grzeschik @ 2019-04-08 10:44 ` Michael Grzeschik 0 siblings, 0 replies; 10+ messages in thread From: Michael Grzeschik @ 2019-04-08 10:44 UTC (permalink / raw) To: Andrew Lunn Cc: Baruch Siach, Shawn Guo, Russell King - ARM Linux, stable, Soeren Moch, Steve Twiss, NXP Linux Team, Sascha Hauer, Marc Kleine-Budde, Fabio Estevam, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE [-- Attachment #1.1: Type: text/plain, Size: 1203 bytes --] Hi, On Wed, Apr 03, 2019 at 08:40:39AM +0200, Michael Grzeschik wrote: > On Tue, Apr 02, 2019 at 03:21:22PM +0200, Andrew Lunn wrote: > > > The fix for KSZ9021 would require some explanation as to what commit > > > exposed or caused the issue and would be better to be on a separate > > > patch. > > > > +1 > > > > Looking at the commit log for that driver, i don't see a patch which > > would cause issues like this. > > > > So a clear explanation is required. > > I will bisect the prbolem and come back after I found the patch causing > the issue. Last week I tried to reproduce the issue with the nitrogen6x_max. But by using the vanilla kernel there was no issue with v5.0. So it was probably some stack specific issue or my hw setup was somehow buggy. As long it is not possible for me to reproduce the issue this should be ignored. Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-01 12:47 [PATCH] ARM: dts: imx: Fix the AR803X phy-mode Fabio Estevam 2019-04-01 17:55 ` Michael Grzeschik @ 2019-04-02 15:48 ` Sasha Levin 2019-04-02 16:26 ` Fabio Estevam 1 sibling, 1 reply; 10+ messages in thread From: Sasha Levin @ 2019-04-02 15:48 UTC (permalink / raw) To: Sasha Levin, Fabio Estevam, shawnguo; +Cc: , baruch, stable, linux-arm-kernel Hi, [This is an automated email] This commit has been processed because it contains a -stable tag. The stable tag indicates that it's relevant for the following trees: 4.9+ The bot has tested the following trees: v5.0.5, v4.19.32, v4.14.109, v4.9.166. v5.0.5: Build OK! v4.19.32: Failed to apply! Possible dependencies: 29988e867cb1 ("ARM: dts: imx7: Fix memory node duplication") 4edbe6aa46d1 ("ARM: dts: imx7d-pico-pi: Move SoM related part to imx7d-pico.dtsi") 50536c661194 ("ARM: dts: imx7d-pico: Do not harcode the memory size") v4.14.109: Failed to apply! Possible dependencies: 1bf52b98ae1c ("ARM: dts: imx6sx-sabreauto: add external 24MHz clock source") 221aaa198e70 ("ARM: dts: imx6sx-sabreauto: drop board specific pinctrl container node") 37accf72e04b ("ARM: dts: imx6sx-sabreauto: add fec support") 41bbeadceb03 ("ARM: dts: imx7d-pico-pi: Separate into cpu and baseboard dts") 4edbe6aa46d1 ("ARM: dts: imx7d-pico-pi: Move SoM related part to imx7d-pico.dtsi") 76dc340e7585 ("ARM: dts: imx6qdl: SolidRun: move AR8035 into microsom") 8dccafaa281a ("arm: dts: fix unit-address leading 0s") 99a52450c707 ("ARM: dts: imx7d-pico: Add Wifi support") d9e246cc2880 ("ARM: dts: imx6sx-sabreauto: add PMIC support") v4.9.166: Failed to apply! Possible dependencies: 12de44f55dcd ("ARM: dts: imx: Fix Ethernet PHY reset polarity") 1317efa1699f ("ARM: dts: imx6ul: Add DTS for liteSOM module") 13283626c889 ("ARM: dts: imx/vf: Correct license text") 1bf52b98ae1c ("ARM: dts: imx6sx-sabreauto: add external 24MHz clock source") 221aaa198e70 ("ARM: dts: imx6sx-sabreauto: drop board specific pinctrl container node") 23c787a99ff7 ("ARM: dts: imx7d-pico: Add initial support") 26ca8b52d6e1 ("ARM: dts: add support for Turris Omnia") 37accf72e04b ("ARM: dts: imx6sx-sabreauto: add fec support") 3faa1bb2e89c ("ARM: dts: imx: add Boundary Devices Nitrogen6_SOM2 support") 41bbeadceb03 ("ARM: dts: imx7d-pico-pi: Separate into cpu and baseboard dts") 478c9440b328 ("ARM: dts: imx6ul: Add DTS for liteBoard") 4edbe6aa46d1 ("ARM: dts: imx7d-pico-pi: Move SoM related part to imx7d-pico.dtsi") 6df11287f7c9 ("ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support") 76dc340e7585 ("ARM: dts: imx6qdl: SolidRun: move AR8035 into microsom") 8dccafaa281a ("arm: dts: fix unit-address leading 0s") 99a52450c707 ("ARM: dts: imx7d-pico: Add Wifi support") 9daee3076940 ("ARM: dts: imx6q: Add Engicam i.CoreM6 DualLite/Solo initial support") a58d73340b0e ("ARM: dts: armada-385-synology-ds116: add support for Synology DS116 NAS") c201369d4aa5 ("ARM: dts: imx6ull: add imx6ull support") d9e246cc2880 ("ARM: dts: imx6sx-sabreauto: add PMIC support") fc48e76489fd ("ARM: dts: imx6: Add support for Toradex Colibri iMX6 module") How should we proceed with this patch? -- Thanks, Sasha _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: dts: imx: Fix the AR803X phy-mode 2019-04-02 15:48 ` Sasha Levin @ 2019-04-02 16:26 ` Fabio Estevam 0 siblings, 0 replies; 10+ messages in thread From: Fabio Estevam @ 2019-04-02 16:26 UTC (permalink / raw) To: Sasha Levin Cc: Baruch Siach, Shawn Guo, stable, moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE On Tue, Apr 2, 2019 at 12:48 PM Sasha Levin <sashal@kernel.org> wrote: > > Hi, > > [This is an automated email] > > This commit has been processed because it contains a -stable tag. > The stable tag indicates that it's relevant for the following trees: 4.9+ > > The bot has tested the following trees: v5.0.5, v4.19.32, v4.14.109, v4.9.166. > How should we proceed with this patch? I can manually generate versions for 4.9, 4.14 and 4.19 stable trees when this one hit mainline. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-04-08 10:44 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-04-01 12:47 [PATCH] ARM: dts: imx: Fix the AR803X phy-mode Fabio Estevam 2019-04-01 17:55 ` Michael Grzeschik 2019-04-01 19:47 ` Fabio Estevam 2019-04-02 5:52 ` Michael Grzeschik 2019-04-02 11:06 ` Fabio Estevam 2019-04-02 13:21 ` Andrew Lunn 2019-04-03 6:40 ` Michael Grzeschik 2019-04-08 10:44 ` Michael Grzeschik 2019-04-02 15:48 ` Sasha Levin 2019-04-02 16:26 ` Fabio Estevam
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).