netdev - Re: [PATCH net-next 1/3] dt-bindings: net: ethernet-phy: add property mac-series-termination-ohms

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250311173344.GA3802548-robh@kernel.org>
Date: Tue, 11 Mar 2025 12:33:44 -0500
From: Rob Herring <robh@...nel.org>
To: Dimitri Fedrau <dimitri.fedrau@...bherr.com>
Cc: Andrew Lunn <andrew@...n.ch>, Heiner Kallweit <hkallweit1@...il.com>,
	Russell King <linux@...linux.org.uk>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Florian Fainelli <f.fainelli@...il.com>, netdev@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	Dimitri Fedrau <dima.fedrau@...il.com>
Subject: Re: [PATCH net-next 1/3] dt-bindings: net: ethernet-phy: add
 property mac-series-termination-ohms

On Fri, Mar 07, 2025 at 11:30:01AM +0100, Dimitri Fedrau wrote:
> Add property mac-series-termination-ohms in the device tree bindings for
> selecting the resistance value of the builtin series termination resistors
> of the PHY. Changing the resistance to an appropriate value can reduce
> signal reflections and therefore improve signal quality.
> 
> Signed-off-by: Dimitri Fedrau <dimitri.fedrau@...bherr.com>
> ---
>  Documentation/devicetree/bindings/net/ethernet-phy.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> index 824bbe4333b7ed95cc39737d3c334a20aa890f01..4a710315a83ccf15bfc210ae432ae988cf31e04c 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> @@ -238,6 +238,11 @@ properties:
>        peak-to-peak specified in ANSI X3.263. When omitted, the PHYs default
>        will be left as is.
>  
> +  mac-series-termination-ohms:

A property of the MAC (or associated with it) should be in the MAC's 
node. Also, sounds like either either end could have a property. We 
already have similar properties in other cases: 'termination-ohms' for 
example. That appears to be for series term as well given a value of 
120ohms. 

Rob