On Thu, Sep 30, 2021 at 05:56:29PM +0200, Frieder Schrempf wrote:

From: Frieder Schrempf frieder.schrempf@kontron.de

Previously we falsely relied on the PHY driver to unconditionally enable the internal RX delay. Since the following fix for the PHY driver this is not the case anymore:

commit 7b005a1742be ("net: phy: mscc: configure both RX and TX internal delays for RGMII")

In order to enable the delay we need to set the connection type to "rgmii-rxid".

Could you share some details of the issue that the delay is not enabled, e.g. how broken the Ethernet support is without this change?

Shawn

Fixes: 21c4f45b335f ("arm64: dts: Add the Kontron i.MX8M Mini SoMs and baseboards") Cc: stable@vger.kernel.org Signed-off-by: Frieder Schrempf frieder.schrempf@kontron.de

---

arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-s.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-s.dts b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-s.dts index dbf11e03ecce..0e4509287a92 100644 --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-s.dts +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-s.dts @@ -114,7 +114,7 @@ &ecspi3 { &fec1 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_enet>;

• phy-connection-type = "rgmii";

• phy-connection-type = "rgmii-rxid"; phy-handle = <&ethphy>; status = "okay";

2.33.0