* [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE()
@ 2026-07-04 15:01 Pengpeng Hou
2026-07-06 7:49 ` Peng Fan
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Pengpeng Hou @ 2026-07-04 15:01 UTC (permalink / raw)
To: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
Cc: Pengpeng Hou, linux-clk, imx, linux-arm-kernel, linux-kernel
The driver has a match table for the of 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/clk/imx/clk-imx8qxp-lpcg.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/imx/clk-imx8qxp-lpcg.c b/drivers/clk/imx/clk-imx8qxp-lpcg.c
index 1dae3410ee99..f0cf6cf91899 100644
--- a/drivers/clk/imx/clk-imx8qxp-lpcg.c
+++ b/drivers/clk/imx/clk-imx8qxp-lpcg.c
@@ -354,6 +354,7 @@ static const struct of_device_id imx8qxp_lpcg_match[] = {
{ .compatible = "fsl,imx8qxp-lpcg", NULL },
{ /* sentinel */ }
};
+MODULE_DEVICE_TABLE(of, imx8qxp_lpcg_match);
static struct platform_driver imx8qxp_lpcg_clk_driver = {
.driver = {
--
2.53.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE()
2026-07-04 15:01 [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE() Pengpeng Hou
@ 2026-07-06 7:49 ` Peng Fan
2026-07-06 15:36 ` Brian Masney
2026-07-06 15:50 ` Frank Li
2 siblings, 0 replies; 4+ messages in thread
From: Peng Fan @ 2026-07-06 7:49 UTC (permalink / raw)
To: Pengpeng Hou
Cc: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux-clk,
imx, linux-arm-kernel, linux-kernel
On Sat, Jul 04, 2026 at 11:01:41PM +0800, Pengpeng Hou wrote:
>The driver has a match table for the of 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>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE()
2026-07-04 15:01 [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE() Pengpeng Hou
2026-07-06 7:49 ` Peng Fan
@ 2026-07-06 15:36 ` Brian Masney
2026-07-06 15:50 ` Frank Li
2 siblings, 0 replies; 4+ messages in thread
From: Brian Masney @ 2026-07-06 15:36 UTC (permalink / raw)
To: Pengpeng Hou
Cc: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux-clk,
imx, linux-arm-kernel, linux-kernel
On Sat, Jul 04, 2026 at 11:01:41PM +0800, Pengpeng Hou wrote:
> The driver has a match table for the of 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>
Reviewed-by: Brian Masney <bmasney@redhat.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE()
2026-07-04 15:01 [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE() Pengpeng Hou
2026-07-06 7:49 ` Peng Fan
2026-07-06 15:36 ` Brian Masney
@ 2026-07-06 15:50 ` Frank Li
2 siblings, 0 replies; 4+ messages in thread
From: Frank Li @ 2026-07-06 15:50 UTC (permalink / raw)
To: Pengpeng Hou
Cc: Abel Vesa, Peng Fan, Michael Turquette, Stephen Boyd, Frank Li,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam, linux-clk,
imx, linux-arm-kernel, linux-kernel
On Sat, Jul 04, 2026 at 11:01:41PM +0800, Pengpeng Hou wrote:
> The driver has a match table for the of 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>
> ---
Reviewed-by: Frank Li <Frank.Li@nxp.com>
> drivers/clk/imx/clk-imx8qxp-lpcg.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/imx/clk-imx8qxp-lpcg.c b/drivers/clk/imx/clk-imx8qxp-lpcg.c
> index 1dae3410ee99..f0cf6cf91899 100644
> --- a/drivers/clk/imx/clk-imx8qxp-lpcg.c
> +++ b/drivers/clk/imx/clk-imx8qxp-lpcg.c
> @@ -354,6 +354,7 @@ static const struct of_device_id imx8qxp_lpcg_match[] = {
> { .compatible = "fsl,imx8qxp-lpcg", NULL },
> { /* sentinel */ }
> };
> +MODULE_DEVICE_TABLE(of, imx8qxp_lpcg_match);
>
> static struct platform_driver imx8qxp_lpcg_clk_driver = {
> .driver = {
> --
> 2.53.0
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-07-06 15:50 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-04 15:01 [PATCH] clk: imx: imx8qxp-lpcg: add missing MODULE_DEVICE_TABLE() Pengpeng Hou
2026-07-06 7:49 ` Peng Fan
2026-07-06 15:36 ` Brian Masney
2026-07-06 15:50 ` Frank Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox