Subject: Re: [PATCH v2 1/7] dt: Introduce the FMan 10 Gb/s MDIO binding

From: Emil Medve <[email-masked]>
Date: Fri, 25 Jul 2014 15:00:55 -0500

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=[email-masked]] On Behalf Of Shruti
>> Kanetkar
>> Sent: Thursday, July 17, 2014 4:17 AM
>> To: Wood Scott-B07421
>> Cc: Kanetkar Shruti-B44454; [email-masked]
>> 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 <[email-masked]>
>> ---
>>  .../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
>> [email-masked]
>> https://lists.ozlabs.org/listinfo/linuxppc-dev