[PATCH] PCI: altera-msi: Convert to subsys_platform_driver()

[PATCH] PCI: altera-msi: Convert to subsys_platform_driver()

[Peng Fan (OSS)]

From: Peng Fan <peng.fan@nxp.com>

Replace the open-coded subsys_initcall()/module_exit() platform driver
registration with subsys_platform_driver().

No functional change.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/pci/controller/pcie-altera-msi.c | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/drivers/pci/controller/pcie-altera-msi.c b/drivers/pci/controller/pcie-altera-msi.c
index ea2ca2e70f20..8c0696cbd4ae 100644
--- a/drivers/pci/controller/pcie-altera-msi.c
+++ b/drivers/pci/controller/pcie-altera-msi.c
@@ -267,18 +267,7 @@ static struct platform_driver altera_msi_driver = {
 	.remove = altera_msi_remove,
 };
 
-static int __init altera_msi_init(void)
-{
-	return platform_driver_register(&altera_msi_driver);
-}
-
-static void __exit altera_msi_exit(void)
-{
-	platform_driver_unregister(&altera_msi_driver);
-}
-
-subsys_initcall(altera_msi_init);
+subsys_platform_driver(altera_msi_driver);
 MODULE_DEVICE_TABLE(of, altera_msi_of_match);
-module_exit(altera_msi_exit);
 MODULE_DESCRIPTION("Altera PCIe MSI support driver");
 MODULE_LICENSE("GPL v2");
-- 
2.51.0

Re: [PATCH] PCI: altera-msi: Convert to subsys_platform_driver()

[Manivannan Sadhasivam]

On Wed, 04 Nov 2026 19:32:35 +0800, Peng Fan (OSS) wrote:
> PCI: altera-msi: Convert to subsys_platform_driver()

Applied, thanks!

[1/1] PCI: altera-msi: Convert to subsys_platform_driver()
      commit: e5b8af7c98f184590b82d505f8ade025ba7cafae

Best regards,
-- 
மணிவண்ணன் சதாசிவம்

Re: [PATCH] PCI: altera-msi: Convert to subsys_platform_driver()

[Bjorn Helgaas]

[+cc Altera, X-Gene folks]

On Wed, Nov 04, 2026 at 07:32:35PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> Replace the open-coded subsys_initcall()/module_exit() platform driver
> registration with subsys_platform_driver().
> 
> No functional change.
> 
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  drivers/pci/controller/pcie-altera-msi.c | 13 +------------
>  1 file changed, 1 insertion(+), 12 deletions(-)
> 
> diff --git a/drivers/pci/controller/pcie-altera-msi.c b/drivers/pci/controller/pcie-altera-msi.c
> index ea2ca2e70f20..8c0696cbd4ae 100644
> --- a/drivers/pci/controller/pcie-altera-msi.c
> +++ b/drivers/pci/controller/pcie-altera-msi.c
> @@ -267,18 +267,7 @@ static struct platform_driver altera_msi_driver = {
>  	.remove = altera_msi_remove,
>  };
>  
> -static int __init altera_msi_init(void)
> -{
> -	return platform_driver_register(&altera_msi_driver);
> -}
> -
> -static void __exit altera_msi_exit(void)
> -{
> -	platform_driver_unregister(&altera_msi_driver);
> -}
> -
> -subsys_initcall(altera_msi_init);
> +subsys_platform_driver(altera_msi_driver);
>  MODULE_DEVICE_TABLE(of, altera_msi_of_match);
> -module_exit(altera_msi_exit);
>  MODULE_DESCRIPTION("Altera PCIe MSI support driver");
>  MODULE_LICENSE("GPL v2");

Looks like a nice improvement.

There are only three drivers that have separate MSI Kconfig options
and separate -msi.c files: altera, iproc, and xgene.

pcie-iproc-msi.c doesn't look like it has its own DT .compatible
string and driver registration stuff, but altera and xgene do.  I
wonder if it's possible to integrate them into the main drivers so we
don't need separate Kconfig and separate .c files?