* [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr [not found] <cover.1386752447.git.mohit.kumar@st.com> @ 2013-12-11 9:38 ` Mohit Kumar 2013-12-11 11:09 ` Rajeev kumar 0 siblings, 1 reply; 7+ messages in thread From: Mohit Kumar @ 2013-12-11 9:38 UTC (permalink / raw) To: linux-pci, linux-arm-kernel Cc: Mohit Kumar, devicetree, spear-devel, Viresh Kumar, Pratyush Anand 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>; reg = <0xe2000000 0x8000>; interrupts = <0 33 0x4 0 34 0x4>; -- 1.7.0.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr 2013-12-11 9:38 ` [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr Mohit Kumar @ 2013-12-11 11:09 ` Rajeev kumar 2013-12-11 11:14 ` Rajeev kumar 2013-12-12 0:39 ` Jingoo Han 0 siblings, 2 replies; 7+ messages in thread From: Rajeev kumar @ 2013-12-11 11:09 UTC (permalink / raw) To: Mohit KUMAR Cc: linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Pratyush ANAND, Viresh Kumar, spear-devel, devicetree@vger.kernel.org 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>; ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr 2013-12-11 11:09 ` Rajeev kumar @ 2013-12-11 11:14 ` Rajeev kumar 2013-12-12 0:39 ` Jingoo Han 1 sibling, 0 replies; 7+ messages in thread From: Rajeev kumar @ 2013-12-11 11:14 UTC (permalink / raw) To: Mohit KUMAR DCG Cc: linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Pratyush ANAND, Viresh Kumar, spear-devel, devicetree@vger.kernel.org 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>; > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr 2013-12-11 11:09 ` Rajeev kumar 2013-12-11 11:14 ` Rajeev kumar @ 2013-12-12 0:39 ` Jingoo Han 2013-12-12 3:59 ` Rajeev kumar 1 sibling, 1 reply; 7+ messages in thread From: Jingoo Han @ 2013-12-12 0:39 UTC (permalink / raw) To: 'Rajeev kumar', 'Mohit KUMAR' Cc: linux-pci, linux-arm-kernel, 'Pratyush ANAND', 'Viresh Kumar', 'spear-devel', devicetree, 'Jingoo Han' 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr 2013-12-12 0:39 ` Jingoo Han @ 2013-12-12 3:59 ` Rajeev kumar 2013-12-12 4:07 ` Chen-Yu Tsai 0 siblings, 1 reply; 7+ messages in thread From: Rajeev kumar @ 2013-12-12 3:59 UTC (permalink / raw) To: Jingoo Han Cc: Mohit KUMAR DCG, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Pratyush ANAND, 'Viresh Kumar', spear-devel, devicetree@vger.kernel.org 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 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr 2013-12-12 3:59 ` Rajeev kumar @ 2013-12-12 4:07 ` Chen-Yu Tsai 2013-12-12 5:13 ` Mohit KUMAR DCG 0 siblings, 1 reply; 7+ messages in thread From: Chen-Yu Tsai @ 2013-12-12 4:07 UTC (permalink / raw) 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 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr 2013-12-12 4:07 ` Chen-Yu Tsai @ 2013-12-12 5:13 ` Mohit KUMAR DCG 0 siblings, 0 replies; 7+ messages in thread From: Mohit KUMAR DCG @ 2013-12-12 5:13 UTC (permalink / raw) To: Chen-Yu Tsai, Rajeev KUMAR, Shiraz HASHIM Cc: Jingoo Han, Pratyush ANAND, devicetree@vger.kernel.org, spear-devel, Viresh Kumar, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Florian Fainelli 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-12-12 5:13 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1386752447.git.mohit.kumar@st.com>
2013-12-11 9:38 ` [PATCH 01/12] SPEAr13xx: Correct dt field name for stmmac phy-addr Mohit Kumar
2013-12-11 11:09 ` Rajeev kumar
2013-12-11 11:14 ` Rajeev kumar
2013-12-12 0:39 ` Jingoo Han
2013-12-12 3:59 ` Rajeev kumar
2013-12-12 4:07 ` Chen-Yu Tsai
2013-12-12 5:13 ` Mohit KUMAR DCG
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).