* [PATCH 0/3] dmaengine: convert to subsys_platform_driver() and subsys_platform_driver_probe()
@ 2026-11-07 0:06 ` Brian Masney
0 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in these drivers and convert them over to use
the macros subsys_platform_driver() and subsys_platform_driver_probe().
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
Brian Masney (3):
dmaengine: convert to subsys_platform_driver()
dmaengine: convert to subsys_platform_driver_probe()
dmaengine: fsldma: convert to subsys_platform_driver()
drivers/dma/at_xdmac.c | 12 +-----------
drivers/dma/bestcomm/bestcomm.c | 18 +-----------------
drivers/dma/dma-jz4780.c | 12 +-----------
drivers/dma/dw/platform.c | 12 +-----------
drivers/dma/fsl-edma-main.c | 12 +-----------
drivers/dma/fsldma.c | 14 +-------------
drivers/dma/imx-dma.c | 6 +-----
drivers/dma/mcf-edma-main.c | 12 +-----------
drivers/dma/moxart-dma.c | 12 +-----------
drivers/dma/owl-dma.c | 12 +-----------
drivers/dma/sa11x0-dma.c | 12 +-----------
drivers/dma/sf-pdma/sf-pdma.c | 14 +-------------
drivers/dma/ste_dma40.c | 6 +-----
drivers/dma/ti/omap-dma.c | 12 +-----------
14 files changed, 14 insertions(+), 152 deletions(-)
---
base-commit: ddfe51b0a0f5dc9e415cc2c8ad8db17532dd426b
base-tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
change-id: 20261105-dmaengine-subsys-1dc716130f14
Best regards,
--
Brian Masney <bmasney@redhat.com>
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 1/3] dmaengine: convert to subsys_platform_driver()
2026-11-07 0:06 ` Brian Masney
@ 2026-11-07 0:06 ` Brian Masney
-1 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in these drivers and convert them over to use
the macro subsys_platform_driver().
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
drivers/dma/at_xdmac.c | 12 +-----------
drivers/dma/bestcomm/bestcomm.c | 18 +-----------------
drivers/dma/dma-jz4780.c | 12 +-----------
drivers/dma/dw/platform.c | 12 +-----------
drivers/dma/fsl-edma-main.c | 12 +-----------
drivers/dma/mcf-edma-main.c | 12 +-----------
drivers/dma/moxart-dma.c | 12 +-----------
drivers/dma/owl-dma.c | 12 +-----------
drivers/dma/sa11x0-dma.c | 12 +-----------
drivers/dma/sf-pdma/sf-pdma.c | 14 +-------------
drivers/dma/ti/omap-dma.c | 12 +-----------
11 files changed, 11 insertions(+), 129 deletions(-)
diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
index d40a8b40dddc..8b19d46ac812 100644
--- a/drivers/dma/at_xdmac.c
+++ b/drivers/dma/at_xdmac.c
@@ -2488,17 +2488,7 @@ static struct platform_driver at_xdmac_driver = {
}
};
-static int __init at_xdmac_init(void)
-{
- return platform_driver_register(&at_xdmac_driver);
-}
-subsys_initcall(at_xdmac_init);
-
-static void __exit at_xdmac_exit(void)
-{
- platform_driver_unregister(&at_xdmac_driver);
-}
-module_exit(at_xdmac_exit);
+subsys_platform_driver(at_xdmac_driver);
MODULE_DESCRIPTION("Atmel Extended DMA Controller driver");
MODULE_AUTHOR("Ludovic Desroches <ludovic.desroches@atmel.com>");
diff --git a/drivers/dma/bestcomm/bestcomm.c b/drivers/dma/bestcomm/bestcomm.c
index 241a91936fe8..1dadcc259382 100644
--- a/drivers/dma/bestcomm/bestcomm.c
+++ b/drivers/dma/bestcomm/bestcomm.c
@@ -474,23 +474,7 @@ static struct platform_driver mpc52xx_bcom_of_platform_driver = {
/* Module */
/* ======================================================================== */
-static int __init
-mpc52xx_bcom_init(void)
-{
- return platform_driver_register(&mpc52xx_bcom_of_platform_driver);
-}
-
-static void __exit
-mpc52xx_bcom_exit(void)
-{
- platform_driver_unregister(&mpc52xx_bcom_of_platform_driver);
-}
-
-/* If we're not a module, we must make sure everything is setup before */
-/* anyone tries to use us ... that's why we use subsys_initcall instead */
-/* of module_init. */
-subsys_initcall(mpc52xx_bcom_init);
-module_exit(mpc52xx_bcom_exit);
+subsys_platform_driver(mpc52xx_bcom_of_platform_driver);
MODULE_DESCRIPTION("Freescale MPC52xx BestComm DMA");
MODULE_AUTHOR("Sylvain Munaut <tnt@246tNt.com>");
diff --git a/drivers/dma/dma-jz4780.c b/drivers/dma/dma-jz4780.c
index ac2094e65ecf..32a40c17e0cd 100644
--- a/drivers/dma/dma-jz4780.c
+++ b/drivers/dma/dma-jz4780.c
@@ -1129,17 +1129,7 @@ static struct platform_driver jz4780_dma_driver = {
},
};
-static int __init jz4780_dma_init(void)
-{
- return platform_driver_register(&jz4780_dma_driver);
-}
-subsys_initcall(jz4780_dma_init);
-
-static void __exit jz4780_dma_exit(void)
-{
- platform_driver_unregister(&jz4780_dma_driver);
-}
-module_exit(jz4780_dma_exit);
+subsys_platform_driver(jz4780_dma_driver);
MODULE_AUTHOR("Alex Smith <alex@alex-smith.me.uk>");
MODULE_DESCRIPTION("Ingenic JZ4780 DMA controller driver");
diff --git a/drivers/dma/dw/platform.c b/drivers/dma/dw/platform.c
index 5c1c97eb28a4..7576ad255343 100644
--- a/drivers/dma/dw/platform.c
+++ b/drivers/dma/dw/platform.c
@@ -195,17 +195,7 @@ static struct platform_driver dw_driver = {
},
};
-static int __init dw_init(void)
-{
- return platform_driver_register(&dw_driver);
-}
-subsys_initcall(dw_init);
-
-static void __exit dw_exit(void)
-{
- platform_driver_unregister(&dw_driver);
-}
-module_exit(dw_exit);
+subsys_platform_driver(dw_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("Synopsys DesignWare DMA Controller platform driver");
diff --git a/drivers/dma/fsl-edma-main.c b/drivers/dma/fsl-edma-main.c
index 630285c85482..c731104c3306 100644
--- a/drivers/dma/fsl-edma-main.c
+++ b/drivers/dma/fsl-edma-main.c
@@ -978,17 +978,7 @@ static struct platform_driver fsl_edma_driver = {
.remove = fsl_edma_remove,
};
-static int __init fsl_edma_init(void)
-{
- return platform_driver_register(&fsl_edma_driver);
-}
-subsys_initcall(fsl_edma_init);
-
-static void __exit fsl_edma_exit(void)
-{
- platform_driver_unregister(&fsl_edma_driver);
-}
-module_exit(fsl_edma_exit);
+subsys_platform_driver(fsl_edma_driver);
MODULE_DESCRIPTION("Freescale eDMA engine driver");
MODULE_LICENSE("GPL v2");
diff --git a/drivers/dma/mcf-edma-main.c b/drivers/dma/mcf-edma-main.c
index 9e1c6400c77b..fb6c06d4a8d1 100644
--- a/drivers/dma/mcf-edma-main.c
+++ b/drivers/dma/mcf-edma-main.c
@@ -282,17 +282,7 @@ bool mcf_edma_filter_fn(struct dma_chan *chan, void *param)
}
EXPORT_SYMBOL(mcf_edma_filter_fn);
-static int __init mcf_edma_init(void)
-{
- return platform_driver_register(&mcf_edma_driver);
-}
-subsys_initcall(mcf_edma_init);
-
-static void __exit mcf_edma_exit(void)
-{
- platform_driver_unregister(&mcf_edma_driver);
-}
-module_exit(mcf_edma_exit);
+subsys_platform_driver(mcf_edma_driver);
MODULE_ALIAS("platform:mcf-edma");
MODULE_DESCRIPTION("Freescale eDMA engine driver, ColdFire family");
diff --git a/drivers/dma/moxart-dma.c b/drivers/dma/moxart-dma.c
index 28be93eed4cc..d33733afce9d 100644
--- a/drivers/dma/moxart-dma.c
+++ b/drivers/dma/moxart-dma.c
@@ -646,17 +646,7 @@ static struct platform_driver moxart_driver = {
},
};
-static int moxart_init(void)
-{
- return platform_driver_register(&moxart_driver);
-}
-subsys_initcall(moxart_init);
-
-static void __exit moxart_exit(void)
-{
- platform_driver_unregister(&moxart_driver);
-}
-module_exit(moxart_exit);
+subsys_platform_driver(moxart_driver);
MODULE_AUTHOR("Jonas Jensen <jonas.jensen@gmail.com>");
MODULE_DESCRIPTION("MOXART DMA engine driver");
diff --git a/drivers/dma/owl-dma.c b/drivers/dma/owl-dma.c
index 9dc88d804a2e..c13c449ad05b 100644
--- a/drivers/dma/owl-dma.c
+++ b/drivers/dma/owl-dma.c
@@ -1254,17 +1254,7 @@ static struct platform_driver owl_dma_driver = {
},
};
-static int owl_dma_init(void)
-{
- return platform_driver_register(&owl_dma_driver);
-}
-subsys_initcall(owl_dma_init);
-
-static void __exit owl_dma_exit(void)
-{
- platform_driver_unregister(&owl_dma_driver);
-}
-module_exit(owl_dma_exit);
+subsys_platform_driver(owl_dma_driver);
MODULE_AUTHOR("David Liu <liuwei@actions-semi.com>");
MODULE_AUTHOR("Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>");
diff --git a/drivers/dma/sa11x0-dma.c b/drivers/dma/sa11x0-dma.c
index a6fa431530e3..70b191519053 100644
--- a/drivers/dma/sa11x0-dma.c
+++ b/drivers/dma/sa11x0-dma.c
@@ -1080,17 +1080,7 @@ static struct platform_driver sa11x0_dma_driver = {
.remove = sa11x0_dma_remove,
};
-static int __init sa11x0_dma_init(void)
-{
- return platform_driver_register(&sa11x0_dma_driver);
-}
-subsys_initcall(sa11x0_dma_init);
-
-static void __exit sa11x0_dma_exit(void)
-{
- platform_driver_unregister(&sa11x0_dma_driver);
-}
-module_exit(sa11x0_dma_exit);
+subsys_platform_driver(sa11x0_dma_driver);
MODULE_AUTHOR("Russell King");
MODULE_DESCRIPTION("SA-11x0 DMA driver");
diff --git a/drivers/dma/sf-pdma/sf-pdma.c b/drivers/dma/sf-pdma/sf-pdma.c
index 6f79cc28703e..d62563f6c7f0 100644
--- a/drivers/dma/sf-pdma/sf-pdma.c
+++ b/drivers/dma/sf-pdma/sf-pdma.c
@@ -639,19 +639,7 @@ static struct platform_driver sf_pdma_driver = {
},
};
-static int __init sf_pdma_init(void)
-{
- return platform_driver_register(&sf_pdma_driver);
-}
-
-static void __exit sf_pdma_exit(void)
-{
- platform_driver_unregister(&sf_pdma_driver);
-}
-
-/* do early init */
-subsys_initcall(sf_pdma_init);
-module_exit(sf_pdma_exit);
+subsys_platform_driver(sf_pdma_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("SiFive Platform DMA driver");
diff --git a/drivers/dma/ti/omap-dma.c b/drivers/dma/ti/omap-dma.c
index 55ece7fd0d99..da1f4eec9eb2 100644
--- a/drivers/dma/ti/omap-dma.c
+++ b/drivers/dma/ti/omap-dma.c
@@ -1941,17 +1941,7 @@ static bool omap_dma_filter_fn(struct dma_chan *chan, void *param)
return false;
}
-static int omap_dma_init(void)
-{
- return platform_driver_register(&omap_dma_driver);
-}
-subsys_initcall(omap_dma_init);
-
-static void __exit omap_dma_exit(void)
-{
- platform_driver_unregister(&omap_dma_driver);
-}
-module_exit(omap_dma_exit);
+subsys_platform_driver(omap_dma_driver);
MODULE_AUTHOR("Russell King");
MODULE_DESCRIPTION("Texas Instruments sDMA DMAengine support");
--
2.56.0
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 1/3] dmaengine: convert to subsys_platform_driver()
@ 2026-11-07 0:06 ` Brian Masney
0 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in these drivers and convert them over to use
the macro subsys_platform_driver().
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
drivers/dma/at_xdmac.c | 12 +-----------
drivers/dma/bestcomm/bestcomm.c | 18 +-----------------
drivers/dma/dma-jz4780.c | 12 +-----------
drivers/dma/dw/platform.c | 12 +-----------
drivers/dma/fsl-edma-main.c | 12 +-----------
drivers/dma/mcf-edma-main.c | 12 +-----------
drivers/dma/moxart-dma.c | 12 +-----------
drivers/dma/owl-dma.c | 12 +-----------
drivers/dma/sa11x0-dma.c | 12 +-----------
drivers/dma/sf-pdma/sf-pdma.c | 14 +-------------
drivers/dma/ti/omap-dma.c | 12 +-----------
11 files changed, 11 insertions(+), 129 deletions(-)
diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
index d40a8b40dddc..8b19d46ac812 100644
--- a/drivers/dma/at_xdmac.c
+++ b/drivers/dma/at_xdmac.c
@@ -2488,17 +2488,7 @@ static struct platform_driver at_xdmac_driver = {
}
};
-static int __init at_xdmac_init(void)
-{
- return platform_driver_register(&at_xdmac_driver);
-}
-subsys_initcall(at_xdmac_init);
-
-static void __exit at_xdmac_exit(void)
-{
- platform_driver_unregister(&at_xdmac_driver);
-}
-module_exit(at_xdmac_exit);
+subsys_platform_driver(at_xdmac_driver);
MODULE_DESCRIPTION("Atmel Extended DMA Controller driver");
MODULE_AUTHOR("Ludovic Desroches <ludovic.desroches@atmel.com>");
diff --git a/drivers/dma/bestcomm/bestcomm.c b/drivers/dma/bestcomm/bestcomm.c
index 241a91936fe8..1dadcc259382 100644
--- a/drivers/dma/bestcomm/bestcomm.c
+++ b/drivers/dma/bestcomm/bestcomm.c
@@ -474,23 +474,7 @@ static struct platform_driver mpc52xx_bcom_of_platform_driver = {
/* Module */
/* ======================================================================== */
-static int __init
-mpc52xx_bcom_init(void)
-{
- return platform_driver_register(&mpc52xx_bcom_of_platform_driver);
-}
-
-static void __exit
-mpc52xx_bcom_exit(void)
-{
- platform_driver_unregister(&mpc52xx_bcom_of_platform_driver);
-}
-
-/* If we're not a module, we must make sure everything is setup before */
-/* anyone tries to use us ... that's why we use subsys_initcall instead */
-/* of module_init. */
-subsys_initcall(mpc52xx_bcom_init);
-module_exit(mpc52xx_bcom_exit);
+subsys_platform_driver(mpc52xx_bcom_of_platform_driver);
MODULE_DESCRIPTION("Freescale MPC52xx BestComm DMA");
MODULE_AUTHOR("Sylvain Munaut <tnt@246tNt.com>");
diff --git a/drivers/dma/dma-jz4780.c b/drivers/dma/dma-jz4780.c
index ac2094e65ecf..32a40c17e0cd 100644
--- a/drivers/dma/dma-jz4780.c
+++ b/drivers/dma/dma-jz4780.c
@@ -1129,17 +1129,7 @@ static struct platform_driver jz4780_dma_driver = {
},
};
-static int __init jz4780_dma_init(void)
-{
- return platform_driver_register(&jz4780_dma_driver);
-}
-subsys_initcall(jz4780_dma_init);
-
-static void __exit jz4780_dma_exit(void)
-{
- platform_driver_unregister(&jz4780_dma_driver);
-}
-module_exit(jz4780_dma_exit);
+subsys_platform_driver(jz4780_dma_driver);
MODULE_AUTHOR("Alex Smith <alex@alex-smith.me.uk>");
MODULE_DESCRIPTION("Ingenic JZ4780 DMA controller driver");
diff --git a/drivers/dma/dw/platform.c b/drivers/dma/dw/platform.c
index 5c1c97eb28a4..7576ad255343 100644
--- a/drivers/dma/dw/platform.c
+++ b/drivers/dma/dw/platform.c
@@ -195,17 +195,7 @@ static struct platform_driver dw_driver = {
},
};
-static int __init dw_init(void)
-{
- return platform_driver_register(&dw_driver);
-}
-subsys_initcall(dw_init);
-
-static void __exit dw_exit(void)
-{
- platform_driver_unregister(&dw_driver);
-}
-module_exit(dw_exit);
+subsys_platform_driver(dw_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("Synopsys DesignWare DMA Controller platform driver");
diff --git a/drivers/dma/fsl-edma-main.c b/drivers/dma/fsl-edma-main.c
index 630285c85482..c731104c3306 100644
--- a/drivers/dma/fsl-edma-main.c
+++ b/drivers/dma/fsl-edma-main.c
@@ -978,17 +978,7 @@ static struct platform_driver fsl_edma_driver = {
.remove = fsl_edma_remove,
};
-static int __init fsl_edma_init(void)
-{
- return platform_driver_register(&fsl_edma_driver);
-}
-subsys_initcall(fsl_edma_init);
-
-static void __exit fsl_edma_exit(void)
-{
- platform_driver_unregister(&fsl_edma_driver);
-}
-module_exit(fsl_edma_exit);
+subsys_platform_driver(fsl_edma_driver);
MODULE_DESCRIPTION("Freescale eDMA engine driver");
MODULE_LICENSE("GPL v2");
diff --git a/drivers/dma/mcf-edma-main.c b/drivers/dma/mcf-edma-main.c
index 9e1c6400c77b..fb6c06d4a8d1 100644
--- a/drivers/dma/mcf-edma-main.c
+++ b/drivers/dma/mcf-edma-main.c
@@ -282,17 +282,7 @@ bool mcf_edma_filter_fn(struct dma_chan *chan, void *param)
}
EXPORT_SYMBOL(mcf_edma_filter_fn);
-static int __init mcf_edma_init(void)
-{
- return platform_driver_register(&mcf_edma_driver);
-}
-subsys_initcall(mcf_edma_init);
-
-static void __exit mcf_edma_exit(void)
-{
- platform_driver_unregister(&mcf_edma_driver);
-}
-module_exit(mcf_edma_exit);
+subsys_platform_driver(mcf_edma_driver);
MODULE_ALIAS("platform:mcf-edma");
MODULE_DESCRIPTION("Freescale eDMA engine driver, ColdFire family");
diff --git a/drivers/dma/moxart-dma.c b/drivers/dma/moxart-dma.c
index 28be93eed4cc..d33733afce9d 100644
--- a/drivers/dma/moxart-dma.c
+++ b/drivers/dma/moxart-dma.c
@@ -646,17 +646,7 @@ static struct platform_driver moxart_driver = {
},
};
-static int moxart_init(void)
-{
- return platform_driver_register(&moxart_driver);
-}
-subsys_initcall(moxart_init);
-
-static void __exit moxart_exit(void)
-{
- platform_driver_unregister(&moxart_driver);
-}
-module_exit(moxart_exit);
+subsys_platform_driver(moxart_driver);
MODULE_AUTHOR("Jonas Jensen <jonas.jensen@gmail.com>");
MODULE_DESCRIPTION("MOXART DMA engine driver");
diff --git a/drivers/dma/owl-dma.c b/drivers/dma/owl-dma.c
index 9dc88d804a2e..c13c449ad05b 100644
--- a/drivers/dma/owl-dma.c
+++ b/drivers/dma/owl-dma.c
@@ -1254,17 +1254,7 @@ static struct platform_driver owl_dma_driver = {
},
};
-static int owl_dma_init(void)
-{
- return platform_driver_register(&owl_dma_driver);
-}
-subsys_initcall(owl_dma_init);
-
-static void __exit owl_dma_exit(void)
-{
- platform_driver_unregister(&owl_dma_driver);
-}
-module_exit(owl_dma_exit);
+subsys_platform_driver(owl_dma_driver);
MODULE_AUTHOR("David Liu <liuwei@actions-semi.com>");
MODULE_AUTHOR("Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>");
diff --git a/drivers/dma/sa11x0-dma.c b/drivers/dma/sa11x0-dma.c
index a6fa431530e3..70b191519053 100644
--- a/drivers/dma/sa11x0-dma.c
+++ b/drivers/dma/sa11x0-dma.c
@@ -1080,17 +1080,7 @@ static struct platform_driver sa11x0_dma_driver = {
.remove = sa11x0_dma_remove,
};
-static int __init sa11x0_dma_init(void)
-{
- return platform_driver_register(&sa11x0_dma_driver);
-}
-subsys_initcall(sa11x0_dma_init);
-
-static void __exit sa11x0_dma_exit(void)
-{
- platform_driver_unregister(&sa11x0_dma_driver);
-}
-module_exit(sa11x0_dma_exit);
+subsys_platform_driver(sa11x0_dma_driver);
MODULE_AUTHOR("Russell King");
MODULE_DESCRIPTION("SA-11x0 DMA driver");
diff --git a/drivers/dma/sf-pdma/sf-pdma.c b/drivers/dma/sf-pdma/sf-pdma.c
index 6f79cc28703e..d62563f6c7f0 100644
--- a/drivers/dma/sf-pdma/sf-pdma.c
+++ b/drivers/dma/sf-pdma/sf-pdma.c
@@ -639,19 +639,7 @@ static struct platform_driver sf_pdma_driver = {
},
};
-static int __init sf_pdma_init(void)
-{
- return platform_driver_register(&sf_pdma_driver);
-}
-
-static void __exit sf_pdma_exit(void)
-{
- platform_driver_unregister(&sf_pdma_driver);
-}
-
-/* do early init */
-subsys_initcall(sf_pdma_init);
-module_exit(sf_pdma_exit);
+subsys_platform_driver(sf_pdma_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("SiFive Platform DMA driver");
diff --git a/drivers/dma/ti/omap-dma.c b/drivers/dma/ti/omap-dma.c
index 55ece7fd0d99..da1f4eec9eb2 100644
--- a/drivers/dma/ti/omap-dma.c
+++ b/drivers/dma/ti/omap-dma.c
@@ -1941,17 +1941,7 @@ static bool omap_dma_filter_fn(struct dma_chan *chan, void *param)
return false;
}
-static int omap_dma_init(void)
-{
- return platform_driver_register(&omap_dma_driver);
-}
-subsys_initcall(omap_dma_init);
-
-static void __exit omap_dma_exit(void)
-{
- platform_driver_unregister(&omap_dma_driver);
-}
-module_exit(omap_dma_exit);
+subsys_platform_driver(omap_dma_driver);
MODULE_AUTHOR("Russell King");
MODULE_DESCRIPTION("Texas Instruments sDMA DMAengine support");
--
2.56.0
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/3] dmaengine: convert to subsys_platform_driver_probe()
2026-11-07 0:06 ` Brian Masney
@ 2026-11-07 0:06 ` Brian Masney
-1 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in these drivers and convert them over to use
the macro subsys_platform_driver_probe().
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
drivers/dma/imx-dma.c | 6 +-----
drivers/dma/ste_dma40.c | 6 +-----
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/drivers/dma/imx-dma.c b/drivers/dma/imx-dma.c
index 81c6276436f8..8e0e82df8c5b 100644
--- a/drivers/dma/imx-dma.c
+++ b/drivers/dma/imx-dma.c
@@ -1238,11 +1238,7 @@ static struct platform_driver imxdma_driver = {
.remove = imxdma_remove,
};
-static int __init imxdma_module_init(void)
-{
- return platform_driver_probe(&imxdma_driver, imxdma_probe);
-}
-subsys_initcall(imxdma_module_init);
+subsys_platform_driver_probe(imxdma_driver, imxdma_probe);
MODULE_AUTHOR("Sascha Hauer, Pengutronix <s.hauer@pengutronix.de>");
MODULE_DESCRIPTION("i.MX dma driver");
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 2e83503f71d1..4fb54c8f2d23 100644
--- a/drivers/dma/ste_dma40.c
+++ b/drivers/dma/ste_dma40.c
@@ -3990,8 +3990,4 @@ static struct platform_driver d40_driver = {
},
};
-static int __init stedma40_init(void)
-{
- return platform_driver_probe(&d40_driver, d40_probe);
-}
-subsys_initcall(stedma40_init);
+subsys_platform_driver_probe(d40_driver, d40_probe);
--
2.56.0
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 2/3] dmaengine: convert to subsys_platform_driver_probe()
@ 2026-11-07 0:06 ` Brian Masney
0 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in these drivers and convert them over to use
the macro subsys_platform_driver_probe().
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
drivers/dma/imx-dma.c | 6 +-----
drivers/dma/ste_dma40.c | 6 +-----
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/drivers/dma/imx-dma.c b/drivers/dma/imx-dma.c
index 81c6276436f8..8e0e82df8c5b 100644
--- a/drivers/dma/imx-dma.c
+++ b/drivers/dma/imx-dma.c
@@ -1238,11 +1238,7 @@ static struct platform_driver imxdma_driver = {
.remove = imxdma_remove,
};
-static int __init imxdma_module_init(void)
-{
- return platform_driver_probe(&imxdma_driver, imxdma_probe);
-}
-subsys_initcall(imxdma_module_init);
+subsys_platform_driver_probe(imxdma_driver, imxdma_probe);
MODULE_AUTHOR("Sascha Hauer, Pengutronix <s.hauer@pengutronix.de>");
MODULE_DESCRIPTION("i.MX dma driver");
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 2e83503f71d1..4fb54c8f2d23 100644
--- a/drivers/dma/ste_dma40.c
+++ b/drivers/dma/ste_dma40.c
@@ -3990,8 +3990,4 @@ static struct platform_driver d40_driver = {
},
};
-static int __init stedma40_init(void)
-{
- return platform_driver_probe(&d40_driver, d40_probe);
-}
-subsys_initcall(stedma40_init);
+subsys_platform_driver_probe(d40_driver, d40_probe);
--
2.56.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH 2/3] dmaengine: convert to subsys_platform_driver_probe()
2026-11-07 0:06 ` Brian Masney
@ 2026-11-09 9:57 ` Linus Walleij
-1 siblings, 0 replies; 10+ messages in thread
From: Linus Walleij @ 2026-11-09 9:57 UTC (permalink / raw)
To: Brian Masney
Cc: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Zhang Wei, linux-arm-kernel, dmaengine,
linux-kernel, linux-mips, imx, linux-actions, linux-riscv,
linuxppc-dev
On Sat, Nov 7, 2026 at 1:06 AM Brian Masney <bmasney@redhat.com> wrote:
> Reduce boilerplate code in these drivers and convert them over to use
> the macro subsys_platform_driver_probe().
>
> Signed-off-by: Brian Masney <bmasney@redhat.com>
Makes sense.
Reviewed-by: Linus Walleij <linusw@kernel.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] dmaengine: convert to subsys_platform_driver_probe()
@ 2026-11-09 9:57 ` Linus Walleij
0 siblings, 0 replies; 10+ messages in thread
From: Linus Walleij @ 2026-11-09 9:57 UTC (permalink / raw)
To: Brian Masney
Cc: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Zhang Wei, linux-arm-kernel, dmaengine,
linux-kernel, linux-mips, imx, linux-actions, linux-riscv,
linuxppc-dev
On Sat, Nov 7, 2026 at 1:06 AM Brian Masney <bmasney@redhat.com> wrote:
> Reduce boilerplate code in these drivers and convert them over to use
> the macro subsys_platform_driver_probe().
>
> Signed-off-by: Brian Masney <bmasney@redhat.com>
Makes sense.
Reviewed-by: Linus Walleij <linusw@kernel.org>
Yours,
Linus Walleij
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 3/3] dmaengine: fsldma: convert to subsys_platform_driver()
2026-11-07 0:06 ` Brian Masney
@ 2026-11-07 0:06 ` Brian Masney
-1 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in this driver and convert it over to use
the macro subsys_platform_driver().
Note that this driver previously displayed a log message during init
time, and this change drops that message.
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
drivers/dma/fsldma.c | 14 +-------------
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/dma/fsldma.c b/drivers/dma/fsldma.c
index 99a878ed1444..fd3dca7616a8 100644
--- a/drivers/dma/fsldma.c
+++ b/drivers/dma/fsldma.c
@@ -1431,19 +1431,7 @@ static struct platform_driver fsldma_of_driver = {
/* Module Init / Exit */
/*----------------------------------------------------------------------------*/
-static __init int fsldma_init(void)
-{
- pr_info("Freescale Elo series DMA driver\n");
- return platform_driver_register(&fsldma_of_driver);
-}
-
-static void __exit fsldma_exit(void)
-{
- platform_driver_unregister(&fsldma_of_driver);
-}
-
-subsys_initcall(fsldma_init);
-module_exit(fsldma_exit);
+subsys_platform_driver(fsldma_of_driver);
MODULE_DESCRIPTION("Freescale Elo series DMA driver");
MODULE_LICENSE("GPL");
--
2.56.0
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 3/3] dmaengine: fsldma: convert to subsys_platform_driver()
@ 2026-11-07 0:06 ` Brian Masney
0 siblings, 0 replies; 10+ messages in thread
From: Brian Masney @ 2026-11-07 0:06 UTC (permalink / raw)
To: Ludovic Desroches, Vinod Koul, Frank Li, Paul Cercueil,
Viresh Kumar, Andy Shevchenko, Andreas Färber,
Manivannan Sadhasivam, Paul Walmsley, Samuel Holland,
Vignesh Raghavendra, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, Linus Walleij, Zhang Wei
Cc: linux-arm-kernel, dmaengine, linux-kernel, linux-mips, imx,
linux-actions, linux-riscv, linuxppc-dev, Brian Masney
Reduce boilerplate code in this driver and convert it over to use
the macro subsys_platform_driver().
Note that this driver previously displayed a log message during init
time, and this change drops that message.
Signed-off-by: Brian Masney <bmasney@redhat.com>
---
drivers/dma/fsldma.c | 14 +-------------
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/dma/fsldma.c b/drivers/dma/fsldma.c
index 99a878ed1444..fd3dca7616a8 100644
--- a/drivers/dma/fsldma.c
+++ b/drivers/dma/fsldma.c
@@ -1431,19 +1431,7 @@ static struct platform_driver fsldma_of_driver = {
/* Module Init / Exit */
/*----------------------------------------------------------------------------*/
-static __init int fsldma_init(void)
-{
- pr_info("Freescale Elo series DMA driver\n");
- return platform_driver_register(&fsldma_of_driver);
-}
-
-static void __exit fsldma_exit(void)
-{
- platform_driver_unregister(&fsldma_of_driver);
-}
-
-subsys_initcall(fsldma_init);
-module_exit(fsldma_exit);
+subsys_platform_driver(fsldma_of_driver);
MODULE_DESCRIPTION("Freescale Elo series DMA driver");
MODULE_LICENSE("GPL");
--
2.56.0
^ permalink raw reply related [flat|nested] 10+ messages in thread