linux-kernel - Re: [PATCH 2/2] gpio: adg1712: add driver 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: <3ead5d7aa5e6be2b6df3bb91b35fec37e23353f3.camel@gmail.com>
Date: Mon, 10 Nov 2025 12:32:42 +0000
From: Nuno Sá <noname.nuno@...il.com>
To: Linus Walleij <linus.walleij@...aro.org>, Antoniu Miclaus
	 <antoniu.miclaus@...log.com>
Cc: Bartosz Golaszewski <brgl@...ev.pl>, Rob Herring <robh@...nel.org>, 
 Krzysztof Kozlowski
	 <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	linux-gpio@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] gpio: adg1712: add driver support

On Mon, 2025-11-10 at 11:30 +0100, Linus Walleij wrote:
> Hi Antoniu,
> 
> thanks for your patch!
> 
> On Fri, Oct 31, 2025 at 5:08 PM Antoniu Miclaus
> <antoniu.miclaus@...log.com> wrote:
> 
> > Add driver support for the ADG1712, which contains four independent
> > single-pole/single-throw (SPST) switches and operates with a
> > low-voltage single supply range from +1.08V to +5.5V or a low-voltage
> > dual supply range from ±1.08V to ±2.75V.
> > 
> > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@...log.com>
> 
> So tying into the binding discussion:
> 
> GPIO means "general purpose input/output".
> 
> I am really confused as whether this is:
> 
> - General purpose - seems to be for the purpose of switching
>   currents and nothing else.
> 
> - Input/Output - It's switching something else and not inputting
>   or outputting anything and this makes the driver look strange.
> 
> 

Not the first time a part like this pops up [1]. At the time, the final
conclusion was to go with gpiolib. Naturally you can think otherwise now :)

Also, it looks like that series did not went anywhere (I'll probably ping the author internally)


[1]: https://lore.kernel.org/linux-gpio/20250213-for_upstream-v2-2-ec4eff3b3cd5@analog.com/

- Nuno Sá