Skip to contentSkip to conversation

uio: convert drivers/uio/* to use module_platform_driver()

PATCH In review Axel Lin 2 participants 2 messages Nov 26, 2011 – Nov 28, 2011 Userspace I/O (UIO)

lore
Share & tools
lore

Axel Lin

author
#0 lore raw

PATCH 1/1 uio: convert drivers/uio/* to use module_platform_driver()

This patch converts the drivers in drivers/uio/* to use the module_platform_driver() macro which makes the code smaller and a bit simpler.

Cc Uwe Kleine-König <[email protected]>
Cc Magnus Damm <[email protected]>
Cc Amit Chatterjee <[email protected]>
Cc Pratheesh Gangadhar <[email protected]>
---
 drivers/uio/uio_pdrv.c        |   12 +-----------
 drivers/uio/uio_pdrv_genirq.c |   13 +------------
 drivers/uio/uio_pruss.c       |   14 +-------------
 3 files changed, 3 insertions(+), 36 deletions(-)
drivers/uio/uio_pdrv.c +1 -11
index ff50595..72d3646 100644
@@ -104,17 +104,7 @@ static struct platform_driver uio_pdrv = {
 	},
 };
 
-static int __init uio_pdrv_init(void)
-{
-	return platform_driver_register(&uio_pdrv);
-}
-
-static void __exit uio_pdrv_exit(void)
-{
-	platform_driver_unregister(&uio_pdrv);
-}
-module_init(uio_pdrv_init);
-module_exit(uio_pdrv_exit);
+module_platform_driver(uio_pdrv);
 
 MODULE_AUTHOR("Uwe Kleine-Koenig");
 MODULE_DESCRIPTION("Userspace I/O platform driver");
drivers/uio/uio_pdrv_genirq.c +1 -12
index 25de302..b98371d 100644
@@ -273,18 +273,7 @@ static struct platform_driver uio_pdrv_genirq = {
 	},
 };
 
-static int __init uio_pdrv_genirq_init(void)
-{
-	return platform_driver_register(&uio_pdrv_genirq);
-}
-
-static void __exit uio_pdrv_genirq_exit(void)
-{
-	platform_driver_unregister(&uio_pdrv_genirq);
-}
-
-module_init(uio_pdrv_genirq_init);
-module_exit(uio_pdrv_genirq_exit);
+module_platform_driver(uio_pdrv_genirq);
 
 MODULE_AUTHOR("Magnus Damm");
 MODULE_DESCRIPTION("Userspace I/O platform driver with generic IRQ handling");
drivers/uio/uio_pruss.c +1 -13
index e67b566..33a7a27 100644
@@ -227,19 +227,7 @@ static struct platform_driver pruss_driver = {
 		   },
 };
 
-static int __init pruss_init_module(void)
-{
-	return platform_driver_register(&pruss_driver);
-}
-
-module_init(pruss_init_module);
-
-static void __exit pruss_exit_module(void)
-{
-	platform_driver_unregister(&pruss_driver);
-}
-
-module_exit(pruss_exit_module);
+module_platform_driver(pruss_driver);
 
 MODULE_LICENSE("GPL v2");
 MODULE_VERSION(DRV_VERSION);
Signature · 1 line
1.7.5.4
#1 lore raw
Quoted 9 lines · Axel Lin

This patch converts the drivers in drivers/uio/* to use the module_platform_driver() macro which makes the code smaller and a bit simpler.

Cc Uwe Kleine-König <[email protected]>
Cc Magnus Damm <[email protected]>
Cc Amit Chatterjee <[email protected]>
Cc Pratheesh Gangadhar <[email protected]>

Looks alright to me.

Thanks,
Hans

Quoted 84 lines
---
 drivers/uio/uio_pdrv.c        |   12 +-----------
 drivers/uio/uio_pdrv_genirq.c |   13 +------------
 drivers/uio/uio_pruss.c       |   14 +-------------
 3 files changed, 3 insertions(+), 36 deletions(-)
drivers/uio/uio_pdrv.c +1 -11
index ff50595..72d3646 100644
@@ -104,17 +104,7 @@ static struct platform_driver uio_pdrv = {
 	},
 };
 
-static int __init uio_pdrv_init(void)
-{
-	return platform_driver_register(&uio_pdrv);
-}
-
-static void __exit uio_pdrv_exit(void)
-{
-	platform_driver_unregister(&uio_pdrv);
-}
-module_init(uio_pdrv_init);
-module_exit(uio_pdrv_exit);
+module_platform_driver(uio_pdrv);
 
 MODULE_AUTHOR("Uwe Kleine-Koenig");
 MODULE_DESCRIPTION("Userspace I/O platform driver");
drivers/uio/uio_pdrv_genirq.c +1 -12
index 25de302..b98371d 100644
@@ -273,18 +273,7 @@ static struct platform_driver uio_pdrv_genirq = {
 	},
 };
 
-static int __init uio_pdrv_genirq_init(void)
-{
-	return platform_driver_register(&uio_pdrv_genirq);
-}
-
-static void __exit uio_pdrv_genirq_exit(void)
-{
-	platform_driver_unregister(&uio_pdrv_genirq);
-}
-
-module_init(uio_pdrv_genirq_init);
-module_exit(uio_pdrv_genirq_exit);
+module_platform_driver(uio_pdrv_genirq);
 
 MODULE_AUTHOR("Magnus Damm");
 MODULE_DESCRIPTION("Userspace I/O platform driver with generic IRQ handling");
drivers/uio/uio_pruss.c +1 -13
index e67b566..33a7a27 100644
@@ -227,19 +227,7 @@ static struct platform_driver pruss_driver = {
 		   },
 };
 
-static int __init pruss_init_module(void)
-{
-	return platform_driver_register(&pruss_driver);
-}
-
-module_init(pruss_init_module);
-
-static void __exit pruss_exit_module(void)
-{
-	platform_driver_unregister(&pruss_driver);
-}
-
-module_exit(pruss_exit_module);
+module_platform_driver(pruss_driver);
 
 MODULE_LICENSE("GPL v2");
 MODULE_VERSION(DRV_VERSION);
Signature · 1 line
1.7.5.4