Subject: Re: [PATCH net-next v4 12/13] Documentation: devicetree: update dsa binding with optional properties

From: Sergei Shtylyov <[email-masked]>
Date: Wed, 27 Aug 2014 02:40:34 +0400

Hello.

On 08/26/2014 08:49 AM, Florian Fainelli wrote:

> Add documentation for a bunch of new optional properties described in
> ethernet.txt and fixed-link.txt, this includes: 'phy-handle', 'phy-mode'
> and the 'fixed-link' subnode.

> Signed-off-by: Florian Fainelli <[email-masked]>
> ---
>   Documentation/devicetree/bindings/net/dsa/dsa.txt | 17 +++++++++++++++++
>   1 file changed, 17 insertions(+)

> diff --git a/Documentation/devicetree/bindings/net/dsa/dsa.txt b/Documentation/devicetree/bindings/net/dsa/dsa.txt
> index 49f4f7ae3f51..6192f9a7aa89 100644
> --- a/Documentation/devicetree/bindings/net/dsa/dsa.txt
> +++ b/Documentation/devicetree/bindings/net/dsa/dsa.txt
> @@ -39,6 +39,22 @@ Optionnal property:
>   			  This property is only used when switches are being
>   			  chained/cascaded together.
>
> +- phy-handle		: Phandle to a PHY on an external MDIO bus, not the
> +			  switch internal one. See
> +			  Documentation/devicetree/bindings/net/ethernet.txt
> +			  for details.
> +
> +- phy-mode		: String representing the connection to the designeted

    Designated?

> +			  PHY node specified by the 'phy-handle' property. See
> +			  Documentation/devicetree/bindings/net/ethernet.txt
> +			  for details.
> +
> +Optionnal subnodes:

    One 'n' is enough (although DaveM may disagree ;-).

WBR, Sergei