[PATCH] iio: adc: max1241: add missing MODULE_DEVICE_TABLE()

[PATCH] iio: adc: max1241: add missing MODULE_DEVICE_TABLE()

[Pengpeng Hou]

The driver has a match table for the spi bus wired into its driver
structure, but the table is not exported with MODULE_DEVICE_TABLE().

Add the missing MODULE_DEVICE_TABLE() entry so module alias information
is generated for automatic module loading.

This is a source-level fix.  It does not claim dynamic hardware
reproduction; the evidence is the driver-owned match table, its use by
the driver registration structure, and the missing module alias
publication.

Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
---
 drivers/iio/adc/max1241.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/iio/adc/max1241.c b/drivers/iio/adc/max1241.c
index d62c1a011659..cab04beae729 100644
--- a/drivers/iio/adc/max1241.c
+++ b/drivers/iio/adc/max1241.c
@@ -179,6 +179,7 @@ static const struct spi_device_id max1241_id[] = {
 	{ "max1241", max1241 },
 	{ }
 };
+MODULE_DEVICE_TABLE(spi, max1241_id);
 
 static const struct of_device_id max1241_dt_ids[] = {
 	{ .compatible = "maxim,max1241" },
-- 
2.53.0

Re: [PATCH] iio: adc: max1241: add missing MODULE_DEVICE_TABLE()

[Andy Shevchenko]

On Sat, Jul 04, 2026 at 11:16:33PM +0800, Pengpeng Hou wrote:
> The driver has a match table for the spi bus wired into its driver
> structure, but the table is not exported with MODULE_DEVICE_TABLE().
> 
> Add the missing MODULE_DEVICE_TABLE() entry so module alias information
> is generated for automatic module loading.
> 
> This is a source-level fix.  It does not claim dynamic hardware
> reproduction; the evidence is the driver-owned match table, its use by
> the driver registration structure, and the missing module alias
> publication.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] iio: adc: max1241: add missing MODULE_DEVICE_TABLE()

[Jonathan Cameron]

On Sat, 4 Jul 2026 19:57:37 +0300
Andy Shevchenko <andriy.shevchenko@intel.com> wrote:

> On Sat, Jul 04, 2026 at 11:16:33PM +0800, Pengpeng Hou wrote:
> > The driver has a match table for the spi bus wired into its driver
> > structure, but the table is not exported with MODULE_DEVICE_TABLE().
> > 
> > Add the missing MODULE_DEVICE_TABLE() entry so module alias information
> > is generated for automatic module loading.
> > 
> > This is a source-level fix.  It does not claim dynamic hardware
> > reproduction; the evidence is the driver-owned match table, its use by
> > the driver registration structure, and the missing module alias
> > publication.  
> 
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>

I'm not rushing this in given we don't have reports of it actually
causing problems for any users. Applied to the testing branch of iio.git
which means it'll go upstream in the next merge window

Thanks,

Jonathan

>