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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20251110-unwound-award-a11d69b9da4f@spud>
Date: Mon, 10 Nov 2025 18:43:37 +0000
From: Conor Dooley <conor@...nel.org>
To: Horatiu Vultur <horatiu.vultur@...rochip.com>
Cc: vkoul@...nel.org, kishon@...nel.org, robh@...nel.org,
	krzk+dt@...nel.org, conor+dt@...nel.org,
	linux-phy@...ts.infradead.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] dt-bindings: phy: lan966x: Add optional
 microchip,sx-tx/rx-inverted

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@...rochip.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
> 

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)