* [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity @ 2021-01-07 23:50 Fabio Estevam 2021-01-07 23:50 ` [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander Fabio Estevam ` (3 more replies) 0 siblings, 4 replies; 15+ messages in thread From: Fabio Estevam @ 2021-01-07 23:50 UTC (permalink / raw) To: shawnguo; +Cc: linux-arm-kernel, andrew, Fabio Estevam, linux-imx, kernel The GPIO expander SPI chipselect is active low. Mark it as such to avoid the following warning: [ 6.839213] gpio@0 enforce active low on chipselect handle Signed-off-by: Fabio Estevam <festevam@gmail.com> --- arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi index 64c2d1e9f7fc..a6dbfa85bb6a 100644 --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi @@ -101,7 +101,7 @@ status = "okay"; gpio-sck = <&gpio5 11 0>; gpio-mosi = <&gpio5 10 0>; - cs-gpios = <&gpio5 7 0>; + cs-gpios = <&gpio5 7 GPIO_ACTIVE_LOW>; num-chipselects = <1>; #address-cells = <1>; #size-cells = <0>; -- 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] 15+ messages in thread
* [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander 2021-01-07 23:50 [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Fabio Estevam @ 2021-01-07 23:50 ` Fabio Estevam 2021-01-08 1:02 ` Andrew Lunn 2021-01-18 0:10 ` Shawn Guo 2021-01-07 23:50 ` [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Fabio Estevam ` (2 subsequent siblings) 3 siblings, 2 replies; 15+ messages in thread From: Fabio Estevam @ 2021-01-07 23:50 UTC (permalink / raw) To: shawnguo; +Cc: linux-arm-kernel, andrew, Fabio Estevam, linux-imx, kernel Currently the 74LV595PW GPIO expander is not functional because its OE pin is not driven low. Make it funcional by passing the 'enable-gpios' property inside the GPIO expander node. After putting the OE pin in low state, the outputs of the 74LV595PW all go low. The two KSZ8081 Ethernet PHYs reset lines are driven from the the GPIO expander and as they remain low, this causes the Ethernet PHYs not to be detected. There is one solution to this problem as suggested by Andrew Lunn: "Some devices will respond to MDIO while held in reset, some don't. If your PHYs don't you need to add a compatible of the form ethernet-phy-id[a-f0-9]{4}\\.[a-f0-9]{4}$ with the PHY ID. The PHY will then be probed, independent of if it can be found on the bus or not, and that probing will enable the GPIO." So pass the "ethernet-phy-id0022.1560" for the KSZ8081 PHYs so that they both can be functional after 74LV595PW is activated. Suggested-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: Fabio Estevam <festevam@gmail.com> --- arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi index a6dbfa85bb6a..8bf845afd166 100644 --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi @@ -113,6 +113,7 @@ reg = <0>; registers-number = <1>; spi-max-frequency = <100000>; + enable-gpios = <&gpio5 8 GPIO_ACTIVE_LOW>; }; }; @@ -169,6 +170,7 @@ #size-cells = <0>; ethphy0: ethernet-phy@2 { + compatible = "ethernet-phy-id0022.1560"; reg = <2>; micrel,led-mode = <1>; clocks = <&clks IMX6UL_CLK_ENET_REF>; @@ -176,6 +178,7 @@ }; ethphy1: ethernet-phy@1 { + compatible = "ethernet-phy-id0022.1560"; reg = <1>; micrel,led-mode = <1>; clocks = <&clks IMX6UL_CLK_ENET2_REF>; -- 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] 15+ messages in thread
* Re: [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander 2021-01-07 23:50 ` [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander Fabio Estevam @ 2021-01-08 1:02 ` Andrew Lunn 2021-01-18 0:10 ` Shawn Guo 1 sibling, 0 replies; 15+ messages in thread From: Andrew Lunn @ 2021-01-08 1:02 UTC (permalink / raw) To: Fabio Estevam; +Cc: kernel, shawnguo, linux-imx, linux-arm-kernel On Thu, Jan 07, 2021 at 08:50:42PM -0300, Fabio Estevam wrote: > Currently the 74LV595PW GPIO expander is not functional because its OE > pin is not driven low. > > Make it funcional by passing the 'enable-gpios' property inside the > GPIO expander node. > > After putting the OE pin in low state, the outputs of the 74LV595PW all > go low. > > The two KSZ8081 Ethernet PHYs reset lines are driven from the > the GPIO expander and as they remain low, this causes the Ethernet > PHYs not to be detected. > > There is one solution to this problem as suggested by Andrew Lunn: > > "Some devices will respond to MDIO while held in reset, some don't. > If your PHYs don't you need to add a compatible of the form > ethernet-phy-id[a-f0-9]{4}\\.[a-f0-9]{4}$ with the PHY ID. The PHY > will then be probed, independent of if it can be found on the bus or > not, and that probing will enable the GPIO." > > So pass the "ethernet-phy-id0022.1560" for the KSZ8081 PHYs so that they > both can be functional after 74LV595PW is activated. > > Suggested-by: Andrew Lunn <andrew@lunn.ch> > Signed-off-by: Fabio Estevam <festevam@gmail.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> 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] 15+ messages in thread
* Re: [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander 2021-01-07 23:50 ` [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander Fabio Estevam 2021-01-08 1:02 ` Andrew Lunn @ 2021-01-18 0:10 ` Shawn Guo 1 sibling, 0 replies; 15+ messages in thread From: Shawn Guo @ 2021-01-18 0:10 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-arm-kernel, andrew, linux-imx, kernel On Thu, Jan 07, 2021 at 08:50:42PM -0300, Fabio Estevam wrote: > Currently the 74LV595PW GPIO expander is not functional because its OE > pin is not driven low. > > Make it funcional by passing the 'enable-gpios' property inside the > GPIO expander node. > > After putting the OE pin in low state, the outputs of the 74LV595PW all > go low. > > The two KSZ8081 Ethernet PHYs reset lines are driven from the > the GPIO expander and as they remain low, this causes the Ethernet > PHYs not to be detected. > > There is one solution to this problem as suggested by Andrew Lunn: > > "Some devices will respond to MDIO while held in reset, some don't. > If your PHYs don't you need to add a compatible of the form > ethernet-phy-id[a-f0-9]{4}\\.[a-f0-9]{4}$ with the PHY ID. The PHY > will then be probed, independent of if it can be found on the bus or > not, and that probing will enable the GPIO." > > So pass the "ethernet-phy-id0022.1560" for the KSZ8081 PHYs so that they > both can be functional after 74LV595PW is activated. > > Suggested-by: Andrew Lunn <andrew@lunn.ch> > Signed-off-by: Fabio Estevam <festevam@gmail.com> Applied, 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] 15+ messages in thread
* [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-01-07 23:50 [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Fabio Estevam 2021-01-07 23:50 ` [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander Fabio Estevam @ 2021-01-07 23:50 ` Fabio Estevam 2021-01-08 1:03 ` Andrew Lunn ` (2 more replies) 2021-01-07 23:50 ` [PATCH 4/4] ARM: dts: imx6ul-14x14-evk: Add camera support Fabio Estevam 2021-01-18 0:09 ` [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Shawn Guo 3 siblings, 3 replies; 15+ messages in thread From: Fabio Estevam @ 2021-01-07 23:50 UTC (permalink / raw) To: shawnguo; +Cc: linux-arm-kernel, andrew, Fabio Estevam, linux-imx, kernel There are two KSZ8081 on the imx6ul-14x14-evk: one has the reset line controlled by the Q1 output of the 74LV595PW expander, and the other by the Q2 output. The KSZ8081 datasheet states that the tsr parameter (Stable supply voltage (VDDIO, VDDA_3.3) to reset high) to be 10ms minimum and "After the de-assertion of reset, wait a minimum of 100 µs before starting programming on the MIIM (MDC/MDIO) interface." Describe these parameters in the devicetree. Signed-off-by: Fabio Estevam <festevam@gmail.com> --- arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi index 8bf845afd166..ac2ec96f54e7 100644 --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi @@ -175,6 +175,10 @@ micrel,led-mode = <1>; clocks = <&clks IMX6UL_CLK_ENET_REF>; clock-names = "rmii-ref"; + reset-gpios = <&gpio_spi 1 GPIO_ACTIVE_LOW>; + reset-assert-us = <10000>; + reset-deassert-us = <100>; + }; ethphy1: ethernet-phy@1 { @@ -183,6 +187,9 @@ micrel,led-mode = <1>; clocks = <&clks IMX6UL_CLK_ENET2_REF>; clock-names = "rmii-ref"; + reset-gpios = <&gpio_spi 2 GPIO_ACTIVE_LOW>; + reset-assert-us = <10000>; + reset-deassert-us = <100>; }; }; }; -- 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] 15+ messages in thread
* Re: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-01-07 23:50 ` [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Fabio Estevam @ 2021-01-08 1:03 ` Andrew Lunn 2021-01-18 0:10 ` Shawn Guo 2021-03-10 11:04 ` Joakim Zhang 2 siblings, 0 replies; 15+ messages in thread From: Andrew Lunn @ 2021-01-08 1:03 UTC (permalink / raw) To: Fabio Estevam; +Cc: kernel, shawnguo, linux-imx, linux-arm-kernel On Thu, Jan 07, 2021 at 08:50:43PM -0300, Fabio Estevam wrote: > There are two KSZ8081 on the imx6ul-14x14-evk: one has the reset line > controlled by the Q1 output of the 74LV595PW expander, and the other > by the Q2 output. > > The KSZ8081 datasheet states that the tsr parameter (Stable supply > voltage (VDDIO, VDDA_3.3) to reset high) to be 10ms minimum and > "After the de-assertion of reset, wait a minimum of 100 µs before > starting programming on the MIIM (MDC/MDIO) interface." > > Describe these parameters in the devicetree. > > Signed-off-by: Fabio Estevam <festevam@gmail.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> 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] 15+ messages in thread
* Re: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-01-07 23:50 ` [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Fabio Estevam 2021-01-08 1:03 ` Andrew Lunn @ 2021-01-18 0:10 ` Shawn Guo 2021-03-10 11:04 ` Joakim Zhang 2 siblings, 0 replies; 15+ messages in thread From: Shawn Guo @ 2021-01-18 0:10 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-arm-kernel, andrew, linux-imx, kernel On Thu, Jan 07, 2021 at 08:50:43PM -0300, Fabio Estevam wrote: > There are two KSZ8081 on the imx6ul-14x14-evk: one has the reset line > controlled by the Q1 output of the 74LV595PW expander, and the other > by the Q2 output. > > The KSZ8081 datasheet states that the tsr parameter (Stable supply > voltage (VDDIO, VDDA_3.3) to reset high) to be 10ms minimum and > "After the de-assertion of reset, wait a minimum of 100 µs before > starting programming on the MIIM (MDC/MDIO) interface." > > Describe these parameters in the devicetree. > > Signed-off-by: Fabio Estevam <festevam@gmail.com> Applied, 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] 15+ messages in thread
* RE: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-01-07 23:50 ` [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Fabio Estevam 2021-01-08 1:03 ` Andrew Lunn 2021-01-18 0:10 ` Shawn Guo @ 2021-03-10 11:04 ` Joakim Zhang 2021-03-11 11:33 ` Joakim Zhang 2 siblings, 1 reply; 15+ messages in thread From: Joakim Zhang @ 2021-03-10 11:04 UTC (permalink / raw) To: Fabio Estevam, shawnguo; +Cc: kernel, andrew, linux-arm-kernel Hi Fabio, Andrew, After applying Fabio's patch to support PHY hardware reset. I found i.MX6UL 14x14 EVK board is broken in latest upstream kernel. ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Case 1: root@imx6ul7d:~# ifconfig eth0 down root@imx6ul7d:~# [ 279.386551] fec 2188000.ethernet eth1: Link is Down After down eth0, eth1 would down, then eth1 can't be used any more. The same behavior for another ethernet, root@imx6ul7d:~# ifconfig eth1 down root@imx6ul7d:~# [ 118.106726] fec 20b4000.ethernet eth0: Link is Down After down eth1, eth0 would down, then eth0 can't be used any more. Case 2: [ 9.182109] Micrel KSZ8081 or KSZ8091 20b4000.ethernet-1:01: attached PHY driver (mii_bus:phy_addr=20b4000.ethernet-1:01, irq=POLL) [ 9.309462] Micrel KSZ8081 or KSZ8091 20b4000.ethernet-1:02: attached PHY driver (mii_bus:phy_addr=20b4000.ethernet-1:02, irq=POLL) [ 11.397952] fec 2188000.ethernet eth1: Link is Up - 100Mbps/Full - flow control rx/tx [ 11.414343] IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready [ 11.429546] Sending DHCP requests ., OK [ 11.596850] IP-Config: Got DHCP answer from 10.193.102.252, my address is 10.193.102.87 [ 11.605689] IP-Config: Complete: [ 11.609595] device=eth1, hwaddr=00:04:9f:04:3e:92, ipaddr=10.193.102.87, mask=255.255.255.0, gw=10.193.102.254 [ 11.620621] host=10.193.102.87, domain=ap.freescale.net, nis-domain=(none) [ 11.628118] bootserver=0.0.0.0, rootserver=10.193.108.176, rootpath= [ 11.628230] nameserver0=165.114.89.4, nameserver1=134.27.184.42 [ 11.697259] cfg80211: Loading compiled-in X.509 certificates for regulatory database [ 11.731941] cfg80211: Loaded X.509 cert 'sforshee: 00b28ddf47aef9cea7' [ 11.742445] platform regulatory.0: Direct firmware load for regulatory.db failed with error -2 [ 11.751605] platform regulatory.0: Falling back to sysfs fallback for: regulatory.db [ 11.762730] ALSA device list: [ 11.765818] #0: mx6ul-wm8960 [ 12.433430] fec 2188000.ethernet eth1: Link is Down It is broken when mounting NFS, per the log, we can see that eth0 and eth1 is up, and when it down the eth0, eth1 would link down too. The root cause should be down one ethernet would affect another ethernet when it calling mdio_device_reset to pull Q1 or Q2 to low. After the ethernet down, I check Q1 and Q2 is 1.2V. I also check the code, each PHY has one phy_device, containing it's own mdio_device. So each PHY should be independent, when do mdio device level hw reset, it should not affect others. Could you give some hints? There is no any process after one debugging. Thanks a lot. Best Regards, Joakim Zhang > -----Original Message----- > From: Fabio Estevam <festevam@gmail.com> > Sent: 2021年1月8日 7:51 > To: shawnguo@kernel.org > Cc: kernel@pengutronix.de; dl-linux-imx <linux-imx@nxp.com>; > andrew@lunn.ch; linux-arm-kernel@lists.infradead.org; Fabio Estevam > <festevam@gmail.com> > Subject: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset > > There are two KSZ8081 on the imx6ul-14x14-evk: one has the reset line > controlled by the Q1 output of the 74LV595PW expander, and the other by the > Q2 output. > > The KSZ8081 datasheet states that the tsr parameter (Stable supply voltage > (VDDIO, VDDA_3.3) to reset high) to be 10ms minimum and "After the > de-assertion of reset, wait a minimum of 100 µs before starting programming > on the MIIM (MDC/MDIO) interface." > > Describe these parameters in the devicetree. > > Signed-off-by: Fabio Estevam <festevam@gmail.com> > --- > arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > index 8bf845afd166..ac2ec96f54e7 100644 > --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > @@ -175,6 +175,10 @@ > micrel,led-mode = <1>; > clocks = <&clks IMX6UL_CLK_ENET_REF>; > clock-names = "rmii-ref"; > + reset-gpios = <&gpio_spi 1 GPIO_ACTIVE_LOW>; > + reset-assert-us = <10000>; > + reset-deassert-us = <100>; > + > }; > > ethphy1: ethernet-phy@1 { > @@ -183,6 +187,9 @@ > micrel,led-mode = <1>; > clocks = <&clks IMX6UL_CLK_ENET2_REF>; > clock-names = "rmii-ref"; > + reset-gpios = <&gpio_spi 2 GPIO_ACTIVE_LOW>; > + reset-assert-us = <10000>; > + reset-deassert-us = <100>; > }; > }; > }; > -- > 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 [flat|nested] 15+ messages in thread
* RE: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-03-10 11:04 ` Joakim Zhang @ 2021-03-11 11:33 ` Joakim Zhang 2021-03-11 14:17 ` Andrew Lunn 2021-03-12 11:34 ` Fabio Estevam 0 siblings, 2 replies; 15+ messages in thread From: Joakim Zhang @ 2021-03-11 11:33 UTC (permalink / raw) To: Fabio Estevam, shawnguo, andrew; +Cc: kernel, linux-arm-kernel Hi Fabio, Shawn, Andrew, I found the root cause is the schematic diagram defect, both ENET1_nRST and ENET2_nRST are connect to ENET_nRST, which means PHY0 and PHY1 can't be hardware reset independently. Revert the patch "ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset" is not enough, after IO expander probing, Q1/Q2 default is low level, PHY attached would fail. At our local side, give IO expander a default value, but I don't know if it is a common method. If you think it is ok, I can try to upstream that patch. Best Regards, Joakim Zhang > -----Original Message----- > From: Joakim Zhang > Sent: 2021年3月10日 19:04 > To: Fabio Estevam <festevam@gmail.com>; shawnguo@kernel.org > Cc: kernel@pengutronix.de; andrew@lunn.ch; > linux-arm-kernel@lists.infradead.org > Subject: RE: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 > reset > > > Hi Fabio, Andrew, > > After applying Fabio's patch to support PHY hardware reset. I found i.MX6UL > 14x14 EVK board is broken in latest upstream kernel. > ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander > ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset > > Case 1: > root@imx6ul7d:~# ifconfig eth0 down > root@imx6ul7d:~# [ 279.386551] fec 2188000.ethernet eth1: Link is Down > After down eth0, eth1 would down, then eth1 can't be used any more. > > The same behavior for another ethernet, > root@imx6ul7d:~# ifconfig eth1 down > root@imx6ul7d:~# [ 118.106726] fec 20b4000.ethernet eth0: Link is Down > After down eth1, eth0 would down, then eth0 can't be used any more. > > Case 2: > [ 9.182109] Micrel KSZ8081 or KSZ8091 20b4000.ethernet-1:01: attached > PHY driver (mii_bus:phy_addr=20b4000.ethernet-1:01, irq=POLL) > [ 9.309462] Micrel KSZ8081 or KSZ8091 20b4000.ethernet-1:02: attached > PHY driver (mii_bus:phy_addr=20b4000.ethernet-1:02, irq=POLL) > [ 11.397952] fec 2188000.ethernet eth1: Link is Up - 100Mbps/Full - flow > control rx/tx > [ 11.414343] IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready > [ 11.429546] Sending DHCP requests ., OK > [ 11.596850] IP-Config: Got DHCP answer from 10.193.102.252, my address > is 10.193.102.87 > [ 11.605689] IP-Config: Complete: > [ 11.609595] device=eth1, hwaddr=00:04:9f:04:3e:92, > ipaddr=10.193.102.87, mask=255.255.255.0, gw=10.193.102.254 > [ 11.620621] host=10.193.102.87, domain=ap.freescale.net, > nis-domain=(none) > [ 11.628118] bootserver=0.0.0.0, rootserver=10.193.108.176, > rootpath= > [ 11.628230] nameserver0=165.114.89.4, > nameserver1=134.27.184.42 > [ 11.697259] cfg80211: Loading compiled-in X.509 certificates for regulatory > database > [ 11.731941] cfg80211: Loaded X.509 cert 'sforshee: 00b28ddf47aef9cea7' > [ 11.742445] platform regulatory.0: Direct firmware load for regulatory.db > failed with error -2 > [ 11.751605] platform regulatory.0: Falling back to sysfs fallback for: > regulatory.db > [ 11.762730] ALSA device list: > [ 11.765818] #0: mx6ul-wm8960 > [ 12.433430] fec 2188000.ethernet eth1: Link is Down > > It is broken when mounting NFS, per the log, we can see that eth0 and eth1 is > up, and when it down the eth0, eth1 would link down too. > > The root cause should be down one ethernet would affect another ethernet > when it calling mdio_device_reset to pull Q1 or Q2 to low. After the ethernet > down, I check Q1 and Q2 is 1.2V. > I also check the code, each PHY has one phy_device, containing it's own > mdio_device. So each PHY should be independent, when do mdio device level > hw reset, it should not affect others. > Could you give some hints? There is no any process after one debugging. > Thanks a lot. > > Best Regards, > Joakim Zhang > > > > -----Original Message----- > > From: Fabio Estevam <festevam@gmail.com> > > Sent: 2021年1月8日 7:51 > > To: shawnguo@kernel.org > > Cc: kernel@pengutronix.de; dl-linux-imx <linux-imx@nxp.com>; > > andrew@lunn.ch; linux-arm-kernel@lists.infradead.org; Fabio Estevam > > <festevam@gmail.com> > > Subject: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 > > reset > > > > There are two KSZ8081 on the imx6ul-14x14-evk: one has the reset line > > controlled by the Q1 output of the 74LV595PW expander, and the other > > by the > > Q2 output. > > > > The KSZ8081 datasheet states that the tsr parameter (Stable supply > > voltage (VDDIO, VDDA_3.3) to reset high) to be 10ms minimum and "After > > the de-assertion of reset, wait a minimum of 100 µs before starting > > programming on the MIIM (MDC/MDIO) interface." > > > > Describe these parameters in the devicetree. > > > > Signed-off-by: Fabio Estevam <festevam@gmail.com> > > --- > > arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > > b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > > index 8bf845afd166..ac2ec96f54e7 100644 > > --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > > +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > > @@ -175,6 +175,10 @@ > > micrel,led-mode = <1>; > > clocks = <&clks IMX6UL_CLK_ENET_REF>; > > clock-names = "rmii-ref"; > > + reset-gpios = <&gpio_spi 1 GPIO_ACTIVE_LOW>; > > + reset-assert-us = <10000>; > > + reset-deassert-us = <100>; > > + > > }; > > > > ethphy1: ethernet-phy@1 { > > @@ -183,6 +187,9 @@ > > micrel,led-mode = <1>; > > clocks = <&clks IMX6UL_CLK_ENET2_REF>; > > clock-names = "rmii-ref"; > > + reset-gpios = <&gpio_spi 2 GPIO_ACTIVE_LOW>; > > + reset-assert-us = <10000>; > > + reset-deassert-us = <100>; > > }; > > }; > > }; > > -- > > 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 [flat|nested] 15+ messages in thread
* Re: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-03-11 11:33 ` Joakim Zhang @ 2021-03-11 14:17 ` Andrew Lunn 2021-03-12 2:02 ` Joakim Zhang 2021-03-12 11:34 ` Fabio Estevam 1 sibling, 1 reply; 15+ messages in thread From: Andrew Lunn @ 2021-03-11 14:17 UTC (permalink / raw) To: Joakim Zhang; +Cc: Fabio Estevam, shawnguo, kernel, linux-arm-kernel On Thu, Mar 11, 2021 at 11:33:04AM +0000, Joakim Zhang wrote: > > Hi Fabio, Shawn, Andrew, > > I found the root cause is the schematic diagram defect, both > ENET1_nRST and ENET2_nRST are connect to ENET_nRST, which means PHY0 > and PHY1 can't be hardware reset independently. You can move the reset-gpio up into the MDIO node. It then becomes an MDIO bus reset, resetting all PHYs on the bus before they are probed. 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] 15+ messages in thread
* RE: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-03-11 14:17 ` Andrew Lunn @ 2021-03-12 2:02 ` Joakim Zhang 0 siblings, 0 replies; 15+ messages in thread From: Joakim Zhang @ 2021-03-12 2:02 UTC (permalink / raw) To: Andrew Lunn; +Cc: Fabio Estevam, shawnguo, kernel, linux-arm-kernel > -----Original Message----- > From: Andrew Lunn <andrew@lunn.ch> > Sent: 2021年3月11日 22:18 > To: Joakim Zhang <qiangqing.zhang@nxp.com> > Cc: Fabio Estevam <festevam@gmail.com>; shawnguo@kernel.org; > kernel@pengutronix.de; linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 > reset > > On Thu, Mar 11, 2021 at 11:33:04AM +0000, Joakim Zhang wrote: > > > > Hi Fabio, Shawn, Andrew, > > > > > I found the root cause is the schematic diagram defect, both > > ENET1_nRST and ENET2_nRST are connect to ENET_nRST, which means PHY0 > > and PHY1 can't be hardware reset independently. > > You can move the reset-gpio up into the MDIO node. It then becomes an MDIO > bus reset, resetting all PHYs on the bus before they are probed. Hi Andrew, I did below change, it can't work. It should be PHY framework have not support multiple reset-gpios for single MDIO bus. --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi @@ -203,6 +203,9 @@ mdio { #address-cells = <1>; #size-cells = <0>; + reset-gpios = <&gpio_spi 1 GPIO_ACTIVE_LOW>, <&gpio_spi 2 GPIO_ACTIVE_LOW>; + reset-delay-us = <10000>; + reset-post-delay-us = <100>; ethphy0: ethernet-phy@2 { compatible = "ethernet-phy-id0022.1560"; @@ -210,9 +213,6 @@ micrel,led-mode = <1>; clocks = <&clks IMX6UL_CLK_ENET_REF>; clock-names = "rmii-ref"; - reset-gpios = <&gpio_spi 1 GPIO_ACTIVE_LOW>; - reset-assert-us = <10000>; - reset-deassert-us = <100>; }; @@ -222,9 +222,6 @@ micrel,led-mode = <1>; clocks = <&clks IMX6UL_CLK_ENET2_REF>; clock-names = "rmii-ref"; - reset-gpios = <&gpio_spi 2 GPIO_ACTIVE_LOW>; - reset-assert-us = <10000>; - reset-deassert-us = <100>; }; }; }; In i.MX6UL 14x14 EVK board, the hareware design is as below: -----ENET1_nRST--- -----------IO expander Q1 |-----------ENET_nRST--------| -----ENET2_nRST--- -----------IO expander Q2 Best Regards, Joakim Zhang > 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] 15+ messages in thread
* Re: [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset 2021-03-11 11:33 ` Joakim Zhang 2021-03-11 14:17 ` Andrew Lunn @ 2021-03-12 11:34 ` Fabio Estevam 1 sibling, 0 replies; 15+ messages in thread From: Fabio Estevam @ 2021-03-12 11:34 UTC (permalink / raw) To: Joakim Zhang; +Cc: shawnguo, andrew, kernel, linux-arm-kernel Hi Joakim, On Thu, Mar 11, 2021 at 8:33 AM Joakim Zhang <qiangqing.zhang@nxp.com> wrote: > > > Hi Fabio, Shawn, Andrew, > > I found the root cause is the schematic diagram defect, both ENET1_nRST and ENET2_nRST are connect to ENET_nRST, which means PHY0 and PHY1 can't be hardware reset independently. Revert the patch > "ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset" is not enough, after IO expander probing, Q1/Q2 default is low level, PHY attached would fail. At our local side, give IO expander a default value, but I don't know if it is a common method. If you think it is ok, I can try to upstream that patch. Sorry for the breakage. I have only tested normal operation when I submitted the patch, without putting any interface down. I was able to reproduce the problem and I will submit a patch to fix the problem. 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] 15+ messages in thread
* [PATCH 4/4] ARM: dts: imx6ul-14x14-evk: Add camera support 2021-01-07 23:50 [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Fabio Estevam 2021-01-07 23:50 ` [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander Fabio Estevam 2021-01-07 23:50 ` [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Fabio Estevam @ 2021-01-07 23:50 ` Fabio Estevam 2021-01-18 0:11 ` Shawn Guo 2021-01-18 0:09 ` [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Shawn Guo 3 siblings, 1 reply; 15+ messages in thread From: Fabio Estevam @ 2021-01-07 23:50 UTC (permalink / raw) To: shawnguo; +Cc: linux-arm-kernel, andrew, Fabio Estevam, linux-imx, kernel Add support for the OV5640 parallel camera module. Signed-off-by: Fabio Estevam <festevam@gmail.com> --- arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 43 ++++++++++++++++++++++++- 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi index ac2ec96f54e7..ad039821a767 100644 --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi @@ -146,6 +146,41 @@ reg = <0x1a>; wlf,shared-lrclk; }; + + camera@3c { + compatible = "ovti,ov5640"; + reg = <0x3c>; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_camera_clock>; + clocks = <&clks IMX6UL_CLK_CSI>; + clock-names = "xclk"; + powerdown-gpios = <&gpio_spi 6 GPIO_ACTIVE_HIGH>; + reset-gpios = <&gpio_spi 5 GPIO_ACTIVE_LOW>; + + port { + ov5640_to_parallel: endpoint { + remote-endpoint = <¶llel_from_ov5640>; + bus-width = <8>; + data-shift = <2>; /* lines 9:2 are used */ + hsync-active = <0>; + vsync-active = <0>; + pclk-sample = <1>; + }; + }; + }; +}; + +&csi { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_csi1>; + status = "okay"; + + port { + parallel_from_ov5640: endpoint { + remote-endpoint = <&ov5640_to_parallel>; + bus-type = <5>; /* Parallel bus */ + }; + }; }; &fec1 { @@ -353,9 +388,15 @@ &iomuxc { pinctrl-names = "default"; - pinctrl_csi1: csi1grp { + pinctrl_camera_clock: cameraclockgrp { fsl,pins = < MX6UL_PAD_CSI_MCLK__CSI_MCLK 0x1b088 + >; + }; + + pinctrl_csi1: csi1grp { + fsl,pins = < + MX6UL_PAD_CSI_PIXCLK__CSI_PIXCLK 0x1b088 MX6UL_PAD_CSI_VSYNC__CSI_VSYNC 0x1b088 MX6UL_PAD_CSI_HSYNC__CSI_HSYNC 0x1b088 -- 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] 15+ messages in thread
* Re: [PATCH 4/4] ARM: dts: imx6ul-14x14-evk: Add camera support 2021-01-07 23:50 ` [PATCH 4/4] ARM: dts: imx6ul-14x14-evk: Add camera support Fabio Estevam @ 2021-01-18 0:11 ` Shawn Guo 0 siblings, 0 replies; 15+ messages in thread From: Shawn Guo @ 2021-01-18 0:11 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-arm-kernel, andrew, linux-imx, kernel On Thu, Jan 07, 2021 at 08:50:44PM -0300, Fabio Estevam wrote: > Add support for the OV5640 parallel camera module. > > Signed-off-by: Fabio Estevam <festevam@gmail.com> > --- > arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 43 ++++++++++++++++++++++++- > 1 file changed, 42 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > index ac2ec96f54e7..ad039821a767 100644 > --- a/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > +++ b/arch/arm/boot/dts/imx6ul-14x14-evk.dtsi > @@ -146,6 +146,41 @@ > reg = <0x1a>; > wlf,shared-lrclk; > }; > + > + camera@3c { > + compatible = "ovti,ov5640"; > + reg = <0x3c>; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_camera_clock>; > + clocks = <&clks IMX6UL_CLK_CSI>; > + clock-names = "xclk"; > + powerdown-gpios = <&gpio_spi 6 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&gpio_spi 5 GPIO_ACTIVE_LOW>; > + > + port { > + ov5640_to_parallel: endpoint { > + remote-endpoint = <¶llel_from_ov5640>; > + bus-width = <8>; > + data-shift = <2>; /* lines 9:2 are used */ > + hsync-active = <0>; > + vsync-active = <0>; > + pclk-sample = <1>; > + }; > + }; > + }; > +}; > + > +&csi { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_csi1>; > + status = "okay"; > + > + port { > + parallel_from_ov5640: endpoint { > + remote-endpoint = <&ov5640_to_parallel>; > + bus-type = <5>; /* Parallel bus */ > + }; > + }; > }; > > &fec1 { > @@ -353,9 +388,15 @@ > &iomuxc { > pinctrl-names = "default"; > > - pinctrl_csi1: csi1grp { > + pinctrl_camera_clock: cameraclockgrp { > fsl,pins = < > MX6UL_PAD_CSI_MCLK__CSI_MCLK 0x1b088 > + >; > + }; > + > + pinctrl_csi1: csi1grp { > + fsl,pins = < > + Unneeded newline. Fixed it up and applied. Shawn > MX6UL_PAD_CSI_PIXCLK__CSI_PIXCLK 0x1b088 > MX6UL_PAD_CSI_VSYNC__CSI_VSYNC 0x1b088 > MX6UL_PAD_CSI_HSYNC__CSI_HSYNC 0x1b088 > -- > 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 [flat|nested] 15+ messages in thread
* Re: [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity 2021-01-07 23:50 [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Fabio Estevam ` (2 preceding siblings ...) 2021-01-07 23:50 ` [PATCH 4/4] ARM: dts: imx6ul-14x14-evk: Add camera support Fabio Estevam @ 2021-01-18 0:09 ` Shawn Guo 3 siblings, 0 replies; 15+ messages in thread From: Shawn Guo @ 2021-01-18 0:09 UTC (permalink / raw) To: Fabio Estevam; +Cc: linux-arm-kernel, andrew, linux-imx, kernel On Thu, Jan 07, 2021 at 08:50:41PM -0300, Fabio Estevam wrote: > The GPIO expander SPI chipselect is active low. Mark it as such to avoid > the following warning: > > [ 6.839213] gpio@0 enforce active low on chipselect handle > > Signed-off-by: Fabio Estevam <festevam@gmail.com> Applied, 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] 15+ messages in thread
end of thread, other threads:[~2021-03-12 11:36 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-01-07 23:50 [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Fabio Estevam 2021-01-07 23:50 ` [PATCH 2/4] ARM: dts: imx6ul-14x14-evk: Enable the GPIO expander Fabio Estevam 2021-01-08 1:02 ` Andrew Lunn 2021-01-18 0:10 ` Shawn Guo 2021-01-07 23:50 ` [PATCH 3/4] ARM: dts: imx6ul-14x14-evk: Describe the KSZ8081 reset Fabio Estevam 2021-01-08 1:03 ` Andrew Lunn 2021-01-18 0:10 ` Shawn Guo 2021-03-10 11:04 ` Joakim Zhang 2021-03-11 11:33 ` Joakim Zhang 2021-03-11 14:17 ` Andrew Lunn 2021-03-12 2:02 ` Joakim Zhang 2021-03-12 11:34 ` Fabio Estevam 2021-01-07 23:50 ` [PATCH 4/4] ARM: dts: imx6ul-14x14-evk: Add camera support Fabio Estevam 2021-01-18 0:11 ` Shawn Guo 2021-01-18 0:09 ` [PATCH 1/4] ARM: dts: imx6ul-14x14-evk: Fix SPI chipselect polarity Shawn Guo
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).