Hello Shao-Hui,
On 07/24/2014 10:11 PM, Shaohui Xie wrote: > Hello, Scott, Shruti, > > There are two types MDIO controllers, each MAC has a corresponding > MDIO controller to access internal PHYs inside Serdes, another is > dedicated MDIO controller to access external PHYs (PHYs on board or > riser card, etc). Current MDIO driver does not distinguish the two > controllers, if we need to distinguish the controllers, what could be > the proper way? Perhaps a new property is needed in dts? As per the intended binding, FMan v3 MDIO must use the "fsl,fman-memac-mdio" compatible. Why is that not enough to differentiate the two controllers? Cheers, > Please comment! > Thanks! > > Best Regards, > Shaohui Xie >> -----Original Message----- >> From: Linuxppc-dev [mailto:linuxppc-dev- >> bounces+b21989=freescale....@lists.ozlabs.org] On Behalf Of Shruti >> Kanetkar >> Sent: Thursday, July 17, 2014 4:17 AM >> To: Wood Scott-B07421 >> Cc: Kanetkar Shruti-B44454; linuxppc-dev@lists.ozlabs.org >> Subject: [PATCH v2 1/7] dt: Introduce the FMan 10 Gb/s MDIO binding >> >> This document covers FManv2 and FManv3 mEMAC MDIO >> >> Signed-off-by: Shruti Kanetkar <shr...@freescale.com> >> --- >> .../devicetree/bindings/net/fsl-xgmac-phy.txt | 29 >> ++++++++++++++++++++++ >> 1 file changed, 29 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/net/fsl-xgmac- >> phy.txt >> >> diff --git a/Documentation/devicetree/bindings/net/fsl-xgmac-phy.txt >> b/Documentation/devicetree/bindings/net/fsl-xgmac-phy.txt >> new file mode 100644 >> index 0000000..2f2d56d >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/fsl-xgmac-phy.txt >> @@ -0,0 +1,29 @@ >> +*MDIO IO device >> + >> +The MDIO is a bus to which the PHY devices are connected. For each >> +device that exists on this bus, a child node should be created. >> + >> +Required properties: >> + - compatible: Currently supported strings/devices are: >> + - "fsl,fman-xmdio" (Must be used for 10 Gb/s MDIO from >> FMan v2. >> + Can be used for 1/10 Gb/s MDIO from FMan v3) >> + - "fsl,fman-memac-mdio" (Must be used for 1/10 Gb/s >> MDIO from >> + FMan v3) >> + - reg >> + >> + Example for FMan v2: >> + mdio@f1000 { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + compatible = "fsl,fman-xmdio"; >> + reg = <0xf1000 0x1000>; >> + }; >> + >> + Example for FMan v3: >> + >> + mdio@f1000 { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio"; >> + reg = <0xf1000 0x1000>; >> + } >> -- >> 1.8.3.1 >> >> _______________________________________________ >> Linuxppc-dev mailing list >> Linuxppc-dev@lists.ozlabs.org >> https://lists.ozlabs.org/listinfo/linuxppc-dev _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
