[PATCH 2/3] powerpc/dts/fsl: t4240rdb: use the Cortina PHY driver compatible

[Camelia Alexandra Groza]

> -----Original Message-----
> From: Scott Wood <oss at buserror.net>
> Sent: Saturday, December 22, 2018 06:07
> To: Camelia Alexandra Groza <camelia.groza at nxp.com>;
> robh+dt at kernel.org; mark.rutland at arm.com; benh at kernel.crashing.org
> Cc: devicetree at vger.kernel.org; linux-kernel at vger.kernel.org;
> paulus at samba.org; linuxppc-dev at lists.ozlabs.org
> Subject: Re: [PATCH 2/3] powerpc/dts/fsl: t4240rdb: use the Cortina PHY
> driver compatible
> 
> On Wed, 2018-07-18 at 14:46 +0300, Camelia Groza wrote:
> > The Cortina PHY requires the use of the dedicated Cortina PHY driver
> > instead of the generic one.
> >
> > Signed-off-by: Camelia Groza <camelia.groza at nxp.com>
> > ---
> >  arch/powerpc/boot/dts/fsl/t4240rdb.dts | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/powerpc/boot/dts/fsl/t4240rdb.dts
> > b/arch/powerpc/boot/dts/fsl/t4240rdb.dts
> > index 15eb0a3..a56a705 100644
> > --- a/arch/powerpc/boot/dts/fsl/t4240rdb.dts
> > +++ b/arch/powerpc/boot/dts/fsl/t4240rdb.dts
> > @@ -267,22 +267,22 @@
> >
> >  			mdio at fd000 {
> >  				xfiphy1: ethernet-phy at 10 {
> > -					compatible = "ethernet-phy-
> ieee802.3-
> > c45";
> > +					compatible = "ethernet-phy-
> > id13e5.1002";
> >  					reg = <0x10>;
> >  				};
> >
> >  				xfiphy2: ethernet-phy at 11 {
> > -					compatible = "ethernet-phy-
> ieee802.3-
> > c45";
> > +					compatible = "ethernet-phy-
> > id13e5.1002";
> >  					reg = <0x11>;
> >  				};
> >
> >  				xfiphy3: ethernet-phy at 13 {
> > -					compatible = "ethernet-phy-
> ieee802.3-
> > c45";
> > +					compatible = "ethernet-phy-
> > id13e5.1002";
> >  					reg = <0x13>;
> >  				};
> >
> >  				xfiphy4: ethernet-phy at 12 {
> > -					compatible = "ethernet-phy-
> ieee802.3-
> > c45";
> > +					compatible = "ethernet-phy-
> > id13e5.1002";
> >  					reg = <0x12>;
> >  				};
> >  			};
> 
> I get crashes on boot when using a dtb with this change:

Hi

Sorry for the late replay. I was on vacation. I'm looking into it.

Regards,
Camelia