* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-26 14:54 ` Oliver Graute
0 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-26 14:54 UTC (permalink / raw)
To: Peng Fan
Cc: catalin.marinas@arm.com, will@kernel.org, Anson Huang,
linux-kernel@vger.kernel.org, mripard@kernel.org,
bjorn.andersson@linaro.org, dinguyen@kernel.org, dl-linux-imx,
olof@lixom.net, Leonard Crestez, shawnguo@kernel.org,
linux-arm-kernel@lists.infradead.org
On 04/11/19, Peng Fan wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> With phy-reset-gpios are enabled for i.MX8MM-EVK board, phy
> will be reset. Without CONFIG_AT803X_PHY as y, board will stop
> booting in NFS DHCP, because phy is not ready. So mark
> CONFIG_AT803X_PHY from m to y to make board boot when using nfs rootfs.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> arch/arm64/configs/defconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index c9a867ac32d4..cd778c9ea8a4 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -285,7 +285,7 @@ CONFIG_SNI_AVE=y
> CONFIG_SNI_NETSEC=y
> CONFIG_STMMAC_ETH=m
> CONFIG_MDIO_BUS_MUX_MMIOREG=y
> -CONFIG_AT803X_PHY=m
> +CONFIG_AT803X_PHY=y
> CONFIG_MARVELL_PHY=m
> CONFIG_MARVELL_10G_PHY=m
> CONFIG_MESON_GXL_PHY=m
> --
> 2.16.4
Hello Peng,
this patch broke my imx8qm nfs setup. With the generic phy driver my
board is booting fine. But with the AT803X_PHY=y enabled I'am running
into the following phy issue. So on my side it looks inverse as on
yours. What is the best proposal to fix this?
[ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
[ 5.573206] Sending DHCP requests ...... timed out!
[ 95.339702] IP-Config: Retrying forever (NFS root)...
[ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
[ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
[ 99.461206] Sending DHCP requests ...... timed out!
[ 174.419639] IP-Config: Retrying forever (NFS root)...
[ 174.428834] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
[ 178.542418] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
[ 178.565206] Sending DHCP requests .....
[ 209.261271] random: crng init done
[ 230.565202] . timed out!
[ 260.577340] IP-Config: Retrying forever (NFS root)...
[ 260.586497] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
[ 264.686438] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
[ 264.709206] Sending DHCP requests ...... timed out!
[ 339.259701] IP-Config: Retrying forever (NFS root)...
[ 339.268835] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
[ 343.374422] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
[ 343.405206] Sending DHCP requests ...... timed out!
[ 433.171676] IP-Config: Retrying forever (NFS root)...
[ 433.180842] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
[ 437.294439] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
[ 437.317206] Sending DHCP requests ...... timed out!
[ 509.003660] IP-Config: Retrying forever (NFS root)...
[ 509.012836] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
[ 513.102416] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
Best Regards,
Oliver
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-26 14:54 ` Oliver Graute
@ 2019-11-27 8:47 ` Marc Gonzalez
-1 siblings, 0 replies; 20+ messages in thread
From: Marc Gonzalez @ 2019-11-27 8:47 UTC (permalink / raw)
To: Oliver Graute
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx, Linux ARM,
Heiner Kallweit
On 26/11/2019 15:54, Oliver Graute wrote:
> this patch broke my imx8qm nfs setup. With the generic phy driver my
> board is booting fine. But with the AT803X_PHY=y enabled I'm running
> into the following phy issue. So on my side it looks inverse as on
> yours. What is the best proposal to fix this?
>
> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 5.573206] Sending DHCP requests ...... timed out!
> [ 95.339702] IP-Config: Retrying forever (NFS root)...
> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 99.461206] Sending DHCP requests ...... timed out!
Which DTS are you using?
I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
Regards.
_______________________________________________
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] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-27 8:47 ` Marc Gonzalez
0 siblings, 0 replies; 20+ messages in thread
From: Marc Gonzalez @ 2019-11-27 8:47 UTC (permalink / raw)
To: Oliver Graute
Cc: Peng Fan, Anson Huang, dl-linux-imx, Linux ARM, LKML,
Florian Fainelli, Andrew Lunn, Russell King, André Draszik,
Heiner Kallweit
On 26/11/2019 15:54, Oliver Graute wrote:
> this patch broke my imx8qm nfs setup. With the generic phy driver my
> board is booting fine. But with the AT803X_PHY=y enabled I'm running
> into the following phy issue. So on my side it looks inverse as on
> yours. What is the best proposal to fix this?
>
> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 5.573206] Sending DHCP requests ...... timed out!
> [ 95.339702] IP-Config: Retrying forever (NFS root)...
> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 99.461206] Sending DHCP requests ...... timed out!
Which DTS are you using?
I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
Regards.
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-27 8:47 ` Marc Gonzalez
@ 2019-11-27 12:46 ` Oliver Graute
-1 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-27 12:46 UTC (permalink / raw)
To: Marc Gonzalez
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx, Linux ARM,
Heiner Kallweit
On 27/11/19, Marc Gonzalez wrote:
> On 26/11/2019 15:54, Oliver Graute wrote:
>
> > this patch broke my imx8qm nfs setup. With the generic phy driver my
> > board is booting fine. But with the AT803X_PHY=y enabled I'm running
> > into the following phy issue. So on my side it looks inverse as on
> > yours. What is the best proposal to fix this?
> >
> > [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> > [ 5.573206] Sending DHCP requests ...... timed out!
> > [ 95.339702] IP-Config: Retrying forever (NFS root)...
> > [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> > [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> > [ 99.461206] Sending DHCP requests ...... timed out!
>
> Which DTS are you using?
I'am using this DTS which I'am currently working on:
https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
>
> I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
So what should I fix in my device tree?
Best regards,
Oliver
_______________________________________________
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] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-27 12:46 ` Oliver Graute
0 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-27 12:46 UTC (permalink / raw)
To: Marc Gonzalez
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx, Linux ARM,
Heiner Kallweit
On 27/11/19, Marc Gonzalez wrote:
> On 26/11/2019 15:54, Oliver Graute wrote:
>
> > this patch broke my imx8qm nfs setup. With the generic phy driver my
> > board is booting fine. But with the AT803X_PHY=y enabled I'm running
> > into the following phy issue. So on my side it looks inverse as on
> > yours. What is the best proposal to fix this?
> >
> > [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> > [ 5.573206] Sending DHCP requests ...... timed out!
> > [ 95.339702] IP-Config: Retrying forever (NFS root)...
> > [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> > [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> > [ 99.461206] Sending DHCP requests ...... timed out!
>
> Which DTS are you using?
I'am using this DTS which I'am currently working on:
https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
>
> I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
So what should I fix in my device tree?
Best regards,
Oliver
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-27 12:46 ` Oliver Graute
@ 2019-11-27 12:56 ` Fabio Estevam
-1 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2019-11-27 12:56 UTC (permalink / raw)
To: Oliver Graute
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
Marc Gonzalez, André Draszik, LKML, Russell King,
dl-linux-imx, Linux ARM, Heiner Kallweit
Hi Oliver,
On Wed, Nov 27, 2019 at 9:47 AM Oliver Graute <oliver.graute@gmail.com> wrote:
> I'am using this DTS which I'am currently working on:
>
> https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
> >
> > I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
>
> So what should I fix in my device tree?
Some suggestions you could try:
- Try to use phy-mode = "rgmii-id"; instead,
- The PHY address 0 does not match the reg value of 4, so you need to
double check the PHY address and make the @ and reg values to match.
- If you have a GPIO connected to the Ethernet PHY reset pin, then you
should describe it in the dts and also provide a delay as per the
AR803X datasheet.
Regards,
Fabio Estevam
_______________________________________________
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] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-27 12:56 ` Fabio Estevam
0 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2019-11-27 12:56 UTC (permalink / raw)
To: Oliver Graute
Cc: Marc Gonzalez, Andrew Lunn, Peng Fan, Florian Fainelli,
Anson Huang, André Draszik, LKML, Russell King, dl-linux-imx,
Linux ARM, Heiner Kallweit
Hi Oliver,
On Wed, Nov 27, 2019 at 9:47 AM Oliver Graute <oliver.graute@gmail.com> wrote:
> I'am using this DTS which I'am currently working on:
>
> https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
> >
> > I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
>
> So what should I fix in my device tree?
Some suggestions you could try:
- Try to use phy-mode = "rgmii-id"; instead,
- The PHY address 0 does not match the reg value of 4, so you need to
double check the PHY address and make the @ and reg values to match.
- If you have a GPIO connected to the Ethernet PHY reset pin, then you
should describe it in the dts and also provide a delay as per the
AR803X datasheet.
Regards,
Fabio Estevam
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-27 12:56 ` Fabio Estevam
@ 2019-11-28 10:06 ` Oliver Graute
-1 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-28 10:06 UTC (permalink / raw)
To: Fabio Estevam
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
Marc Gonzalez, André Draszik, LKML, Russell King,
dl-linux-imx, Linux ARM, Heiner Kallweit
On 27/11/19, Fabio Estevam wrote:
> Hi Oliver,
>
> On Wed, Nov 27, 2019 at 9:47 AM Oliver Graute <oliver.graute@gmail.com> wrote:
>
> > I'am using this DTS which I'am currently working on:
> >
> > https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
> > >
> > > I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
> >
> > So what should I fix in my device tree?
>
> Some suggestions you could try:
>
> - Try to use phy-mode = "rgmii-id"; instead,
> - The PHY address 0 does not match the reg value of 4, so you need to
> double check the PHY address and make the @ and reg values to match.
ok I fix that in my dts.
> - If you have a GPIO connected to the Ethernet PHY reset pin, then you
> should describe it in the dts and also provide a delay as per the
> AR803X datasheet.
it seems that currently no ethernet phy reset gpio is connected on that
advantech eval board. The vendor says there is a optional resistor which can
be mounted.
Best regards,
Oliver
_______________________________________________
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] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-28 10:06 ` Oliver Graute
0 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-28 10:06 UTC (permalink / raw)
To: Fabio Estevam
Cc: Marc Gonzalez, Andrew Lunn, Peng Fan, Florian Fainelli,
Anson Huang, André Draszik, LKML, Russell King, dl-linux-imx,
Linux ARM, Heiner Kallweit
On 27/11/19, Fabio Estevam wrote:
> Hi Oliver,
>
> On Wed, Nov 27, 2019 at 9:47 AM Oliver Graute <oliver.graute@gmail.com> wrote:
>
> > I'am using this DTS which I'am currently working on:
> >
> > https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
> > >
> > > I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
> >
> > So what should I fix in my device tree?
>
> Some suggestions you could try:
>
> - Try to use phy-mode = "rgmii-id"; instead,
> - The PHY address 0 does not match the reg value of 4, so you need to
> double check the PHY address and make the @ and reg values to match.
ok I fix that in my dts.
> - If you have a GPIO connected to the Ethernet PHY reset pin, then you
> should describe it in the dts and also provide a delay as per the
> AR803X datasheet.
it seems that currently no ethernet phy reset gpio is connected on that
advantech eval board. The vendor says there is a optional resistor which can
be mounted.
Best regards,
Oliver
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-27 12:46 ` Oliver Graute
@ 2019-11-28 9:11 ` Marc Gonzalez
-1 siblings, 0 replies; 20+ messages in thread
From: Marc Gonzalez @ 2019-11-28 9:11 UTC (permalink / raw)
To: Oliver Graute
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx,
Fabio Estevam, Linux ARM, Heiner Kallweit
On 27/11/2019 13:46, Oliver Graute wrote:
> On 27/11/19, Marc Gonzalez wrote:
>
>> On 26/11/2019 15:54, Oliver Graute wrote:
>>
>>> this patch broke my imx8qm nfs setup. With the generic phy driver my
>>> board is booting fine. But with the AT803X_PHY=y enabled I'm running
>>> into the following phy issue. So on my side it looks inverse as on
>>> yours. What is the best proposal to fix this?
>>>
>>> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>>> [ 5.573206] Sending DHCP requests ...... timed out!
>>> [ 95.339702] IP-Config: Retrying forever (NFS root)...
>>> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
>>> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>>> [ 99.461206] Sending DHCP requests ...... timed out!
>>
>> Which DTS are you using?
>
> I'am using this DTS which I'am currently working on:
>
> https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
>>
>> I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
>
> So what should I fix in my device tree?
In the board DTS I used to work on, I had this:
ð0 {
phy-connection-type = "rgmii-id";
phy-handle = <ð0_phy>;
#address-cells = <1>;
#size-cells = <0>;
/* Atheros AR8035 */
eth0_phy: ethernet-phy@4 {
compatible = "ethernet-phy-id004d.d072",
"ethernet-phy-ieee802.3-c22";
interrupts = <37 IRQ_TYPE_EDGE_RISING>;
reg = <4>;
};
};
In your DTS, you #include "imx8qm.dtsi"
I found no such file:
$ git ls-files | grep imx8qm
drivers/pinctrl/freescale/pinctrl-imx8qm.c
include/dt-bindings/pinctrl/pads-imx8qm.h
In your patch:
https://patchwork.kernel.org/patch/11211567/
+&fec1 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_fec1>;
+ phy-mode = "rgmii-txid";
+ phy-handle = <ðphy0>;
+ fsl,magic-packet;
+ fsl,rgmii_rxc_dly;
+ status = "okay";
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ ethphy0: ethernet-phy@0 {
+ compatible = "ethernet-phy-ieee802.3-c22";
+ reg = <4>;
+ at803x,eee-disabled;
+ at803x,vddio-1p8v;
+ };
+ };
+};
Try all possible 'phy-mode' (rgmii, rgmii-id, rgmii-rxid, rgmii-txid)
Investigate 'fsl,rgmii_rxc_dly' (it's not a standard Linux DT prop)
Documentation/devicetree/bindings/net/ethernet-controller.yaml
Regards.
_______________________________________________
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] 20+ messages in thread* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-28 9:11 ` Marc Gonzalez
0 siblings, 0 replies; 20+ messages in thread
From: Marc Gonzalez @ 2019-11-28 9:11 UTC (permalink / raw)
To: Oliver Graute
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx, Linux ARM,
Heiner Kallweit, Fabio Estevam
On 27/11/2019 13:46, Oliver Graute wrote:
> On 27/11/19, Marc Gonzalez wrote:
>
>> On 26/11/2019 15:54, Oliver Graute wrote:
>>
>>> this patch broke my imx8qm nfs setup. With the generic phy driver my
>>> board is booting fine. But with the AT803X_PHY=y enabled I'm running
>>> into the following phy issue. So on my side it looks inverse as on
>>> yours. What is the best proposal to fix this?
>>>
>>> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>>> [ 5.573206] Sending DHCP requests ...... timed out!
>>> [ 95.339702] IP-Config: Retrying forever (NFS root)...
>>> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
>>> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>>> [ 99.461206] Sending DHCP requests ...... timed out!
>>
>> Which DTS are you using?
>
> I'am using this DTS which I'am currently working on:
>
> https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
>>
>> I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
>
> So what should I fix in my device tree?
In the board DTS I used to work on, I had this:
ð0 {
phy-connection-type = "rgmii-id";
phy-handle = <ð0_phy>;
#address-cells = <1>;
#size-cells = <0>;
/* Atheros AR8035 */
eth0_phy: ethernet-phy@4 {
compatible = "ethernet-phy-id004d.d072",
"ethernet-phy-ieee802.3-c22";
interrupts = <37 IRQ_TYPE_EDGE_RISING>;
reg = <4>;
};
};
In your DTS, you #include "imx8qm.dtsi"
I found no such file:
$ git ls-files | grep imx8qm
drivers/pinctrl/freescale/pinctrl-imx8qm.c
include/dt-bindings/pinctrl/pads-imx8qm.h
In your patch:
https://patchwork.kernel.org/patch/11211567/
+&fec1 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_fec1>;
+ phy-mode = "rgmii-txid";
+ phy-handle = <ðphy0>;
+ fsl,magic-packet;
+ fsl,rgmii_rxc_dly;
+ status = "okay";
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ ethphy0: ethernet-phy@0 {
+ compatible = "ethernet-phy-ieee802.3-c22";
+ reg = <4>;
+ at803x,eee-disabled;
+ at803x,vddio-1p8v;
+ };
+ };
+};
Try all possible 'phy-mode' (rgmii, rgmii-id, rgmii-rxid, rgmii-txid)
Investigate 'fsl,rgmii_rxc_dly' (it's not a standard Linux DT prop)
Documentation/devicetree/bindings/net/ethernet-controller.yaml
Regards.
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-28 9:11 ` Marc Gonzalez
@ 2019-11-28 12:16 ` Oliver Graute
-1 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-28 12:16 UTC (permalink / raw)
To: Marc Gonzalez
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx,
Fabio Estevam, Linux ARM, Heiner Kallweit
On 28/11/19, Marc Gonzalez wrote:
> On 27/11/2019 13:46, Oliver Graute wrote:
>
> > On 27/11/19, Marc Gonzalez wrote:
> >
> >> On 26/11/2019 15:54, Oliver Graute wrote:
> >>
> >>> this patch broke my imx8qm nfs setup. With the generic phy driver my
> >>> board is booting fine. But with the AT803X_PHY=y enabled I'm running
> >>> into the following phy issue. So on my side it looks inverse as on
> >>> yours. What is the best proposal to fix this?
> >>>
> >>> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> >>> [ 5.573206] Sending DHCP requests ...... timed out!
> >>> [ 95.339702] IP-Config: Retrying forever (NFS root)...
> >>> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> >>> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> >>> [ 99.461206] Sending DHCP requests ...... timed out!
> >>
> >> Which DTS are you using?
> >
> > I'am using this DTS which I'am currently working on:
> >
> > https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
> >>
> >> I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
> >
> > So what should I fix in my device tree?
>
> In the board DTS I used to work on, I had this:
>
> ð0 {
> phy-connection-type = "rgmii-id";
> phy-handle = <ð0_phy>;
> #address-cells = <1>;
> #size-cells = <0>;
>
> /* Atheros AR8035 */
> eth0_phy: ethernet-phy@4 {
> compatible = "ethernet-phy-id004d.d072",
> "ethernet-phy-ieee802.3-c22";
> interrupts = <37 IRQ_TYPE_EDGE_RISING>;
> reg = <4>;
> };
> };
>
> In your DTS, you #include "imx8qm.dtsi"
> I found no such file:
> $ git ls-files | grep imx8qm
yes this file is not yet added to Shawn Guos next tree.
Latest patch can be found here:
https://patchwork.kernel.org/patch/11248331/
>
> In your patch:
> https://patchwork.kernel.org/patch/11211567/
>
> +&fec1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_fec1>;
> + phy-mode = "rgmii-txid";
> + phy-handle = <ðphy0>;
> + fsl,magic-packet;
> + fsl,rgmii_rxc_dly;
> + status = "okay";
> +
> + mdio {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + ethphy0: ethernet-phy@0 {
> + compatible = "ethernet-phy-ieee802.3-c22";
> + reg = <4>;
> + at803x,eee-disabled;
> + at803x,vddio-1p8v;
> + };
> + };
> +};
>
> Try all possible 'phy-mode' (rgmii, rgmii-id, rgmii-rxid, rgmii-txid)
> Investigate 'fsl,rgmii_rxc_dly' (it's not a standard Linux DT prop)
> Documentation/devicetree/bindings/net/ethernet-controller.yaml
thx for this hint.
Best regards,
Oliver
_______________________________________________
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] 20+ messages in thread* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-11-28 12:16 ` Oliver Graute
0 siblings, 0 replies; 20+ messages in thread
From: Oliver Graute @ 2019-11-28 12:16 UTC (permalink / raw)
To: Marc Gonzalez
Cc: Andrew Lunn, Peng Fan, Florian Fainelli, Anson Huang,
André Draszik, LKML, Russell King, dl-linux-imx, Linux ARM,
Heiner Kallweit, Fabio Estevam
On 28/11/19, Marc Gonzalez wrote:
> On 27/11/2019 13:46, Oliver Graute wrote:
>
> > On 27/11/19, Marc Gonzalez wrote:
> >
> >> On 26/11/2019 15:54, Oliver Graute wrote:
> >>
> >>> this patch broke my imx8qm nfs setup. With the generic phy driver my
> >>> board is booting fine. But with the AT803X_PHY=y enabled I'm running
> >>> into the following phy issue. So on my side it looks inverse as on
> >>> yours. What is the best proposal to fix this?
> >>>
> >>> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> >>> [ 5.573206] Sending DHCP requests ...... timed out!
> >>> [ 95.339702] IP-Config: Retrying forever (NFS root)...
> >>> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> >>> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> >>> [ 99.461206] Sending DHCP requests ...... timed out!
> >>
> >> Which DTS are you using?
> >
> > I'am using this DTS which I'am currently working on:
> >
> > https://lists.infradead.org/pipermail/linux-arm-kernel/2019-October/689501.html
> >>
> >> I bet one dollar that 6d4cd041f0af triggered a latent bug in the DTS.
> >
> > So what should I fix in my device tree?
>
> In the board DTS I used to work on, I had this:
>
> ð0 {
> phy-connection-type = "rgmii-id";
> phy-handle = <ð0_phy>;
> #address-cells = <1>;
> #size-cells = <0>;
>
> /* Atheros AR8035 */
> eth0_phy: ethernet-phy@4 {
> compatible = "ethernet-phy-id004d.d072",
> "ethernet-phy-ieee802.3-c22";
> interrupts = <37 IRQ_TYPE_EDGE_RISING>;
> reg = <4>;
> };
> };
>
> In your DTS, you #include "imx8qm.dtsi"
> I found no such file:
> $ git ls-files | grep imx8qm
yes this file is not yet added to Shawn Guos next tree.
Latest patch can be found here:
https://patchwork.kernel.org/patch/11248331/
>
> In your patch:
> https://patchwork.kernel.org/patch/11211567/
>
> +&fec1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_fec1>;
> + phy-mode = "rgmii-txid";
> + phy-handle = <ðphy0>;
> + fsl,magic-packet;
> + fsl,rgmii_rxc_dly;
> + status = "okay";
> +
> + mdio {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + ethphy0: ethernet-phy@0 {
> + compatible = "ethernet-phy-ieee802.3-c22";
> + reg = <4>;
> + at803x,eee-disabled;
> + at803x,vddio-1p8v;
> + };
> + };
> +};
>
> Try all possible 'phy-mode' (rgmii, rgmii-id, rgmii-rxid, rgmii-txid)
> Investigate 'fsl,rgmii_rxc_dly' (it's not a standard Linux DT prop)
> Documentation/devicetree/bindings/net/ethernet-controller.yaml
thx for this hint.
Best regards,
Oliver
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
2019-11-26 14:54 ` Oliver Graute
@ 2019-12-02 8:38 ` Shawn Guo
-1 siblings, 0 replies; 20+ messages in thread
From: Shawn Guo @ 2019-12-02 8:38 UTC (permalink / raw)
To: Oliver Graute
Cc: Peng Fan, Anson Huang, catalin.marinas@arm.com,
linux-kernel@vger.kernel.org, mripard@kernel.org,
bjorn.andersson@linaro.org, dinguyen@kernel.org, dl-linux-imx,
olof@lixom.net, Leonard Crestez, will@kernel.org,
linux-arm-kernel@lists.infradead.org
On Tue, Nov 26, 2019 at 03:54:50PM +0100, Oliver Graute wrote:
> On 04/11/19, Peng Fan wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > With phy-reset-gpios are enabled for i.MX8MM-EVK board, phy
> > will be reset. Without CONFIG_AT803X_PHY as y, board will stop
> > booting in NFS DHCP, because phy is not ready. So mark
> > CONFIG_AT803X_PHY from m to y to make board boot when using nfs rootfs.
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> > arch/arm64/configs/defconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index c9a867ac32d4..cd778c9ea8a4 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -285,7 +285,7 @@ CONFIG_SNI_AVE=y
> > CONFIG_SNI_NETSEC=y
> > CONFIG_STMMAC_ETH=m
> > CONFIG_MDIO_BUS_MUX_MMIOREG=y
> > -CONFIG_AT803X_PHY=m
> > +CONFIG_AT803X_PHY=y
> > CONFIG_MARVELL_PHY=m
> > CONFIG_MARVELL_10G_PHY=m
> > CONFIG_MESON_GXL_PHY=m
> > --
> > 2.16.4
>
> Hello Peng,
>
> this patch broke my imx8qm nfs setup. With the generic phy driver my
> board is booting fine. But with the AT803X_PHY=y enabled I'am running
> into the following phy issue. So on my side it looks inverse as on
> yours. What is the best proposal to fix this?
So you are saying your Atheros 8035 device works with generic phy
driver but not with AT803X_PHY driver? That makes no sense. Did you
add phy-reset-gpios property for your device?
Shawn
>
> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 5.573206] Sending DHCP requests ...... timed out!
> [ 95.339702] IP-Config: Retrying forever (NFS root)...
> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 99.461206] Sending DHCP requests ...... timed out!
> [ 174.419639] IP-Config: Retrying forever (NFS root)...
> [ 174.428834] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 178.542418] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 178.565206] Sending DHCP requests .....
> [ 209.261271] random: crng init done
> [ 230.565202] . timed out!
> [ 260.577340] IP-Config: Retrying forever (NFS root)...
> [ 260.586497] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 264.686438] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 264.709206] Sending DHCP requests ...... timed out!
> [ 339.259701] IP-Config: Retrying forever (NFS root)...
> [ 339.268835] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 343.374422] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 343.405206] Sending DHCP requests ...... timed out!
> [ 433.171676] IP-Config: Retrying forever (NFS root)...
> [ 433.180842] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 437.294439] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 437.317206] Sending DHCP requests ...... timed out!
> [ 509.003660] IP-Config: Retrying forever (NFS root)...
> [ 509.012836] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 513.102416] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>
> Best Regards,
>
> Oliver
_______________________________________________
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] 20+ messages in thread
* Re: [PATCH] arm64: defconfig: Change CONFIG_AT803X_PHY from m to y
@ 2019-12-02 8:38 ` Shawn Guo
0 siblings, 0 replies; 20+ messages in thread
From: Shawn Guo @ 2019-12-02 8:38 UTC (permalink / raw)
To: Oliver Graute
Cc: Peng Fan, catalin.marinas@arm.com, will@kernel.org, Anson Huang,
linux-kernel@vger.kernel.org, mripard@kernel.org,
bjorn.andersson@linaro.org, dinguyen@kernel.org, dl-linux-imx,
olof@lixom.net, Leonard Crestez,
linux-arm-kernel@lists.infradead.org
On Tue, Nov 26, 2019 at 03:54:50PM +0100, Oliver Graute wrote:
> On 04/11/19, Peng Fan wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > With phy-reset-gpios are enabled for i.MX8MM-EVK board, phy
> > will be reset. Without CONFIG_AT803X_PHY as y, board will stop
> > booting in NFS DHCP, because phy is not ready. So mark
> > CONFIG_AT803X_PHY from m to y to make board boot when using nfs rootfs.
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> > arch/arm64/configs/defconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index c9a867ac32d4..cd778c9ea8a4 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -285,7 +285,7 @@ CONFIG_SNI_AVE=y
> > CONFIG_SNI_NETSEC=y
> > CONFIG_STMMAC_ETH=m
> > CONFIG_MDIO_BUS_MUX_MMIOREG=y
> > -CONFIG_AT803X_PHY=m
> > +CONFIG_AT803X_PHY=y
> > CONFIG_MARVELL_PHY=m
> > CONFIG_MARVELL_10G_PHY=m
> > CONFIG_MESON_GXL_PHY=m
> > --
> > 2.16.4
>
> Hello Peng,
>
> this patch broke my imx8qm nfs setup. With the generic phy driver my
> board is booting fine. But with the AT803X_PHY=y enabled I'am running
> into the following phy issue. So on my side it looks inverse as on
> yours. What is the best proposal to fix this?
So you are saying your Atheros 8035 device works with generic phy
driver but not with AT803X_PHY driver? That makes no sense. Did you
add phy-reset-gpios property for your device?
Shawn
>
> [ 5.550442] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 5.573206] Sending DHCP requests ...... timed out!
> [ 95.339702] IP-Config: Retrying forever (NFS root)...
> [ 95.348873] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 99.438443] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 99.461206] Sending DHCP requests ...... timed out!
> [ 174.419639] IP-Config: Retrying forever (NFS root)...
> [ 174.428834] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 178.542418] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 178.565206] Sending DHCP requests .....
> [ 209.261271] random: crng init done
> [ 230.565202] . timed out!
> [ 260.577340] IP-Config: Retrying forever (NFS root)...
> [ 260.586497] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 264.686438] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 264.709206] Sending DHCP requests ...... timed out!
> [ 339.259701] IP-Config: Retrying forever (NFS root)...
> [ 339.268835] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 343.374422] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 343.405206] Sending DHCP requests ...... timed out!
> [ 433.171676] IP-Config: Retrying forever (NFS root)...
> [ 433.180842] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 437.294439] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
> [ 437.317206] Sending DHCP requests ...... timed out!
> [ 509.003660] IP-Config: Retrying forever (NFS root)...
> [ 509.012836] Atheros 8035 ethernet 5b040000.ethernet-1:06: attached PHY driver [Atheros 8035 ethernet] (mii_bus:phy_addr=5b040000.ethernet-1:06, irq=POLL)
> [ 513.102416] fec 5b040000.ethernet eth0: Link is Up - 1Gbps/Full - flow control off
>
> Best Regards,
>
> Oliver
^ permalink raw reply [flat|nested] 20+ messages in thread