Re: [PATCH 2/2] dt-bindings: phy: lan966x: Add optional microchip,sx-tx/rx-inverted

Conor Dooley
2025-11-10T18:43:41Z

[-- Attachment #1: Type: text/plain, Size: 1936 bytes --]

On Mon, Nov 10, 2025 at 12:05:36PM +0100, Horatiu Vultur wrote:
> This allows to invert the N and P signals of the RX and TX Serdes
> signals. This option allows the board designer to trace their signals
> easier on the boards.

Why can't this just be done in software, debugfs or something like that?
Maybe it's just your description is poor, but sounds like the intention
here is to just switch things around for debug purposes.

>
> Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
> ---
> .../phy/microchip,lan966x-serdes.yaml | 24 +++++++++++++++++++
> 1 file changed, 24 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/phy/microchip,lan966x-serdes.yaml b/Documentation/devicetree/bindings/phy/microchip,lan966x-serdes.yaml
> index 6e914fbbac567..21b19e06a75aa 100644
> --- a/Documentation/devicetree/bindings/phy/microchip,lan966x-serdes.yaml
> +++ b/Documentation/devicetree/bindings/phy/microchip,lan966x-serdes.yaml
> @@ -41,6 +41,30 @@ properties:
> - The macro to be used. The macros are defined in
> dt-bindings/phy/phy-lan966x-serdes.
>
> + microchip,s0-tx-inverted:
> + type: boolean
> + description: Invert the TX N and P signals for Serdes 0
> +
> + microchip,s1-tx-inverted:
> + type: boolean
> + description: Invert the TX N and P signals for Serdes 1
> +
> + microchip,s2-tx-inverted:
> + type: boolean
> + description: Invert the TX N and P signals for Serdes 2
> +
> + microchip,s0-rx-inverted:
> + type: boolean
> + description: Invert the RX N and P signals for Serdes 0
> +
> + microchip,s1-rx-inverted:
> + type: boolean
> + description: Invert the RX N and P signals for Serdes 1
> +
> + microchip,s2-rx-inverted:
> + type: boolean
> + description: Invert the RX N and P signals for Serdes 2
> +
> required:
> - compatible
> - reg
> --
> 2.34.1
>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]