Hello Shiraz/Viresh, > -----Original Message----- > From: wens213@xxxxxxxxx [mailto:wens213@xxxxxxxxx] On Behalf Of > Chen-Yu Tsai > Sent: Thursday, December 12, 2013 9:37 AM > To: Rajeev KUMAR > Cc: Jingoo Han; Pratyush ANAND; devicetree@xxxxxxxxxxxxxxx; Mohit > KUMAR DCG; spear-devel; Viresh Kumar; linux-pci@xxxxxxxxxxxxxxx; linux- > arm-kernel@xxxxxxxxxxxxxxxxxxx; Florian Fainelli > Subject: Re: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy- > addr > > Hi, > > On Thu, Dec 12, 2013 at 11:59 AM, Rajeev kumar <rajeev- > dlh.kumar@xxxxxx> wrote: > > On 12/12/2013 6:09 AM, Jingoo Han wrote: > >> > >> On Wednesday, December 11, 2013 8:09 PM, Rajeev kumar wrote: > >>> > >>> On 12/11/2013 3:08 PM, Mohit KUMAR wrote: > >>>> > >>>> DT field name for the phy address changed since kernel 3.10. Set > >>>> the snps,phy-addr to 0xffffffff so that the driver probes for the phy. > >>>> > >>>> Signed-off-by: Mohit Kumar<mohit.kumar@xxxxxx> > >>>> Cc: Pratyush Anand<pratyush.anand@xxxxxx> > >>>> Cc: Viresh Kumar<viresh.linux@xxxxxxxxx> > >>>> Cc: spear-devel@xxxxxxxxxxx > >>>> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > >>>> Cc: devicetree@xxxxxxxxxxxxxxx > >>>> --- > >>>> arch/arm/boot/dts/spear13xx.dtsi | 1 + > >>>> 1 files changed, 1 insertions(+), 0 deletions(-) > >>>> > >>>> diff --git a/arch/arm/boot/dts/spear13xx.dtsi > >>>> b/arch/arm/boot/dts/spear13xx.dtsi > >>>> index 4382547..3518803 100644 > >>>> --- a/arch/arm/boot/dts/spear13xx.dtsi > >>>> +++ b/arch/arm/boot/dts/spear13xx.dtsi > >>>> @@ -155,6 +155,7 @@ > >>>> > >>>> gmac0: eth@e2000000 { > >>>> compatible = "st,spear600-gmac"; > >>>> + snps,phy-addr =<0xffffffff>; > >>> > >>> > >>> Don't you think it should be st,phy-addr =<0xffffffff>, as the > >>> manufacturer is 'st' as the compatible suggest > >> > >> > >> Hi Rajeev Kumar, > >> > >> According to the Documentation, it guides to use 'snps' prefix. > >> > >> In my humble opinion, > >> 'gmac' is a Synopsys based IP. Thus, if 'snps,phy-addr' is not ST > >> specific, 'snps' prefix can be used. > >> > > > > > > Very much true, I raise the point just becuase compatible = > > "st,spear600-gmac". Check the manufacturer name here. > > As documented, "st,spear600-gmac" is there for backwards compatibility. It > should really be updated to '"snps,dwmac-<version>", "snps,dwmac"'. > > > > > >> ./Documentation/devicetree/bindings/net/stmmac.txt > >> - snps,phy-addr phy address to connect to. > > About the "snps,phy-addr" property, Florian recently raised the issue that > this is !standard. I am currently implementing proper DT ethernet phy node > support for stmmac. Just a heads up. :) > - Any comment on this? Regards Mohit -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html