netdev - Re: [PATCH net-next 0/2] net: dsa: microchip: Add strap configuration during reset

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: <393f0c41-57f2-491a-9ac6-5069c7db089e@lunn.ch>
Date: Wed, 10 Sep 2025 18:27:54 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Bastien Curutchet <bastien.curutchet@...tlin.com>
Cc: Woojung Huh <woojung.huh@...rochip.com>, UNGLinuxDriver@...rochip.com,
	Vladimir Oltean <olteanv@...il.com>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, Marek Vasut <marex@...x.de>,
	Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
	Miquèl Raynal <miquel.raynal@...tlin.com>,
	Pascal Eberhard <pascal.eberhard@...com>, netdev@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net-next 0/2] net: dsa: microchip: Add strap
 configuration during reset

On Wed, Sep 10, 2025 at 04:55:23PM +0200, Bastien Curutchet wrote:
> Hi all,
> 
> This small series aims to allow to configure the KSZ8463 switch at
> reset. This configuration is determined by pin states while the chip is
> held in reset. Normally, this kind of configuration is handled with
> pull-ups/pull-downs. However, in some designs these pull-ups/pull-downs
> can be missing (either intentionally to save power or simply by mistake).
> In such cases, we need to manually drive the configuration pins during
> reset to ensure the switch is set up correctly.
> 
> PATCH 0 adds a new property to the bindings that describes the GPIOs to
> be set during reset in order to configure the switch properly. Alongside
> this new property, a new 'reset' pinctrl state is introduced.
> 
> PATCH 1 implements the use of this property in the driver. I only have a
> KSZ8463 to test with, so only its configuration is supported.

Are you setting switch state which cannot be set via register writes?

	Andrew