| Message ID | 1322809549-32622-2-git-send-email-afleming@freescale.com |
|---|---|
| State | Rejected, archived |
| Delegated to: | David Miller |
| Headers | show |
On Dec 2, 2011, at 1:05 AM, Andy Fleming wrote: > The MDIO driver has been changed so that it no longer supports > scanning the MDIO bus for a free address for the TBI PHY. This > feature was fragile, and required scanning the bus before the bus > was fully up and registered. The intended way for FSL devices to > specify the TBI PHY's address is via a tbi node in the device tree. > All of the device trees had such a node, except for the recent p1/p2 > trees. Rather than hand-fixing all of those boards, set a default > value (most boards were using 0x11, anyway), and any board which > wants to change it from that default can then override it in its > board dts file. > > This fixes an issue where p1/p2 boards would fail to bring up > Ethernet, due to not finding a tbi node. Is this only needed on 1st controller because its what has external PHY control? > Signed-off-by: Andy Fleming <afleming@freescale.com> > --- > arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi | 5 +++++ > arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi | 5 +++++ > 2 files changed, 10 insertions(+), 0 deletions(-) This doesn't seem correct, meaning this should really be in the board .dts not in the IP. I think the driver should check and warn if this property doesn't exist. > > diff --git a/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi b/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi > index a1979ae..0a42e21 100644 > --- a/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi > +++ b/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi > @@ -50,4 +50,9 @@ mdio@24520 { > #size-cells = <0>; > compatible = "fsl,gianfar-mdio"; > reg = <0x24520 0x20>; > + > + tbi-phy@11 { > + device-type = "tbi-phy"; > + reg = <0x11>; > + }; > }; > diff --git a/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi b/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi > index 1382fec..964670a 100644 > --- a/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi > +++ b/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi > @@ -38,6 +38,11 @@ mdio@24000 { > #size-cells = <0>; > compatible = "fsl,etsec2-mdio"; > reg = <0x24000 0x1000 0xb0030 0x4>; > + > + tbi-phy@11 { > + device-type = "tbi-phy"; > + reg = <0x11>; > + }; > }; > > ethernet@b0000 { > -- > 1.7.3.4 > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
>> arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi | 5 +++++ >> arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi | 5 +++++ >> 2 files changed, 10 insertions(+), 0 deletions(-) > > This doesn't seem correct, meaning this should really be in the board .dts not in the IP. > > I think the driver should check and warn if this property doesn't exist. Hmm... in principle, I agree with you. I suppose I was just being lazy, and avoiding changing all the device trees again. However, it's not an unreasonable default. A search through *all* of the device trees for the tbi node indicates that 0x11 is chosen about 50% of the time. However, I'll devise a patch which complains if the node isn't found, and another patch which modifies all of the offending device trees. Andy -- To unsubscribe from this list: send the line "unsubscribe netdev" 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/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi b/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi index a1979ae..0a42e21 100644 --- a/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi +++ b/arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi @@ -50,4 +50,9 @@ mdio@24520 { #size-cells = <0>; compatible = "fsl,gianfar-mdio"; reg = <0x24520 0x20>; + + tbi-phy@11 { + device-type = "tbi-phy"; + reg = <0x11>; + }; }; diff --git a/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi b/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi index 1382fec..964670a 100644 --- a/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi +++ b/arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi @@ -38,6 +38,11 @@ mdio@24000 { #size-cells = <0>; compatible = "fsl,etsec2-mdio"; reg = <0x24000 0x1000 0xb0030 0x4>; + + tbi-phy@11 { + device-type = "tbi-phy"; + reg = <0x11>; + }; }; ethernet@b0000 {
The MDIO driver has been changed so that it no longer supports scanning the MDIO bus for a free address for the TBI PHY. This feature was fragile, and required scanning the bus before the bus was fully up and registered. The intended way for FSL devices to specify the TBI PHY's address is via a tbi node in the device tree. All of the device trees had such a node, except for the recent p1/p2 trees. Rather than hand-fixing all of those boards, set a default value (most boards were using 0x11, anyway), and any board which wants to change it from that default can then override it in its board dts file. This fixes an issue where p1/p2 boards would fail to bring up Ethernet, due to not finding a tbi node. Signed-off-by: Andy Fleming <afleming@freescale.com> --- arch/powerpc/boot/dts/fsl/pq3-etsec1-0.dtsi | 5 +++++ arch/powerpc/boot/dts/fsl/pq3-etsec2-0.dtsi | 5 +++++ 2 files changed, 10 insertions(+), 0 deletions(-)