Re: [PATCH 1/2] dt-bindings: iio: adc: add Texas Instruments TLA 2528

[Matti Vaittinen]

On 29/12/2025 11:31, Andy Shevchenko wrote:
> On Mon, Dec 29, 2025 at 10:04 AM Matti Vaittinen
> <mazziesaccount@xxxxxxxxx> wrote:
> > On 23/12/2025 20:26, David Lechner wrote:
> > > On 12/23/25 9:55 AM, Maxime Chevallier wrote:
>
> ...
>
> > > It looks like inputs can also be used as GPIOs, so
> > >
> > > gpio-controller: true
> > > #gpio-cells:
> > >     const: 2
> > >
> > > would be appropriate (it doesn't matter if the driver doesn't
> > > implement it, we know what the correct bindings are).
> > >
> > > > +
> > > > +  "#io-channel-cells":
> > > > +    const: 1
> >
> > I didn't check the data-sheet, but if the pins can be set to be GPIOs or
> > ADC inputs, then I would require channels to be specified. It's only 8
> > channels, so always listing channels that are present shouldn't be that
> > big of a problem - and it should avoid one to add extra properties to
> > denote channels used for GPIO if GPIOs need to be supported.
> >
> > Well, I am not insisting this, there are folks that know this stuff
> > better than I :)
>
> Why would we need an extra property for that? GPIO controller has a
> property for valid_mask, should be enough to handle this case, no?
Ah. You're right. The "valid_mask" should be perfectly usable.

I might still require the channel information to make it explicit - but 
as I said, I leave this for others to decide :)

Yours,
	-- Matti


---
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~