Hi,

On Thu, Dec 12, 2013 at 11:59 AM, Rajeev kumar [off-list ref] wrote:

On 12/12/2013 6:09 AM, Jingoo Han wrote:

quoted

On Wednesday, December 11, 2013 8:09 PM, Rajeev kumar wrote:

quoted

On 12/11/2013 3:08 PM, Mohit KUMAR wrote:

quoted

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<redacted>
Cc: Pratyush Anand<redacted>
Cc: Viresh Kumar<redacted>
Cc: spear-devel at list.st.com
Cc: linux-arm-kernel at lists.infradead.org
Cc: devicetree at 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 at 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"'.

quoted

./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