* [PATCH v2 0/1] iio: Remove MODULE_ALIAS from men_z188_adc
@ 2026-01-08 13:53 Jose Javier Rodriguez Barbarin
2026-01-08 13:53 ` [PATCH v2 1/1] iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS Jose Javier Rodriguez Barbarin
0 siblings, 1 reply; 3+ messages in thread
From: Jose Javier Rodriguez Barbarin @ 2026-01-08 13:53 UTC (permalink / raw)
To: jic23, dlechner, nuno.sa
Cc: andriy.shevchenko, dev-jorge.sanjuangarcia, linux-iio,
linux-kernel, Jose Javier Rodriguez Barbarin
The first patches I sent for fixing the autoload problem encountered
on mcb device drivers were 2. The first [1] was focused on updating the
modpost process for letting kbuild to access to the data within
MODULE_DEVICE_TABLE and the second one [2] for removing the MODULE_ALIAS
on all mcb client drivers.
They were rejected and Andy suggested me to split the second patch
in a per-driver basis instead of sending all drivers' changes in a
single patch once the first patch was merged.
The first patch is already merged on Linus's Git repository for 6.19-rc4
commit 1f4ea4838b13 ("mcb: Add missing modpost build support")
So now I am sending this patch for removing MODULE_ALIAS on the mcb driver
in this subsystem as it is no longer required. This cleanup is being sent
to each affected subsystem separately, as per the review suggestion to ease
the handling for maintainers.
[1] https://lore.kernel.org/all/20251127155452.42660-2-dev-josejavier.rodriguez@duagon.com/
[2] https://lore.kernel.org/all/20251127155452.42660-3-dev-josejavier.rodriguez@duagon.com/
changes in v2:
- Renane commit message
Jose Javier Rodriguez Barbarin (1):
iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS
drivers/iio/adc/men_z188_adc.c | 1 -
1 file changed, 1 deletion(-)
--
2.52.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2 1/1] iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS
2026-01-08 13:53 [PATCH v2 0/1] iio: Remove MODULE_ALIAS from men_z188_adc Jose Javier Rodriguez Barbarin
@ 2026-01-08 13:53 ` Jose Javier Rodriguez Barbarin
2026-01-11 12:21 ` Jonathan Cameron
0 siblings, 1 reply; 3+ messages in thread
From: Jose Javier Rodriguez Barbarin @ 2026-01-08 13:53 UTC (permalink / raw)
To: jic23, dlechner, nuno.sa
Cc: andriy.shevchenko, dev-jorge.sanjuangarcia, linux-iio,
linux-kernel, Jose Javier Rodriguez Barbarin
Since commit 1f4ea4838b13 ("mcb: Add missing modpost build support")
the MODULE_ALIAS() is redundant as the module alias is now
automatically generated from the MODULE_DEVICE_TABLE().
Remove the explicit alias.
No functional change intended.
Reviewed-by: Jorge Sanjuan Garcia <dev-jorge.sanjuangarcia@duagon.com>
Signed-off-by: Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@duagon.com>
---
drivers/iio/adc/men_z188_adc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/iio/adc/men_z188_adc.c b/drivers/iio/adc/men_z188_adc.c
index cf8a8c0412ec..90919d282e7b 100644
--- a/drivers/iio/adc/men_z188_adc.c
+++ b/drivers/iio/adc/men_z188_adc.c
@@ -171,5 +171,4 @@ module_mcb_driver(men_z188_driver);
MODULE_AUTHOR("Johannes Thumshirn <johannes.thumshirn@men.de>");
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("IIO ADC driver for MEN 16z188 ADC Core");
-MODULE_ALIAS("mcb:16z188");
MODULE_IMPORT_NS("MCB");
--
2.52.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2 1/1] iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS
2026-01-08 13:53 ` [PATCH v2 1/1] iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS Jose Javier Rodriguez Barbarin
@ 2026-01-11 12:21 ` Jonathan Cameron
0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Cameron @ 2026-01-11 12:21 UTC (permalink / raw)
To: Jose Javier Rodriguez Barbarin
Cc: dlechner, nuno.sa, andriy.shevchenko, dev-jorge.sanjuangarcia,
linux-iio, linux-kernel
On Thu, 8 Jan 2026 14:53:01 +0100
Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@duagon.com> wrote:
> Since commit 1f4ea4838b13 ("mcb: Add missing modpost build support")
> the MODULE_ALIAS() is redundant as the module alias is now
> automatically generated from the MODULE_DEVICE_TABLE().
>
> Remove the explicit alias.
>
> No functional change intended.
>
> Reviewed-by: Jorge Sanjuan Garcia <dev-jorge.sanjuangarcia@duagon.com>
> Signed-off-by: Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@duagon.com>
Applied. Thanks,
Jonathan
> ---
> drivers/iio/adc/men_z188_adc.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/iio/adc/men_z188_adc.c b/drivers/iio/adc/men_z188_adc.c
> index cf8a8c0412ec..90919d282e7b 100644
> --- a/drivers/iio/adc/men_z188_adc.c
> +++ b/drivers/iio/adc/men_z188_adc.c
> @@ -171,5 +171,4 @@ module_mcb_driver(men_z188_driver);
> MODULE_AUTHOR("Johannes Thumshirn <johannes.thumshirn@men.de>");
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("IIO ADC driver for MEN 16z188 ADC Core");
> -MODULE_ALIAS("mcb:16z188");
> MODULE_IMPORT_NS("MCB");
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-01-11 12:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-01-08 13:53 [PATCH v2 0/1] iio: Remove MODULE_ALIAS from men_z188_adc Jose Javier Rodriguez Barbarin
2026-01-08 13:53 ` [PATCH v2 1/1] iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS Jose Javier Rodriguez Barbarin
2026-01-11 12:21 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®