linux-kernel - Re: [PATCH v3 0/2] Add ADG1712 SPST switch controller support

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: <65d4096a193abb7e5c103936606172005af99860.camel@gmail.com>
Date: Mon, 17 Nov 2025 15:06:04 +0000
From: Nuno Sá <noname.nuno@...il.com>
To: Antoniu Miclaus <antoniu.miclaus@...log.com>, Rob Herring
 <robh@...nel.org>,  Krzysztof Kozlowski	 <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Linus Walleij	 <linus.walleij@...aro.org>, Bartosz
 Golaszewski <brgl@...ev.pl>, 	devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, 	linux-gpio@...r.kernel.org
Subject: Re: [PATCH v3 0/2] Add ADG1712 SPST switch controller support

On Mon, 2025-11-17 at 09:13 +0000, Antoniu Miclaus wrote:
> This series adds support for the Analog Devices ADG1712 quad single-pole,
> single-throw (SPST) switch controller.
> 
> The ADG1712 contains four independent analog switches, each controlled by
> a dedicated GPIO input pin. This implementation configures the switches
> once at probe time based on device tree properties.
> 
> Changes in v3:
> - Moved device tree bindings from gpio/ to switch/ subsystem
> - Completely redesigned driver architecture: removed GPIO controller interface
> - Added 'switch-states' device tree property for configuring initial switch states

Hmm but if we are going that direction, I wonder if this should be in drivers/gpio at
all?

We have drivers/mux/gpio.c. While this is a switch (and not a mux) I guess it would still
be more appropriate in there and probably works out of the box?

- Nuno Sá