On Mon, Dec 29, 2025 at 10:04 AM Matti Vaittinen
[off-list ref] wrote:

On 23/12/2025 20:26, David Lechner wrote:

quoted

On 12/23/25 9:55 AM, Maxime Chevallier wrote:

...

quoted

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).

quoted

+
+  "#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?

-- 
With Best Regards,
Andy Shevchenko