On Wed, Apr 03, 2024 at 11:52:12AM +0200, Krzysztof Kozlowski wrote: > On 03/04/2024 11:40, Andy Shevchenko wrote: > > On Wed, Apr 03, 2024 at 10:06:25AM +0200, Arnd Bergmann wrote: ... > >> -static const struct of_device_id stmpe_ts_ids[] = { > >> +static const struct of_device_id stmpe_ts_ids[] __maybe_unused = { > > > > __maybe_unused? > > > > Why not adding it into .driver as you have done in another patch in this series? > > Because there is no benefit in this. This is instantiated by MFD, so the > only thing you need is entry for module loading. Hmm... Seems to me rather a good candidate for MODULE_ALIAS in this case. No? -- With Best Regards, Andy Shevchenko