On 19-03-20 14:04:42, Fabio Estevam wrote: > Hi Steve, > > [Adding Andrew] > > On Wed, Mar 20, 2019 at 1:03 PM Steve Twiss > <stwiss.opensource@xxxxxxxxxxx> wrote: > > > > Hi Fabio, > > > > On 20 March 2019 12:17, Fabio Estevam wrote: > > > Subject: Re: [PATCH] ARM: dts: imx6qdl-sabresd: change phy-mode to use rgmii-id > > > > > > Hi Steve, > > > > > > On Wed, Mar 20, 2019 at 9:06 AM Steve Twiss > > > <stwiss.opensource@xxxxxxxxxxx> wrote: > > > > > > > > The PHY used on the Freescale i.MX6Q/DL SABRE boards is qualified as > > > > 'rgmii' instead of 'rgmii-id'. Meaning the RX and TX delays that were > > > > > > This patch declares it as 'rgmii-id', which contradicts the commit log. > > > > The PHY used on the Freescale i.MX6Q/DL SABRE boards is qualified as > > 'rgmii' instead of 'rgmii-id'. > > > > Can that be read two ways? > > I meant the phy-mode is currently qualified as 'rgmii' instead of what it > > should be: 'rgmii-id'. > > Thanks for the clarification. > > > > > [...] > > > > > > This patch fixes the network problems seen on the Freescale i.MX6Q/DL > > > > > > Please provide a Fixes tag. It would be good to know if this fix needs > > > to be applied to older kernels. > > > > I didn't put a Fixes tag in because I find this patch fixes v5.1-rc1 (which I see as broken). > > But the patch is not needed in v5.0 because I see that as working. > > So didn't see the need for a Cc: stable. > > It seems we have other boards that need to be fixed and we can not > have an old dtb with functional Ethernet with a new kernel. > > Does anyone know if this issue is AR8031 specific? I can confirm the same fix is works on imx6sx too. > > Thanks