On Mon, Aug 16, 2021 at 03:25:13PM +0200, Rafael J. Wysocki wrote:

On Fri, Aug 6, 2021 at 4:44 PM Andy Shevchenko
[off-list ref] wrote:

quoted

By default the 512 GPIOs is a maximum on any x86 platform.
With, for example, Intel Tiger Lake-H the SoC based controller
occupies up to 480 pins. This leaves only 32 available for
GPIO expanders or other drivers, like PMIC. Hence, bump the
maximum GPIO number to 1024 for X86_64 and leave 512 for X86_32.

Thanks for review, my answers below.

quoted

+# The GPIO number here must be sorted by descending number. In case of
+# a multiplatform kernel, we just want the highest value required by the
+# selected platforms.
+config ARCH_NR_GPIO
+       int
+       default 1024 if X86_64
+       default 512 if X86_32
+       default 0

Wouldn't

default 1024 if X86_64
default 512

be sufficient?

It's either X86_64 or X86_32 anyway AFAICS.

I guess so.

quoted

+       help
+         Maximum number of GPIOs in the system.
+
+         If unsure, leave the default value.

Btw, what do you think. do we need comment above and help text here? I copied
these from ARM, but I'm not sure it would be useful on x86 as much.


-- 
With Best Regards,
Andy Shevchenko