From: "Nuno Sá" <noname.nuno@gmail.com>
To: Linus Walleij <linus.walleij@linaro.org>,
Kim Seer Paller <kimseer.paller@analog.com>
Cc: Bartosz Golaszewski <brgl@bgdev.pl>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 2/2] gpio: gpio-adg1414: New driver
Date: Fri, 14 Feb 2025 13:17:38 +0000 [thread overview]
Message-ID: <880631da17a6d8ed4afe5a8c453fd4f7d0e4fca5.camel@gmail.com> (raw)
In-Reply-To: <CACRpkdZR8X17Bn-i2anqjxf0Gk60V175F7Xfwytkhy7_K+LsSA@mail.gmail.com>
Hi Linus,
On Fri, 2025-02-14 at 00:25 +0100, Linus Walleij wrote:
> Hi Kim,
>
> thanks for your patch!
>
> On Thu, Feb 13, 2025 at 2:17 PM Kim Seer Paller
> <kimseer.paller@analog.com> wrote:
>
> > The ADG1414 is a 9.5 Ω RON ±15 V/+12 V/±5 V iCMOS Serially-Controlled
> > Octal SPST Switches
> >
> > Signed-off-by: Kim Seer Paller <kimseer.paller@analog.com>
>
> OK so I looked at the data sheet and it looks like this:
>
> A o-------/ --------o B
>
> It'a a switch.
>
> Why is this switch a "gpio", other than that it is convenient
> to use the GPIO abstraction to control it?
>
> GPIO is usually devices that can drive a line high or low.
> This is very far from that. This could switch some analog
> line or whatever, right?
I would say so yes but Kim should know better...
>
> Now, the kernel does not have switch subsystem I think,
> so this is something like a special case, so we might be
> compelled to make an exception, if the users will all be in
Exactly, since we could not find anything, the best fit seemed like the gpio
subsystem. I was the one suggesting it since a new subsystem for a simple device
like this looked excessive. If we had more devices that would fit such a class
of devices, maybe it would make more sense to start thinking on such a
subsystem?
> say userspace and make use of this switch for factory lines
> or similar.
Kim should know better again (about usecases) but I would also assume this is
for userspace use.
Thanks!
- Nuno Sá
next prev parent reply other threads:[~2025-02-14 13:17 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-13 13:15 [PATCH v2 0/2] Add support for ADG1414 Serially-Controlled Octal SPST Switches Kim Seer Paller
2025-02-13 13:15 ` [PATCH v2 1/2] dt-bindings: gpio: add adg1414 Kim Seer Paller
2025-02-13 16:11 ` kernel test robot
2025-02-13 18:16 ` Krzysztof Kozlowski
2025-02-14 10:42 ` Linus Walleij
2025-02-13 13:15 ` [PATCH v2 2/2] gpio: gpio-adg1414: New driver Kim Seer Paller
2025-02-13 23:25 ` Linus Walleij
2025-02-14 13:17 ` Nuno Sá [this message]
2025-02-14 23:22 ` Linus Walleij
2025-02-16 14:30 ` Jonathan Cameron
2025-02-17 7:02 ` Paller, Kim Seer
2025-02-17 9:32 ` Nuno Sá
2025-02-27 0:33 ` Linus Walleij
2025-02-15 1:12 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=880631da17a6d8ed4afe5a8c453fd4f7d0e4fca5.camel@gmail.com \
--to=noname.nuno@gmail.com \
--cc=brgl@bgdev.pl \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=kimseer.paller@analog.com \
--cc=krzk+dt@kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.