[PATCH] ARM: dts: imx6qdl-sabresd: Fix AR8031 phy-mode

Tom Rini trini at konsulko.com
Wed Jun 17 19:39:12 CEST 2020


On Wed, Jun 17, 2020 at 02:33:16PM -0300, Fabio Estevam wrote:

> As per kernel commit 0672d22a1924 ("ARM: dts: imx: Fix the AR803X phy-mode)
> the correct phy-mode should be "rgmii-id", so fix it accordingly
> to fix the Ethernet regression.
> 
> This problem has been exposed by commit:
> 
> commit 13114f38e2ccea9386726d8b9831dfc310589548
> Author: Vladimir Oltean <vladimir.oltean at nxp.com>
> Date:   Thu May 7 00:11:51 2020 +0200
> 
>     phy: atheros: Explicitly disable RGMII delays
>     
>     To eliminate any doubts about the out-of-reset value of the PHY, that
>     the driver previously relied on.
>     
>     If bisecting shows that this commit breaks your board you probably have
>     a wrong PHY interface mode. You probably want the
>     PHY_INTERFACE_MODE_RGMII_RXID or PHY_INTERFACE_MODE_RGMII_ID mode.
>     
>     Signed-off-by: Vladimir Oltean <vladimir.oltean at nxp.com>
>     Acked-by: Joe Hershberger <joe.hershberger at ni.com>
> 
> Signed-off-by: Fabio Estevam <festevam at gmail.com>
> ---
> Hi,
> 
> Probably syncing with the mainline kernel would help fixing other boards.
> 
> I tried this approach and noticed some dtb build breakage as there are some
> dtbs that are not in mainline, so to be on the safe side, apply the minimal
> fix for the sabresd board to fix the regression for 2020.07.

As part of tracking down the cubox problem yesterday before posting I
found that some parts of your commit 0672d22a1924 in the kernel applied
in U-Boot, mainly, ah, the sabre* boards.  Others were already fixed or
absent.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200617/ffac7e94/attachment.sig>


More information about the U-Boot mailing list