| Message ID | 1078549fa412fb34ab2c95a6232096570abee12d.1386752447.git.mohit.kumar@st.com |
|---|---|
| State | Changes Requested |
| Headers | show |
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@st.com> > Cc: Pratyush Anand<pratyush.anand@st.com> > Cc: Viresh Kumar<viresh.linux@gmail.com> > Cc: spear-devel@list.st.com > Cc: linux-arm-kernel@lists.infradead.org > Cc: devicetree@vger.kernel.org > --- > 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 ~Rajeev > reg =<0xe2000000 0x8000>; > interrupts =<0 33 0x4 > 0 34 0x4>; -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Recheck the subject line, you are not doing any correction. On 12/11/2013 4:39 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@st.com> >> Cc: Pratyush Anand<pratyush.anand@st.com> >> Cc: Viresh Kumar<viresh.linux@gmail.com> >> Cc: spear-devel@list.st.com >> Cc: linux-arm-kernel@lists.infradead.org >> Cc: devicetree@vger.kernel.org >> --- >> 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 > > ~Rajeev > > >> reg =<0xe2000000 0x8000>; >> interrupts =<0 33 0x4 >> 0 34 0x4>; > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@st.com> > > Cc: Pratyush Anand<pratyush.anand@st.com> > > Cc: Viresh Kumar<viresh.linux@gmail.com> > > Cc: spear-devel@list.st.com > > Cc: linux-arm-kernel@lists.infradead.org > > Cc: devicetree@vger.kernel.org > > --- > > 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. ./Documentation/devicetree/bindings/net/stmmac.txt - snps,phy-addr phy address to connect to. Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@st.com> >>> Cc: Pratyush Anand<pratyush.anand@st.com> >>> Cc: Viresh Kumar<viresh.linux@gmail.com> >>> Cc: spear-devel@list.st.com >>> Cc: linux-arm-kernel@lists.infradead.org >>> Cc: devicetree@vger.kernel.org >>> --- >>> 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. ~Rajeev > ./Documentation/devicetree/bindings/net/stmmac.txt > - snps,phy-addr phy address to connect to. > > Best regards, > Jingoo Han > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Thu, Dec 12, 2013 at 11:59 AM, Rajeev kumar <rajeev-dlh.kumar@st.com> 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@st.com> >>>> Cc: Pratyush Anand<pratyush.anand@st.com> >>>> Cc: Viresh Kumar<viresh.linux@gmail.com> >>>> Cc: spear-devel@list.st.com >>>> Cc: linux-arm-kernel@lists.infradead.org >>>> Cc: devicetree@vger.kernel.org >>>> --- >>>> 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. :) ChenYu -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello Shiraz/Viresh, > -----Original Message----- > From: wens213@gmail.com [mailto:wens213@gmail.com] On Behalf Of > Chen-Yu Tsai > Sent: Thursday, December 12, 2013 9:37 AM > To: Rajeev KUMAR > Cc: Jingoo Han; Pratyush ANAND; devicetree@vger.kernel.org; Mohit > KUMAR DCG; spear-devel; Viresh Kumar; linux-pci@vger.kernel.org; linux- > arm-kernel@lists.infradead.org; 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@st.com> 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@st.com> > >>>> Cc: Pratyush Anand<pratyush.anand@st.com> > >>>> Cc: Viresh Kumar<viresh.linux@gmail.com> > >>>> Cc: spear-devel@list.st.com > >>>> Cc: linux-arm-kernel@lists.infradead.org > >>>> Cc: devicetree@vger.kernel.org > >>>> --- > >>>> 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 linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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>; reg = <0xe2000000 0x8000>; interrupts = <0 33 0x4 0 34 0x4>;
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@st.com> Cc: Pratyush Anand <pratyush.anand@st.com> Cc: Viresh Kumar <viresh.linux@gmail.com> Cc: spear-devel@list.st.com Cc: linux-arm-kernel@lists.infradead.org Cc: devicetree@vger.kernel.org --- arch/arm/boot/dts/spear13xx.dtsi | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)