Re: [PATCH v5 1/4] dt-bindings: net: ethernet-phy: add property enet-phy-lane-order

[Maxime Chevallier]

Hi,

On 02/02/2026 21:04, Damien Dejean wrote:
> Add property enet-phy-lane-order to the device tree bindings to define
> the lane order of the PHY. To simplify PCB design some manufacturers
> allow to wire the pairs in a reverse order, and change the order in
> software.
> 
> The property can be set to 0 to force the normal lane order (ABCD), or 1
> to force the reverse lane order (DCBA).
> 
> Signed-off-by: Damien Dejean <dam.dejean@xxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/net/ethernet-phy.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> index 58634fee9fc4..8347d4e134d2 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> @@ -126,6 +126,12 @@ properties:
>        e.g. wrong bootstrap configuration caused by issues in PCB
>        layout design.
>  
> +  enet-phy-lane-order:

As this is specific to BaseT, maybe rename that "enet-phy-pair-order" ?

> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    enum: [0, 1]
> +    description:
> +      For normal (0) or reverse (1) order of the pairs (ABCD -> DCBA).

We now have a binding in DT for ethernet-connectors, and I'd says this
is typically the kind of info I'd put in there :

https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/tree/Documentation/devicetree/bindings/net/ethernet-connector.yaml

The way you'd use that is by adding a ".attach_mdi_port()" callback in your
phy driver, and in there you'd parse these properties and do the proper settings.

We're missing a few things, such as storing a ref to the of_node corresponding
to the connector into phy_port.

This is brand new though, maybe Maintainers would like for phy_port to stabilize
before building more on top of it ?

Maxime