Re: [PATCH v4 2/7] Documentation: devicetree: add PHY lane swap binding

From: Scott Branden
Date: Tue Nov 01 2016 - 17:01:49 EST




On 16-11-01 01:59 PM, Jon Mason wrote:
On Tue, Nov 01, 2016 at 09:48:26PM +0100, Andrew Lunn wrote:
+- enet-phy-lane-swap: If set, indicates the PHY device requires swapping the
+ TX/RX lanes to function properly.

Is 'requires' the right word here? The PHY performs the actual swap of
the Tx/Rx lanes.

What I'm trying to say here is that without the lane swapping, the PHY
will not function properly. Thus, those PHYs require this parameter
to work. I am open to suggestions to reword the sentence to make it
clearer.
If set, indicates the PHY will swap the TX/RX lanes to compensate for the board being designed with the lanes swapped.

Thanks,
Jon


Andrew