[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aQxVNPRVdJvbqkiz@black.igk.intel.com>
Date: Thu, 6 Nov 2025 08:58:44 +0100
From: Raag Jadav <raag.jadav@...el.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: kees@...nel.org, broonie@...nel.org, gregkh@...uxfoundation.org,
arnd@...db.de, hansg@...nel.org, ilpo.jarvinen@...ux.intel.com,
linus.walleij@...aro.org, brgl@...ev.pl,
platform-driver-x86@...r.kernel.org, linux-gpio@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] mod_devicetable: Bump auxiliary_device_id name size
On Thu, Nov 06, 2025 at 09:40:46AM +0200, Andy Shevchenko wrote:
> On Thu, Nov 06, 2025 at 10:58:38AM +0530, Raag Jadav wrote:
> > We have an upcoming driver named "intel_ehl_pse_io". This creates an
> > auxiliary child device for it's GPIO sub-functionality, which matches
> > against "intel_ehl_pse_io.gpio-elkhartlake" and overshoots the current
>
> Looking at the name there is another question: Why do we need 'elkhartlake in
> the GPIO driver's name now? It's a dup to 'ehl' in the first part.
Just kept it for historic consistency and I'm a bit terrible at naming.
Perhaps "gpio-aux"? But that's too generic from subsystem POV.
Open to suggestions.
Raag
> > maximum limit of 32 bytes for auxiliary device id string. Bump the size
> > to 40 bytes to satisfy such cases.
>
> > ---
> > v2: Describe the use case (Greg, Andy)
>
> Thanks for the update!
Powered by blists - more mailing lists