All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-27 12:45 ` Axel Lin
  0 siblings, 0 replies; 20+ messages in thread
From: Axel Lin @ 2011-11-27 12:45 UTC (permalink / raw)
  To: linux-kernel
  Cc: David Woodhouse, Jiandong Zheng, Scott Branden, Wan ZongShun,
	Dmitry Eremin-Solenikov, Jamie Iles, Olof Johansson,
	Kyungmin Park, Haojian Zhuang, Artem Bityutskiy, Ralf Baechle,
	Brian Norris, Lars-Peter Clausen, Manuel Lauss, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

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

Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Jiandong Zheng <jdzheng@broadcom.com>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Wan ZongShun <mcuos.com@gmail.com>
Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
Cc: Jamie Iles <jamie@jamieiles.com>
Cc: Olof Johansson <olof@lixom.net>
Cc: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: Brian Norris <computersforpeace@gmail.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Manuel Lauss <manuel.lauss@googlemail.com>
Cc: Vimal Singh <vimal.newwork@gmail.com>
Cc: Andres Salomon <dilinger@queued.net>
Cc: Sukumar Ghorai <s-ghorai@ti.com>
Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/mtd/maps/bcm963xx-flash.c   |   13 +------------
 drivers/mtd/maps/bfin-async-flash.c |   12 +-----------
 drivers/mtd/maps/gpio-addr-flash.c  |   12 +-----------
 drivers/mtd/maps/ixp2000.c          |   12 +-----------
 drivers/mtd/maps/ixp4xx.c           |   14 +-------------
 drivers/mtd/maps/latch-addr-flash.c |   12 +-----------
 drivers/mtd/maps/physmap_of.c       |   13 +------------
 drivers/mtd/maps/pxa2xx-flash.c     |   13 +------------
 drivers/mtd/maps/rbtx4939-flash.c   |   13 +------------
 drivers/mtd/maps/sa1100-flash.c     |   13 +------------
 drivers/mtd/maps/sun_uflash.c       |   13 +------------
 drivers/mtd/nand/ams-delta.c        |   12 +-----------
 drivers/mtd/nand/au1550nd.c         |   12 +-----------
 drivers/mtd/nand/bcm_umi_nand.c     |   13 +------------
 drivers/mtd/nand/fsl_elbc_nand.c    |   13 +------------
 drivers/mtd/nand/fsl_upm.c          |   12 +-----------
 drivers/mtd/nand/jz4740_nand.c      |   12 +-----------
 drivers/mtd/nand/mpc5121_nfc.c      |   14 +-------------
 drivers/mtd/nand/ndfc.c             |   13 +------------
 drivers/mtd/nand/nomadik_nand.c     |   14 +-------------
 drivers/mtd/nand/nuc900_nand.c      |   13 +------------
 drivers/mtd/nand/omap2.c            |   15 +--------------
 drivers/mtd/nand/pasemi_nand.c      |   12 +-----------
 drivers/mtd/nand/plat_nand.c        |   13 +------------
 drivers/mtd/nand/pxa3xx_nand.c      |   12 +-----------
 drivers/mtd/nand/sharpsl.c          |   12 +-----------
 drivers/mtd/nand/socrates_nand.c    |   13 +------------
 drivers/mtd/nand/tmio_nand.c        |   13 +------------
 drivers/mtd/onenand/generic.c       |   16 ++--------------
 drivers/mtd/onenand/samsung.c       |   13 +------------
 30 files changed, 31 insertions(+), 356 deletions(-)

diff --git a/drivers/mtd/maps/bcm963xx-flash.c b/drivers/mtd/maps/bcm963xx-flash.c
index 736ca10..56927f5 100644
--- a/drivers/mtd/maps/bcm963xx-flash.c
+++ b/drivers/mtd/maps/bcm963xx-flash.c
@@ -257,18 +257,7 @@ static struct platform_driver bcm63xx_mtd_dev = {
 	},
 };
 
-static int __init bcm963xx_mtd_init(void)
-{
-	return platform_driver_register(&bcm63xx_mtd_dev);
-}
-
-static void __exit bcm963xx_mtd_exit(void)
-{
-	platform_driver_unregister(&bcm63xx_mtd_dev);
-}
-
-module_init(bcm963xx_mtd_init);
-module_exit(bcm963xx_mtd_exit);
+module_platform_driver(bcm63xx_mtd_dev);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("Broadcom BCM63xx MTD driver for CFE and RedBoot");
diff --git a/drivers/mtd/maps/bfin-async-flash.c b/drivers/mtd/maps/bfin-async-flash.c
index 6d6b2b5..650126c 100644
--- a/drivers/mtd/maps/bfin-async-flash.c
+++ b/drivers/mtd/maps/bfin-async-flash.c
@@ -190,17 +190,7 @@ static struct platform_driver bfin_flash_driver = {
 	},
 };
 
-static int __init bfin_flash_init(void)
-{
-	return platform_driver_register(&bfin_flash_driver);
-}
-module_init(bfin_flash_init);
-
-static void __exit bfin_flash_exit(void)
-{
-	platform_driver_unregister(&bfin_flash_driver);
-}
-module_exit(bfin_flash_exit);
+module_platform_driver(bfin_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("MTD map driver for Blackfins with flash/ethernet on same async bank");
diff --git a/drivers/mtd/maps/gpio-addr-flash.c b/drivers/mtd/maps/gpio-addr-flash.c
index 1ec66f0..33cce89 100644
--- a/drivers/mtd/maps/gpio-addr-flash.c
+++ b/drivers/mtd/maps/gpio-addr-flash.c
@@ -279,17 +279,7 @@ static struct platform_driver gpio_flash_driver = {
 	},
 };
 
-static int __init gpio_flash_init(void)
-{
-	return platform_driver_register(&gpio_flash_driver);
-}
-module_init(gpio_flash_init);
-
-static void __exit gpio_flash_exit(void)
-{
-	platform_driver_unregister(&gpio_flash_driver);
-}
-module_exit(gpio_flash_exit);
+module_platform_driver(gpio_flash_driver);
 
 MODULE_AUTHOR("Mike Frysinger <vapier@gentoo.org>");
 MODULE_DESCRIPTION("MTD map driver for flashes addressed physically and with gpios");
diff --git a/drivers/mtd/maps/ixp2000.c b/drivers/mtd/maps/ixp2000.c
index 437fcd2..fc7d4d0 100644
--- a/drivers/mtd/maps/ixp2000.c
+++ b/drivers/mtd/maps/ixp2000.c
@@ -246,18 +246,8 @@ static struct platform_driver ixp2000_flash_driver = {
 	},
 };
 
-static int __init ixp2000_flash_init(void)
-{
-	return platform_driver_register(&ixp2000_flash_driver);
-}
-
-static void __exit ixp2000_flash_exit(void)
-{
-	platform_driver_unregister(&ixp2000_flash_driver);
-}
+module_platform_driver(ixp2000_flash_driver);
 
-module_init(ixp2000_flash_init);
-module_exit(ixp2000_flash_exit);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Deepak Saxena <dsaxena@plexity.net>");
 MODULE_ALIAS("platform:IXP2000-Flash");
diff --git a/drivers/mtd/maps/ixp4xx.c b/drivers/mtd/maps/ixp4xx.c
index 3040901..8b54101 100644
--- a/drivers/mtd/maps/ixp4xx.c
+++ b/drivers/mtd/maps/ixp4xx.c
@@ -270,19 +270,7 @@ static struct platform_driver ixp4xx_flash_driver = {
 	},
 };
 
-static int __init ixp4xx_flash_init(void)
-{
-	return platform_driver_register(&ixp4xx_flash_driver);
-}
-
-static void __exit ixp4xx_flash_exit(void)
-{
-	platform_driver_unregister(&ixp4xx_flash_driver);
-}
-
-
-module_init(ixp4xx_flash_init);
-module_exit(ixp4xx_flash_exit);
+module_platform_driver(ixp4xx_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("MTD map driver for Intel IXP4xx systems");
diff --git a/drivers/mtd/maps/latch-addr-flash.c b/drivers/mtd/maps/latch-addr-flash.c
index 119baa7..8fed58e 100644
--- a/drivers/mtd/maps/latch-addr-flash.c
+++ b/drivers/mtd/maps/latch-addr-flash.c
@@ -223,17 +223,7 @@ static struct platform_driver latch_addr_flash_driver = {
 	},
 };
 
-static int __init latch_addr_flash_init(void)
-{
-	return platform_driver_register(&latch_addr_flash_driver);
-}
-module_init(latch_addr_flash_init);
-
-static void __exit latch_addr_flash_exit(void)
-{
-	platform_driver_unregister(&latch_addr_flash_driver);
-}
-module_exit(latch_addr_flash_exit);
+module_platform_driver(latch_addr_flash_driver);
 
 MODULE_AUTHOR("David Griego <dgriego@mvista.com>");
 MODULE_DESCRIPTION("MTD map driver for flashes addressed physically with upper "
diff --git a/drivers/mtd/maps/physmap_of.c b/drivers/mtd/maps/physmap_of.c
index 7d65f9d..2e6fb68 100644
--- a/drivers/mtd/maps/physmap_of.c
+++ b/drivers/mtd/maps/physmap_of.c
@@ -338,18 +338,7 @@ static struct platform_driver of_flash_driver = {
 	.remove		= of_flash_remove,
 };
 
-static int __init of_flash_init(void)
-{
-	return platform_driver_register(&of_flash_driver);
-}
-
-static void __exit of_flash_exit(void)
-{
-	platform_driver_unregister(&of_flash_driver);
-}
-
-module_init(of_flash_init);
-module_exit(of_flash_exit);
+module_platform_driver(of_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Vitaly Wool <vwool@ru.mvista.com>");
diff --git a/drivers/mtd/maps/pxa2xx-flash.c b/drivers/mtd/maps/pxa2xx-flash.c
index 2a25b67..274e399 100644
--- a/drivers/mtd/maps/pxa2xx-flash.c
+++ b/drivers/mtd/maps/pxa2xx-flash.c
@@ -142,18 +142,7 @@ static struct platform_driver pxa2xx_flash_driver = {
 	.shutdown	= pxa2xx_flash_shutdown,
 };
 
-static int __init init_pxa2xx_flash(void)
-{
-	return platform_driver_register(&pxa2xx_flash_driver);
-}
-
-static void __exit cleanup_pxa2xx_flash(void)
-{
-	platform_driver_unregister(&pxa2xx_flash_driver);
-}
-
-module_init(init_pxa2xx_flash);
-module_exit(cleanup_pxa2xx_flash);
+module_platform_driver(pxa2xx_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Nicolas Pitre <nico@fluxnic.net>");
diff --git a/drivers/mtd/maps/rbtx4939-flash.c b/drivers/mtd/maps/rbtx4939-flash.c
index 0237f19..bb7d204 100644
--- a/drivers/mtd/maps/rbtx4939-flash.c
+++ b/drivers/mtd/maps/rbtx4939-flash.c
@@ -137,18 +137,7 @@ static struct platform_driver rbtx4939_flash_driver = {
 	},
 };
 
-static int __init rbtx4939_flash_init(void)
-{
-	return platform_driver_register(&rbtx4939_flash_driver);
-}
-
-static void __exit rbtx4939_flash_exit(void)
-{
-	platform_driver_unregister(&rbtx4939_flash_driver);
-}
-
-module_init(rbtx4939_flash_init);
-module_exit(rbtx4939_flash_exit);
+module_platform_driver(rbtx4939_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("RBTX4939 MTD map driver");
diff --git a/drivers/mtd/maps/sa1100-flash.c b/drivers/mtd/maps/sa1100-flash.c
index fa9c0a9..ac3a290 100644
--- a/drivers/mtd/maps/sa1100-flash.c
+++ b/drivers/mtd/maps/sa1100-flash.c
@@ -394,18 +394,7 @@ static struct platform_driver sa1100_mtd_driver = {
 	},
 };
 
-static int __init sa1100_mtd_init(void)
-{
-	return platform_driver_register(&sa1100_mtd_driver);
-}
-
-static void __exit sa1100_mtd_exit(void)
-{
-	platform_driver_unregister(&sa1100_mtd_driver);
-}
-
-module_init(sa1100_mtd_init);
-module_exit(sa1100_mtd_exit);
+module_platform_driver(sa1100_mtd_driver);
 
 MODULE_AUTHOR("Nicolas Pitre");
 MODULE_DESCRIPTION("SA1100 CFI map driver");
diff --git a/drivers/mtd/maps/sun_uflash.c b/drivers/mtd/maps/sun_uflash.c
index 2d66234..175e537 100644
--- a/drivers/mtd/maps/sun_uflash.c
+++ b/drivers/mtd/maps/sun_uflash.c
@@ -158,15 +158,4 @@ static struct platform_driver uflash_driver = {
 	.remove		= __devexit_p(uflash_remove),
 };
 
-static int __init uflash_init(void)
-{
-	return platform_driver_register(&uflash_driver);
-}
-
-static void __exit uflash_exit(void)
-{
-	platform_driver_unregister(&uflash_driver);
-}
-
-module_init(uflash_init);
-module_exit(uflash_exit);
+module_platform_driver(uflash_driver);
diff --git a/drivers/mtd/nand/ams-delta.c b/drivers/mtd/nand/ams-delta.c
index 9e6b498..3197e97 100644
--- a/drivers/mtd/nand/ams-delta.c
+++ b/drivers/mtd/nand/ams-delta.c
@@ -280,17 +280,7 @@ static struct platform_driver ams_delta_nand_driver = {
 	},
 };
 
-static int __init ams_delta_nand_init(void)
-{
-	return platform_driver_register(&ams_delta_nand_driver);
-}
-module_init(ams_delta_nand_init);
-
-static void __exit ams_delta_nand_exit(void)
-{
-	platform_driver_unregister(&ams_delta_nand_driver);
-}
-module_exit(ams_delta_nand_exit);
+module_platform_driver(ams_delta_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Jonathan McDowell <noodles@earth.li>");
diff --git a/drivers/mtd/nand/au1550nd.c b/drivers/mtd/nand/au1550nd.c
index 77fb4e6..73abbc3 100644
--- a/drivers/mtd/nand/au1550nd.c
+++ b/drivers/mtd/nand/au1550nd.c
@@ -560,17 +560,7 @@ static struct platform_driver au1550nd_driver = {
 	.remove		= __devexit_p(au1550nd_remove),
 };
 
-static int __init au1550nd_load(void)
-{
-	return platform_driver_register(&au1550nd_driver);
-};
-
-static void __exit au1550nd_exit(void)
-{
-	platform_driver_unregister(&au1550nd_driver);
-};
-module_init(au1550nd_load);
-module_exit(au1550nd_exit);
+module_platform_driver(au1550nd_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Embedded Edge, LLC");
diff --git a/drivers/mtd/nand/bcm_umi_nand.c b/drivers/mtd/nand/bcm_umi_nand.c
index 46b58d6..50387fd 100644
--- a/drivers/mtd/nand/bcm_umi_nand.c
+++ b/drivers/mtd/nand/bcm_umi_nand.c
@@ -546,18 +546,7 @@ static struct platform_driver nand_driver = {
 	.resume = bcm_umi_nand_resume,
 };
 
-static int __init nand_init(void)
-{
-	return platform_driver_register(&nand_driver);
-}
-
-static void __exit nand_exit(void)
-{
-	platform_driver_unregister(&nand_driver);
-}
-
-module_init(nand_init);
-module_exit(nand_exit);
+module_platform_driver(nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Broadcom");
diff --git a/drivers/mtd/nand/fsl_elbc_nand.c b/drivers/mtd/nand/fsl_elbc_nand.c
index eedd8ee..f2f6e87 100644
--- a/drivers/mtd/nand/fsl_elbc_nand.c
+++ b/drivers/mtd/nand/fsl_elbc_nand.c
@@ -971,18 +971,7 @@ static struct platform_driver fsl_elbc_nand_driver = {
 	.remove = fsl_elbc_nand_remove,
 };
 
-static int __init fsl_elbc_nand_init(void)
-{
-	return platform_driver_register(&fsl_elbc_nand_driver);
-}
-
-static void __exit fsl_elbc_nand_exit(void)
-{
-	platform_driver_unregister(&fsl_elbc_nand_driver);
-}
-
-module_init(fsl_elbc_nand_init);
-module_exit(fsl_elbc_nand_exit);
+module_platform_driver(fsl_elbc_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Freescale");
diff --git a/drivers/mtd/nand/fsl_upm.c b/drivers/mtd/nand/fsl_upm.c
index b4f3cc9..45df542 100644
--- a/drivers/mtd/nand/fsl_upm.c
+++ b/drivers/mtd/nand/fsl_upm.c
@@ -353,17 +353,7 @@ static struct platform_driver of_fun_driver = {
 	.remove		= __devexit_p(fun_remove),
 };
 
-static int __init fun_module_init(void)
-{
-	return platform_driver_register(&of_fun_driver);
-}
-module_init(fun_module_init);
-
-static void __exit fun_module_exit(void)
-{
-	platform_driver_unregister(&of_fun_driver);
-}
-module_exit(fun_module_exit);
+module_platform_driver(of_fun_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
diff --git a/drivers/mtd/nand/jz4740_nand.c b/drivers/mtd/nand/jz4740_nand.c
index e266407..ac3b9f2 100644
--- a/drivers/mtd/nand/jz4740_nand.c
+++ b/drivers/mtd/nand/jz4740_nand.c
@@ -423,17 +423,7 @@ static struct platform_driver jz_nand_driver = {
 	},
 };
 
-static int __init jz_nand_init(void)
-{
-	return platform_driver_register(&jz_nand_driver);
-}
-module_init(jz_nand_init);
-
-static void __exit jz_nand_exit(void)
-{
-	platform_driver_unregister(&jz_nand_driver);
-}
-module_exit(jz_nand_exit);
+module_platform_driver(jz_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Lars-Peter Clausen <lars@metafoo.de>");
diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c
index 5ede647..c240cf1 100644
--- a/drivers/mtd/nand/mpc5121_nfc.c
+++ b/drivers/mtd/nand/mpc5121_nfc.c
@@ -879,19 +879,7 @@ static struct platform_driver mpc5121_nfc_driver = {
 	},
 };
 
-static int __init mpc5121_nfc_init(void)
-{
-	return platform_driver_register(&mpc5121_nfc_driver);
-}
-
-module_init(mpc5121_nfc_init);
-
-static void __exit mpc5121_nfc_cleanup(void)
-{
-	platform_driver_unregister(&mpc5121_nfc_driver);
-}
-
-module_exit(mpc5121_nfc_cleanup);
+module_platform_driver(mpc5121_nfc_driver);
 
 MODULE_AUTHOR("Freescale Semiconductor, Inc.");
 MODULE_DESCRIPTION("MPC5121 NAND MTD driver");
diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
index f8aacf4..ec68854 100644
--- a/drivers/mtd/nand/ndfc.c
+++ b/drivers/mtd/nand/ndfc.c
@@ -294,18 +294,7 @@ static struct platform_driver ndfc_driver = {
 	.remove = __devexit_p(ndfc_remove),
 };
 
-static int __init ndfc_nand_init(void)
-{
-	return platform_driver_register(&ndfc_driver);
-}
-
-static void __exit ndfc_nand_exit(void)
-{
-	platform_driver_unregister(&ndfc_driver);
-}
-
-module_init(ndfc_nand_init);
-module_exit(ndfc_nand_exit);
+module_platform_driver(ndfc_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Thomas Gleixner <tglx@linutronix.de>");
diff --git a/drivers/mtd/nand/nomadik_nand.c b/drivers/mtd/nand/nomadik_nand.c
index b463ecf..673dc6c 100644
--- a/drivers/mtd/nand/nomadik_nand.c
+++ b/drivers/mtd/nand/nomadik_nand.c
@@ -228,19 +228,7 @@ static struct platform_driver nomadik_nand_driver = {
 	},
 };
 
-static int __init nand_nomadik_init(void)
-{
-	pr_info("Nomadik NAND driver\n");
-	return platform_driver_register(&nomadik_nand_driver);
-}
-
-static void __exit nand_nomadik_exit(void)
-{
-	platform_driver_unregister(&nomadik_nand_driver);
-}
-
-module_init(nand_nomadik_init);
-module_exit(nand_nomadik_exit);
+module_platform_driver(nomadik_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("ST Microelectronics (sachin.verma@st.com)");
diff --git a/drivers/mtd/nand/nuc900_nand.c b/drivers/mtd/nand/nuc900_nand.c
index fa8faed..8febe46 100644
--- a/drivers/mtd/nand/nuc900_nand.c
+++ b/drivers/mtd/nand/nuc900_nand.c
@@ -364,18 +364,7 @@ static struct platform_driver nuc900_nand_driver = {
 	},
 };
 
-static int __init nuc900_nand_init(void)
-{
-	return platform_driver_register(&nuc900_nand_driver);
-}
-
-static void __exit nuc900_nand_exit(void)
-{
-	platform_driver_unregister(&nuc900_nand_driver);
-}
-
-module_init(nuc900_nand_init);
-module_exit(nuc900_nand_exit);
+module_platform_driver(nuc900_nand_driver);
 
 MODULE_AUTHOR("Wan ZongShun <mcuos.com@gmail.com>");
 MODULE_DESCRIPTION("w90p910/NUC9xx nand driver!");
diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c
index f745f00..b3a883e 100644
--- a/drivers/mtd/nand/omap2.c
+++ b/drivers/mtd/nand/omap2.c
@@ -1145,20 +1145,7 @@ static struct platform_driver omap_nand_driver = {
 	},
 };
 
-static int __init omap_nand_init(void)
-{
-	pr_info("%s driver initializing\n", DRIVER_NAME);
-
-	return platform_driver_register(&omap_nand_driver);
-}
-
-static void __exit omap_nand_exit(void)
-{
-	platform_driver_unregister(&omap_nand_driver);
-}
-
-module_init(omap_nand_init);
-module_exit(omap_nand_exit);
+module_platform_driver(omap_nand_driver);
 
 MODULE_ALIAS("platform:" DRIVER_NAME);
 MODULE_LICENSE("GPL");
diff --git a/drivers/mtd/nand/pasemi_nand.c b/drivers/mtd/nand/pasemi_nand.c
index a97264e..974dbf8 100644
--- a/drivers/mtd/nand/pasemi_nand.c
+++ b/drivers/mtd/nand/pasemi_nand.c
@@ -230,17 +230,7 @@ static struct platform_driver pasemi_nand_driver =
 	.remove		= pasemi_nand_remove,
 };
 
-static int __init pasemi_nand_init(void)
-{
-	return platform_driver_register(&pasemi_nand_driver);
-}
-module_init(pasemi_nand_init);
-
-static void __exit pasemi_nand_exit(void)
-{
-	platform_driver_unregister(&pasemi_nand_driver);
-}
-module_exit(pasemi_nand_exit);
+module_platform_driver(pasemi_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Egor Martovetsky <egor@pasemi.com>");
diff --git a/drivers/mtd/nand/plat_nand.c b/drivers/mtd/nand/plat_nand.c
index ea8e123..7f2da69 100644
--- a/drivers/mtd/nand/plat_nand.c
+++ b/drivers/mtd/nand/plat_nand.c
@@ -148,18 +148,7 @@ static struct platform_driver plat_nand_driver = {
 	},
 };
 
-static int __init plat_nand_init(void)
-{
-	return platform_driver_register(&plat_nand_driver);
-}
-
-static void __exit plat_nand_exit(void)
-{
-	platform_driver_unregister(&plat_nand_driver);
-}
-
-module_init(plat_nand_init);
-module_exit(plat_nand_exit);
+module_platform_driver(plat_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Vitaly Wool");
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 9eb7f87..90d6016 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1311,17 +1311,7 @@ static struct platform_driver pxa3xx_nand_driver = {
 	.resume		= pxa3xx_nand_resume,
 };
 
-static int __init pxa3xx_nand_init(void)
-{
-	return platform_driver_register(&pxa3xx_nand_driver);
-}
-module_init(pxa3xx_nand_init);
-
-static void __exit pxa3xx_nand_exit(void)
-{
-	platform_driver_unregister(&pxa3xx_nand_driver);
-}
-module_exit(pxa3xx_nand_exit);
+module_platform_driver(pxa3xx_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("PXA3xx NAND controller driver");
diff --git a/drivers/mtd/nand/sharpsl.c b/drivers/mtd/nand/sharpsl.c
index 619d2a5..b175c0f 100644
--- a/drivers/mtd/nand/sharpsl.c
+++ b/drivers/mtd/nand/sharpsl.c
@@ -230,17 +230,7 @@ static struct platform_driver sharpsl_nand_driver = {
 	.remove		= __devexit_p(sharpsl_nand_remove),
 };
 
-static int __init sharpsl_nand_init(void)
-{
-	return platform_driver_register(&sharpsl_nand_driver);
-}
-module_init(sharpsl_nand_init);
-
-static void __exit sharpsl_nand_exit(void)
-{
-	platform_driver_unregister(&sharpsl_nand_driver);
-}
-module_exit(sharpsl_nand_exit);
+module_platform_driver(sharpsl_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Richard Purdie <rpurdie@rpsys.net>");
diff --git a/drivers/mtd/nand/socrates_nand.c b/drivers/mtd/nand/socrates_nand.c
index 0fb24f9..e02b08b 100644
--- a/drivers/mtd/nand/socrates_nand.c
+++ b/drivers/mtd/nand/socrates_nand.c
@@ -273,18 +273,7 @@ static struct platform_driver socrates_nand_driver = {
 	.remove		= __devexit_p(socrates_nand_remove),
 };
 
-static int __init socrates_nand_init(void)
-{
-	return platform_driver_register(&socrates_nand_driver);
-}
-
-static void __exit socrates_nand_exit(void)
-{
-	platform_driver_unregister(&socrates_nand_driver);
-}
-
-module_init(socrates_nand_init);
-module_exit(socrates_nand_exit);
+module_platform_driver(socrates_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Ilya Yanok");
diff --git a/drivers/mtd/nand/tmio_nand.c b/drivers/mtd/nand/tmio_nand.c
index beebd95..6caa0cd 100644
--- a/drivers/mtd/nand/tmio_nand.c
+++ b/drivers/mtd/nand/tmio_nand.c
@@ -533,18 +533,7 @@ static struct platform_driver tmio_driver = {
 	.resume		= tmio_resume,
 };
 
-static int __init tmio_init(void)
-{
-	return platform_driver_register(&tmio_driver);
-}
-
-static void __exit tmio_exit(void)
-{
-	platform_driver_unregister(&tmio_driver);
-}
-
-module_init(tmio_init);
-module_exit(tmio_exit);
+module_platform_driver(tmio_driver);
 
 MODULE_LICENSE("GPL v2");
 MODULE_AUTHOR("Ian Molton, Dirk Opfer, Chris Humbert, Dmitry Baryshkov");
diff --git a/drivers/mtd/onenand/generic.c b/drivers/mtd/onenand/generic.c
index 7813095..0ccd5bf 100644
--- a/drivers/mtd/onenand/generic.c
+++ b/drivers/mtd/onenand/generic.c
@@ -115,21 +115,9 @@ static struct platform_driver generic_onenand_driver = {
 	.remove		= __devexit_p(generic_onenand_remove),
 };
 
-MODULE_ALIAS("platform:" DRIVER_NAME);
-
-static int __init generic_onenand_init(void)
-{
-	return platform_driver_register(&generic_onenand_driver);
-}
-
-static void __exit generic_onenand_exit(void)
-{
-	platform_driver_unregister(&generic_onenand_driver);
-}
-
-module_init(generic_onenand_init);
-module_exit(generic_onenand_exit);
+module_platform_driver(generic_onenand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");
 MODULE_DESCRIPTION("Glue layer for OneNAND flash on generic boards");
+MODULE_ALIAS("platform:" DRIVER_NAME);
diff --git a/drivers/mtd/onenand/samsung.c b/drivers/mtd/onenand/samsung.c
index 5474547..fa1ee43 100644
--- a/drivers/mtd/onenand/samsung.c
+++ b/drivers/mtd/onenand/samsung.c
@@ -1133,18 +1133,7 @@ static struct platform_driver s3c_onenand_driver = {
 	.remove         = __devexit_p(s3c_onenand_remove),
 };
 
-static int __init s3c_onenand_init(void)
-{
-	return platform_driver_register(&s3c_onenand_driver);
-}
-
-static void __exit s3c_onenand_exit(void)
-{
-	platform_driver_unregister(&s3c_onenand_driver);
-}
-
-module_init(s3c_onenand_init);
-module_exit(s3c_onenand_exit);
+module_platform_driver(s3c_onenand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 20+ messages in thread
* [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-27 12:45 ` Axel Lin
  0 siblings, 0 replies; 20+ messages in thread
From: Axel Lin @ 2011-11-27 12:45 UTC (permalink / raw)
  To: linux-kernel
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	Manuel Lauss, Haojian Zhuang, Kyungmin Park, Vimal Singh,
	Ralf Baechle, Jiandong Zheng, Andres Salomon, Olof Johansson,
	Jamie Iles, Brian Norris, David Woodhouse, linux-mtd

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

Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Jiandong Zheng <jdzheng@broadcom.com>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Wan ZongShun <mcuos.com@gmail.com>
Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
Cc: Jamie Iles <jamie@jamieiles.com>
Cc: Olof Johansson <olof@lixom.net>
Cc: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: Brian Norris <computersforpeace@gmail.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Manuel Lauss <manuel.lauss@googlemail.com>
Cc: Vimal Singh <vimal.newwork@gmail.com>
Cc: Andres Salomon <dilinger@queued.net>
Cc: Sukumar Ghorai <s-ghorai@ti.com>
Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/mtd/maps/bcm963xx-flash.c   |   13 +------------
 drivers/mtd/maps/bfin-async-flash.c |   12 +-----------
 drivers/mtd/maps/gpio-addr-flash.c  |   12 +-----------
 drivers/mtd/maps/ixp2000.c          |   12 +-----------
 drivers/mtd/maps/ixp4xx.c           |   14 +-------------
 drivers/mtd/maps/latch-addr-flash.c |   12 +-----------
 drivers/mtd/maps/physmap_of.c       |   13 +------------
 drivers/mtd/maps/pxa2xx-flash.c     |   13 +------------
 drivers/mtd/maps/rbtx4939-flash.c   |   13 +------------
 drivers/mtd/maps/sa1100-flash.c     |   13 +------------
 drivers/mtd/maps/sun_uflash.c       |   13 +------------
 drivers/mtd/nand/ams-delta.c        |   12 +-----------
 drivers/mtd/nand/au1550nd.c         |   12 +-----------
 drivers/mtd/nand/bcm_umi_nand.c     |   13 +------------
 drivers/mtd/nand/fsl_elbc_nand.c    |   13 +------------
 drivers/mtd/nand/fsl_upm.c          |   12 +-----------
 drivers/mtd/nand/jz4740_nand.c      |   12 +-----------
 drivers/mtd/nand/mpc5121_nfc.c      |   14 +-------------
 drivers/mtd/nand/ndfc.c             |   13 +------------
 drivers/mtd/nand/nomadik_nand.c     |   14 +-------------
 drivers/mtd/nand/nuc900_nand.c      |   13 +------------
 drivers/mtd/nand/omap2.c            |   15 +--------------
 drivers/mtd/nand/pasemi_nand.c      |   12 +-----------
 drivers/mtd/nand/plat_nand.c        |   13 +------------
 drivers/mtd/nand/pxa3xx_nand.c      |   12 +-----------
 drivers/mtd/nand/sharpsl.c          |   12 +-----------
 drivers/mtd/nand/socrates_nand.c    |   13 +------------
 drivers/mtd/nand/tmio_nand.c        |   13 +------------
 drivers/mtd/onenand/generic.c       |   16 ++--------------
 drivers/mtd/onenand/samsung.c       |   13 +------------
 30 files changed, 31 insertions(+), 356 deletions(-)

diff --git a/drivers/mtd/maps/bcm963xx-flash.c b/drivers/mtd/maps/bcm963xx-flash.c
index 736ca10..56927f5 100644
--- a/drivers/mtd/maps/bcm963xx-flash.c
+++ b/drivers/mtd/maps/bcm963xx-flash.c
@@ -257,18 +257,7 @@ static struct platform_driver bcm63xx_mtd_dev = {
 	},
 };
 
-static int __init bcm963xx_mtd_init(void)
-{
-	return platform_driver_register(&bcm63xx_mtd_dev);
-}
-
-static void __exit bcm963xx_mtd_exit(void)
-{
-	platform_driver_unregister(&bcm63xx_mtd_dev);
-}
-
-module_init(bcm963xx_mtd_init);
-module_exit(bcm963xx_mtd_exit);
+module_platform_driver(bcm63xx_mtd_dev);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("Broadcom BCM63xx MTD driver for CFE and RedBoot");
diff --git a/drivers/mtd/maps/bfin-async-flash.c b/drivers/mtd/maps/bfin-async-flash.c
index 6d6b2b5..650126c 100644
--- a/drivers/mtd/maps/bfin-async-flash.c
+++ b/drivers/mtd/maps/bfin-async-flash.c
@@ -190,17 +190,7 @@ static struct platform_driver bfin_flash_driver = {
 	},
 };
 
-static int __init bfin_flash_init(void)
-{
-	return platform_driver_register(&bfin_flash_driver);
-}
-module_init(bfin_flash_init);
-
-static void __exit bfin_flash_exit(void)
-{
-	platform_driver_unregister(&bfin_flash_driver);
-}
-module_exit(bfin_flash_exit);
+module_platform_driver(bfin_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("MTD map driver for Blackfins with flash/ethernet on same async bank");
diff --git a/drivers/mtd/maps/gpio-addr-flash.c b/drivers/mtd/maps/gpio-addr-flash.c
index 1ec66f0..33cce89 100644
--- a/drivers/mtd/maps/gpio-addr-flash.c
+++ b/drivers/mtd/maps/gpio-addr-flash.c
@@ -279,17 +279,7 @@ static struct platform_driver gpio_flash_driver = {
 	},
 };
 
-static int __init gpio_flash_init(void)
-{
-	return platform_driver_register(&gpio_flash_driver);
-}
-module_init(gpio_flash_init);
-
-static void __exit gpio_flash_exit(void)
-{
-	platform_driver_unregister(&gpio_flash_driver);
-}
-module_exit(gpio_flash_exit);
+module_platform_driver(gpio_flash_driver);
 
 MODULE_AUTHOR("Mike Frysinger <vapier@gentoo.org>");
 MODULE_DESCRIPTION("MTD map driver for flashes addressed physically and with gpios");
diff --git a/drivers/mtd/maps/ixp2000.c b/drivers/mtd/maps/ixp2000.c
index 437fcd2..fc7d4d0 100644
--- a/drivers/mtd/maps/ixp2000.c
+++ b/drivers/mtd/maps/ixp2000.c
@@ -246,18 +246,8 @@ static struct platform_driver ixp2000_flash_driver = {
 	},
 };
 
-static int __init ixp2000_flash_init(void)
-{
-	return platform_driver_register(&ixp2000_flash_driver);
-}
-
-static void __exit ixp2000_flash_exit(void)
-{
-	platform_driver_unregister(&ixp2000_flash_driver);
-}
+module_platform_driver(ixp2000_flash_driver);
 
-module_init(ixp2000_flash_init);
-module_exit(ixp2000_flash_exit);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Deepak Saxena <dsaxena@plexity.net>");
 MODULE_ALIAS("platform:IXP2000-Flash");
diff --git a/drivers/mtd/maps/ixp4xx.c b/drivers/mtd/maps/ixp4xx.c
index 3040901..8b54101 100644
--- a/drivers/mtd/maps/ixp4xx.c
+++ b/drivers/mtd/maps/ixp4xx.c
@@ -270,19 +270,7 @@ static struct platform_driver ixp4xx_flash_driver = {
 	},
 };
 
-static int __init ixp4xx_flash_init(void)
-{
-	return platform_driver_register(&ixp4xx_flash_driver);
-}
-
-static void __exit ixp4xx_flash_exit(void)
-{
-	platform_driver_unregister(&ixp4xx_flash_driver);
-}
-
-
-module_init(ixp4xx_flash_init);
-module_exit(ixp4xx_flash_exit);
+module_platform_driver(ixp4xx_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("MTD map driver for Intel IXP4xx systems");
diff --git a/drivers/mtd/maps/latch-addr-flash.c b/drivers/mtd/maps/latch-addr-flash.c
index 119baa7..8fed58e 100644
--- a/drivers/mtd/maps/latch-addr-flash.c
+++ b/drivers/mtd/maps/latch-addr-flash.c
@@ -223,17 +223,7 @@ static struct platform_driver latch_addr_flash_driver = {
 	},
 };
 
-static int __init latch_addr_flash_init(void)
-{
-	return platform_driver_register(&latch_addr_flash_driver);
-}
-module_init(latch_addr_flash_init);
-
-static void __exit latch_addr_flash_exit(void)
-{
-	platform_driver_unregister(&latch_addr_flash_driver);
-}
-module_exit(latch_addr_flash_exit);
+module_platform_driver(latch_addr_flash_driver);
 
 MODULE_AUTHOR("David Griego <dgriego@mvista.com>");
 MODULE_DESCRIPTION("MTD map driver for flashes addressed physically with upper "
diff --git a/drivers/mtd/maps/physmap_of.c b/drivers/mtd/maps/physmap_of.c
index 7d65f9d..2e6fb68 100644
--- a/drivers/mtd/maps/physmap_of.c
+++ b/drivers/mtd/maps/physmap_of.c
@@ -338,18 +338,7 @@ static struct platform_driver of_flash_driver = {
 	.remove		= of_flash_remove,
 };
 
-static int __init of_flash_init(void)
-{
-	return platform_driver_register(&of_flash_driver);
-}
-
-static void __exit of_flash_exit(void)
-{
-	platform_driver_unregister(&of_flash_driver);
-}
-
-module_init(of_flash_init);
-module_exit(of_flash_exit);
+module_platform_driver(of_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Vitaly Wool <vwool@ru.mvista.com>");
diff --git a/drivers/mtd/maps/pxa2xx-flash.c b/drivers/mtd/maps/pxa2xx-flash.c
index 2a25b67..274e399 100644
--- a/drivers/mtd/maps/pxa2xx-flash.c
+++ b/drivers/mtd/maps/pxa2xx-flash.c
@@ -142,18 +142,7 @@ static struct platform_driver pxa2xx_flash_driver = {
 	.shutdown	= pxa2xx_flash_shutdown,
 };
 
-static int __init init_pxa2xx_flash(void)
-{
-	return platform_driver_register(&pxa2xx_flash_driver);
-}
-
-static void __exit cleanup_pxa2xx_flash(void)
-{
-	platform_driver_unregister(&pxa2xx_flash_driver);
-}
-
-module_init(init_pxa2xx_flash);
-module_exit(cleanup_pxa2xx_flash);
+module_platform_driver(pxa2xx_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Nicolas Pitre <nico@fluxnic.net>");
diff --git a/drivers/mtd/maps/rbtx4939-flash.c b/drivers/mtd/maps/rbtx4939-flash.c
index 0237f19..bb7d204 100644
--- a/drivers/mtd/maps/rbtx4939-flash.c
+++ b/drivers/mtd/maps/rbtx4939-flash.c
@@ -137,18 +137,7 @@ static struct platform_driver rbtx4939_flash_driver = {
 	},
 };
 
-static int __init rbtx4939_flash_init(void)
-{
-	return platform_driver_register(&rbtx4939_flash_driver);
-}
-
-static void __exit rbtx4939_flash_exit(void)
-{
-	platform_driver_unregister(&rbtx4939_flash_driver);
-}
-
-module_init(rbtx4939_flash_init);
-module_exit(rbtx4939_flash_exit);
+module_platform_driver(rbtx4939_flash_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("RBTX4939 MTD map driver");
diff --git a/drivers/mtd/maps/sa1100-flash.c b/drivers/mtd/maps/sa1100-flash.c
index fa9c0a9..ac3a290 100644
--- a/drivers/mtd/maps/sa1100-flash.c
+++ b/drivers/mtd/maps/sa1100-flash.c
@@ -394,18 +394,7 @@ static struct platform_driver sa1100_mtd_driver = {
 	},
 };
 
-static int __init sa1100_mtd_init(void)
-{
-	return platform_driver_register(&sa1100_mtd_driver);
-}
-
-static void __exit sa1100_mtd_exit(void)
-{
-	platform_driver_unregister(&sa1100_mtd_driver);
-}
-
-module_init(sa1100_mtd_init);
-module_exit(sa1100_mtd_exit);
+module_platform_driver(sa1100_mtd_driver);
 
 MODULE_AUTHOR("Nicolas Pitre");
 MODULE_DESCRIPTION("SA1100 CFI map driver");
diff --git a/drivers/mtd/maps/sun_uflash.c b/drivers/mtd/maps/sun_uflash.c
index 2d66234..175e537 100644
--- a/drivers/mtd/maps/sun_uflash.c
+++ b/drivers/mtd/maps/sun_uflash.c
@@ -158,15 +158,4 @@ static struct platform_driver uflash_driver = {
 	.remove		= __devexit_p(uflash_remove),
 };
 
-static int __init uflash_init(void)
-{
-	return platform_driver_register(&uflash_driver);
-}
-
-static void __exit uflash_exit(void)
-{
-	platform_driver_unregister(&uflash_driver);
-}
-
-module_init(uflash_init);
-module_exit(uflash_exit);
+module_platform_driver(uflash_driver);
diff --git a/drivers/mtd/nand/ams-delta.c b/drivers/mtd/nand/ams-delta.c
index 9e6b498..3197e97 100644
--- a/drivers/mtd/nand/ams-delta.c
+++ b/drivers/mtd/nand/ams-delta.c
@@ -280,17 +280,7 @@ static struct platform_driver ams_delta_nand_driver = {
 	},
 };
 
-static int __init ams_delta_nand_init(void)
-{
-	return platform_driver_register(&ams_delta_nand_driver);
-}
-module_init(ams_delta_nand_init);
-
-static void __exit ams_delta_nand_exit(void)
-{
-	platform_driver_unregister(&ams_delta_nand_driver);
-}
-module_exit(ams_delta_nand_exit);
+module_platform_driver(ams_delta_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Jonathan McDowell <noodles@earth.li>");
diff --git a/drivers/mtd/nand/au1550nd.c b/drivers/mtd/nand/au1550nd.c
index 77fb4e6..73abbc3 100644
--- a/drivers/mtd/nand/au1550nd.c
+++ b/drivers/mtd/nand/au1550nd.c
@@ -560,17 +560,7 @@ static struct platform_driver au1550nd_driver = {
 	.remove		= __devexit_p(au1550nd_remove),
 };
 
-static int __init au1550nd_load(void)
-{
-	return platform_driver_register(&au1550nd_driver);
-};
-
-static void __exit au1550nd_exit(void)
-{
-	platform_driver_unregister(&au1550nd_driver);
-};
-module_init(au1550nd_load);
-module_exit(au1550nd_exit);
+module_platform_driver(au1550nd_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Embedded Edge, LLC");
diff --git a/drivers/mtd/nand/bcm_umi_nand.c b/drivers/mtd/nand/bcm_umi_nand.c
index 46b58d6..50387fd 100644
--- a/drivers/mtd/nand/bcm_umi_nand.c
+++ b/drivers/mtd/nand/bcm_umi_nand.c
@@ -546,18 +546,7 @@ static struct platform_driver nand_driver = {
 	.resume = bcm_umi_nand_resume,
 };
 
-static int __init nand_init(void)
-{
-	return platform_driver_register(&nand_driver);
-}
-
-static void __exit nand_exit(void)
-{
-	platform_driver_unregister(&nand_driver);
-}
-
-module_init(nand_init);
-module_exit(nand_exit);
+module_platform_driver(nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Broadcom");
diff --git a/drivers/mtd/nand/fsl_elbc_nand.c b/drivers/mtd/nand/fsl_elbc_nand.c
index eedd8ee..f2f6e87 100644
--- a/drivers/mtd/nand/fsl_elbc_nand.c
+++ b/drivers/mtd/nand/fsl_elbc_nand.c
@@ -971,18 +971,7 @@ static struct platform_driver fsl_elbc_nand_driver = {
 	.remove = fsl_elbc_nand_remove,
 };
 
-static int __init fsl_elbc_nand_init(void)
-{
-	return platform_driver_register(&fsl_elbc_nand_driver);
-}
-
-static void __exit fsl_elbc_nand_exit(void)
-{
-	platform_driver_unregister(&fsl_elbc_nand_driver);
-}
-
-module_init(fsl_elbc_nand_init);
-module_exit(fsl_elbc_nand_exit);
+module_platform_driver(fsl_elbc_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Freescale");
diff --git a/drivers/mtd/nand/fsl_upm.c b/drivers/mtd/nand/fsl_upm.c
index b4f3cc9..45df542 100644
--- a/drivers/mtd/nand/fsl_upm.c
+++ b/drivers/mtd/nand/fsl_upm.c
@@ -353,17 +353,7 @@ static struct platform_driver of_fun_driver = {
 	.remove		= __devexit_p(fun_remove),
 };
 
-static int __init fun_module_init(void)
-{
-	return platform_driver_register(&of_fun_driver);
-}
-module_init(fun_module_init);
-
-static void __exit fun_module_exit(void)
-{
-	platform_driver_unregister(&of_fun_driver);
-}
-module_exit(fun_module_exit);
+module_platform_driver(of_fun_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
diff --git a/drivers/mtd/nand/jz4740_nand.c b/drivers/mtd/nand/jz4740_nand.c
index e266407..ac3b9f2 100644
--- a/drivers/mtd/nand/jz4740_nand.c
+++ b/drivers/mtd/nand/jz4740_nand.c
@@ -423,17 +423,7 @@ static struct platform_driver jz_nand_driver = {
 	},
 };
 
-static int __init jz_nand_init(void)
-{
-	return platform_driver_register(&jz_nand_driver);
-}
-module_init(jz_nand_init);
-
-static void __exit jz_nand_exit(void)
-{
-	platform_driver_unregister(&jz_nand_driver);
-}
-module_exit(jz_nand_exit);
+module_platform_driver(jz_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Lars-Peter Clausen <lars@metafoo.de>");
diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c
index 5ede647..c240cf1 100644
--- a/drivers/mtd/nand/mpc5121_nfc.c
+++ b/drivers/mtd/nand/mpc5121_nfc.c
@@ -879,19 +879,7 @@ static struct platform_driver mpc5121_nfc_driver = {
 	},
 };
 
-static int __init mpc5121_nfc_init(void)
-{
-	return platform_driver_register(&mpc5121_nfc_driver);
-}
-
-module_init(mpc5121_nfc_init);
-
-static void __exit mpc5121_nfc_cleanup(void)
-{
-	platform_driver_unregister(&mpc5121_nfc_driver);
-}
-
-module_exit(mpc5121_nfc_cleanup);
+module_platform_driver(mpc5121_nfc_driver);
 
 MODULE_AUTHOR("Freescale Semiconductor, Inc.");
 MODULE_DESCRIPTION("MPC5121 NAND MTD driver");
diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
index f8aacf4..ec68854 100644
--- a/drivers/mtd/nand/ndfc.c
+++ b/drivers/mtd/nand/ndfc.c
@@ -294,18 +294,7 @@ static struct platform_driver ndfc_driver = {
 	.remove = __devexit_p(ndfc_remove),
 };
 
-static int __init ndfc_nand_init(void)
-{
-	return platform_driver_register(&ndfc_driver);
-}
-
-static void __exit ndfc_nand_exit(void)
-{
-	platform_driver_unregister(&ndfc_driver);
-}
-
-module_init(ndfc_nand_init);
-module_exit(ndfc_nand_exit);
+module_platform_driver(ndfc_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Thomas Gleixner <tglx@linutronix.de>");
diff --git a/drivers/mtd/nand/nomadik_nand.c b/drivers/mtd/nand/nomadik_nand.c
index b463ecf..673dc6c 100644
--- a/drivers/mtd/nand/nomadik_nand.c
+++ b/drivers/mtd/nand/nomadik_nand.c
@@ -228,19 +228,7 @@ static struct platform_driver nomadik_nand_driver = {
 	},
 };
 
-static int __init nand_nomadik_init(void)
-{
-	pr_info("Nomadik NAND driver\n");
-	return platform_driver_register(&nomadik_nand_driver);
-}
-
-static void __exit nand_nomadik_exit(void)
-{
-	platform_driver_unregister(&nomadik_nand_driver);
-}
-
-module_init(nand_nomadik_init);
-module_exit(nand_nomadik_exit);
+module_platform_driver(nomadik_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("ST Microelectronics (sachin.verma@st.com)");
diff --git a/drivers/mtd/nand/nuc900_nand.c b/drivers/mtd/nand/nuc900_nand.c
index fa8faed..8febe46 100644
--- a/drivers/mtd/nand/nuc900_nand.c
+++ b/drivers/mtd/nand/nuc900_nand.c
@@ -364,18 +364,7 @@ static struct platform_driver nuc900_nand_driver = {
 	},
 };
 
-static int __init nuc900_nand_init(void)
-{
-	return platform_driver_register(&nuc900_nand_driver);
-}
-
-static void __exit nuc900_nand_exit(void)
-{
-	platform_driver_unregister(&nuc900_nand_driver);
-}
-
-module_init(nuc900_nand_init);
-module_exit(nuc900_nand_exit);
+module_platform_driver(nuc900_nand_driver);
 
 MODULE_AUTHOR("Wan ZongShun <mcuos.com@gmail.com>");
 MODULE_DESCRIPTION("w90p910/NUC9xx nand driver!");
diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c
index f745f00..b3a883e 100644
--- a/drivers/mtd/nand/omap2.c
+++ b/drivers/mtd/nand/omap2.c
@@ -1145,20 +1145,7 @@ static struct platform_driver omap_nand_driver = {
 	},
 };
 
-static int __init omap_nand_init(void)
-{
-	pr_info("%s driver initializing\n", DRIVER_NAME);
-
-	return platform_driver_register(&omap_nand_driver);
-}
-
-static void __exit omap_nand_exit(void)
-{
-	platform_driver_unregister(&omap_nand_driver);
-}
-
-module_init(omap_nand_init);
-module_exit(omap_nand_exit);
+module_platform_driver(omap_nand_driver);
 
 MODULE_ALIAS("platform:" DRIVER_NAME);
 MODULE_LICENSE("GPL");
diff --git a/drivers/mtd/nand/pasemi_nand.c b/drivers/mtd/nand/pasemi_nand.c
index a97264e..974dbf8 100644
--- a/drivers/mtd/nand/pasemi_nand.c
+++ b/drivers/mtd/nand/pasemi_nand.c
@@ -230,17 +230,7 @@ static struct platform_driver pasemi_nand_driver =
 	.remove		= pasemi_nand_remove,
 };
 
-static int __init pasemi_nand_init(void)
-{
-	return platform_driver_register(&pasemi_nand_driver);
-}
-module_init(pasemi_nand_init);
-
-static void __exit pasemi_nand_exit(void)
-{
-	platform_driver_unregister(&pasemi_nand_driver);
-}
-module_exit(pasemi_nand_exit);
+module_platform_driver(pasemi_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Egor Martovetsky <egor@pasemi.com>");
diff --git a/drivers/mtd/nand/plat_nand.c b/drivers/mtd/nand/plat_nand.c
index ea8e123..7f2da69 100644
--- a/drivers/mtd/nand/plat_nand.c
+++ b/drivers/mtd/nand/plat_nand.c
@@ -148,18 +148,7 @@ static struct platform_driver plat_nand_driver = {
 	},
 };
 
-static int __init plat_nand_init(void)
-{
-	return platform_driver_register(&plat_nand_driver);
-}
-
-static void __exit plat_nand_exit(void)
-{
-	platform_driver_unregister(&plat_nand_driver);
-}
-
-module_init(plat_nand_init);
-module_exit(plat_nand_exit);
+module_platform_driver(plat_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Vitaly Wool");
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 9eb7f87..90d6016 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1311,17 +1311,7 @@ static struct platform_driver pxa3xx_nand_driver = {
 	.resume		= pxa3xx_nand_resume,
 };
 
-static int __init pxa3xx_nand_init(void)
-{
-	return platform_driver_register(&pxa3xx_nand_driver);
-}
-module_init(pxa3xx_nand_init);
-
-static void __exit pxa3xx_nand_exit(void)
-{
-	platform_driver_unregister(&pxa3xx_nand_driver);
-}
-module_exit(pxa3xx_nand_exit);
+module_platform_driver(pxa3xx_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("PXA3xx NAND controller driver");
diff --git a/drivers/mtd/nand/sharpsl.c b/drivers/mtd/nand/sharpsl.c
index 619d2a5..b175c0f 100644
--- a/drivers/mtd/nand/sharpsl.c
+++ b/drivers/mtd/nand/sharpsl.c
@@ -230,17 +230,7 @@ static struct platform_driver sharpsl_nand_driver = {
 	.remove		= __devexit_p(sharpsl_nand_remove),
 };
 
-static int __init sharpsl_nand_init(void)
-{
-	return platform_driver_register(&sharpsl_nand_driver);
-}
-module_init(sharpsl_nand_init);
-
-static void __exit sharpsl_nand_exit(void)
-{
-	platform_driver_unregister(&sharpsl_nand_driver);
-}
-module_exit(sharpsl_nand_exit);
+module_platform_driver(sharpsl_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Richard Purdie <rpurdie@rpsys.net>");
diff --git a/drivers/mtd/nand/socrates_nand.c b/drivers/mtd/nand/socrates_nand.c
index 0fb24f9..e02b08b 100644
--- a/drivers/mtd/nand/socrates_nand.c
+++ b/drivers/mtd/nand/socrates_nand.c
@@ -273,18 +273,7 @@ static struct platform_driver socrates_nand_driver = {
 	.remove		= __devexit_p(socrates_nand_remove),
 };
 
-static int __init socrates_nand_init(void)
-{
-	return platform_driver_register(&socrates_nand_driver);
-}
-
-static void __exit socrates_nand_exit(void)
-{
-	platform_driver_unregister(&socrates_nand_driver);
-}
-
-module_init(socrates_nand_init);
-module_exit(socrates_nand_exit);
+module_platform_driver(socrates_nand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Ilya Yanok");
diff --git a/drivers/mtd/nand/tmio_nand.c b/drivers/mtd/nand/tmio_nand.c
index beebd95..6caa0cd 100644
--- a/drivers/mtd/nand/tmio_nand.c
+++ b/drivers/mtd/nand/tmio_nand.c
@@ -533,18 +533,7 @@ static struct platform_driver tmio_driver = {
 	.resume		= tmio_resume,
 };
 
-static int __init tmio_init(void)
-{
-	return platform_driver_register(&tmio_driver);
-}
-
-static void __exit tmio_exit(void)
-{
-	platform_driver_unregister(&tmio_driver);
-}
-
-module_init(tmio_init);
-module_exit(tmio_exit);
+module_platform_driver(tmio_driver);
 
 MODULE_LICENSE("GPL v2");
 MODULE_AUTHOR("Ian Molton, Dirk Opfer, Chris Humbert, Dmitry Baryshkov");
diff --git a/drivers/mtd/onenand/generic.c b/drivers/mtd/onenand/generic.c
index 7813095..0ccd5bf 100644
--- a/drivers/mtd/onenand/generic.c
+++ b/drivers/mtd/onenand/generic.c
@@ -115,21 +115,9 @@ static struct platform_driver generic_onenand_driver = {
 	.remove		= __devexit_p(generic_onenand_remove),
 };
 
-MODULE_ALIAS("platform:" DRIVER_NAME);
-
-static int __init generic_onenand_init(void)
-{
-	return platform_driver_register(&generic_onenand_driver);
-}
-
-static void __exit generic_onenand_exit(void)
-{
-	platform_driver_unregister(&generic_onenand_driver);
-}
-
-module_init(generic_onenand_init);
-module_exit(generic_onenand_exit);
+module_platform_driver(generic_onenand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");
 MODULE_DESCRIPTION("Glue layer for OneNAND flash on generic boards");
+MODULE_ALIAS("platform:" DRIVER_NAME);
diff --git a/drivers/mtd/onenand/samsung.c b/drivers/mtd/onenand/samsung.c
index 5474547..fa1ee43 100644
--- a/drivers/mtd/onenand/samsung.c
+++ b/drivers/mtd/onenand/samsung.c
@@ -1133,18 +1133,7 @@ static struct platform_driver s3c_onenand_driver = {
 	.remove         = __devexit_p(s3c_onenand_remove),
 };
 
-static int __init s3c_onenand_init(void)
-{
-	return platform_driver_register(&s3c_onenand_driver);
-}
-
-static void __exit s3c_onenand_exit(void)
-{
-	platform_driver_unregister(&s3c_onenand_driver);
-}
-
-module_init(s3c_onenand_init);
-module_exit(s3c_onenand_exit);
+module_platform_driver(s3c_onenand_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");
-- 
1.7.5.4

^ permalink raw reply related	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-27 12:45 ` Axel Lin
@ 2011-11-27 18:20   ` Olof Johansson
  -1 siblings, 0 replies; 20+ messages in thread
From: Olof Johansson @ 2011-11-27 18:20 UTC (permalink / raw)
  To: Axel Lin
  Cc: linux-kernel, David Woodhouse, Jiandong Zheng, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Jamie Iles, Kyungmin Park,
	Haojian Zhuang, Artem Bityutskiy, Ralf Baechle, Brian Norris,
	Lars-Peter Clausen, Manuel Lauss, Vimal Singh, Andres Salomon,
	Sukumar Ghorai, linux-mtd

On Sun, Nov 27, 2011 at 4:45 AM, Axel Lin <axel.lin@gmail.com> wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jiandong Zheng <jdzheng@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wan ZongShun <mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Jamie Iles <jamie@jamieiles.com>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
> Cc: Vimal Singh <vimal.newwork@gmail.com>
> Cc: Andres Salomon <dilinger@queued.net>
> Cc: Sukumar Ghorai <s-ghorai@ti.com>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>

Acked-by: Olof Johansson <olof@lixom.net>   (pasemi portions)


-Olof

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-27 18:20   ` Olof Johansson
  0 siblings, 0 replies; 20+ messages in thread
From: Olof Johansson @ 2011-11-27 18:20 UTC (permalink / raw)
  To: Axel Lin
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Haojian Zhuang, Kyungmin Park, Vimal Singh,
	Ralf Baechle, Jiandong Zheng, Andres Salomon, Jamie Iles,
	Brian Norris, David Woodhouse, linux-mtd, Manuel Lauss

On Sun, Nov 27, 2011 at 4:45 AM, Axel Lin <axel.lin@gmail.com> wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jiandong Zheng <jdzheng@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wan ZongShun <mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Jamie Iles <jamie@jamieiles.com>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
> Cc: Vimal Singh <vimal.newwork@gmail.com>
> Cc: Andres Salomon <dilinger@queued.net>
> Cc: Sukumar Ghorai <s-ghorai@ti.com>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>

Acked-by: Olof Johansson <olof@lixom.net>   (pasemi portions)


-Olof

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-27 18:20   ` Olof Johansson
@ 2011-11-28  0:23     ` Kyungmin Park
  -1 siblings, 0 replies; 20+ messages in thread
From: Kyungmin Park @ 2011-11-28  0:23 UTC (permalink / raw)
  To: Olof Johansson
  Cc: Axel Lin, linux-kernel, David Woodhouse, Jiandong Zheng,
	Scott Branden, Wan ZongShun, Dmitry Eremin-Solenikov, Jamie Iles,
	Haojian Zhuang, Artem Bityutskiy, Ralf Baechle, Brian Norris,
	Lars-Peter Clausen, Manuel Lauss, Vimal Singh, Andres Salomon,
	Sukumar Ghorai, linux-mtd

For OneNAND parts

Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
On 11/28/11, Olof Johansson <olof@lixom.net> wrote:
> On Sun, Nov 27, 2011 at 4:45 AM, Axel Lin <axel.lin@gmail.com> wrote:
>> This patch converts the drivers in drivers/mtd/* to use the
>> module_platform_driver() macro which makes the code smaller and a bit
>> simpler.
>>
>> Cc: David Woodhouse <dwmw2@infradead.org>
>> Cc: Jiandong Zheng <jdzheng@broadcom.com>
>> Cc: Scott Branden <sbranden@broadcom.com>
>> Cc: Wan ZongShun <mcuos.com@gmail.com>
>> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
>> Cc: Jamie Iles <jamie@jamieiles.com>
>> Cc: Olof Johansson <olof@lixom.net>
>> Cc: Kyungmin Park <kyungmin.park@samsung.com>
>> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
>> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
>> Cc: Ralf Baechle <ralf@linux-mips.org>
>> Cc: Brian Norris <computersforpeace@gmail.com>
>> Cc: Lars-Peter Clausen <lars@metafoo.de>
>> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
>> Cc: Vimal Singh <vimal.newwork@gmail.com>
>> Cc: Andres Salomon <dilinger@queued.net>
>> Cc: Sukumar Ghorai <s-ghorai@ti.com>
>> Signed-off-by: Axel Lin <axel.lin@gmail.com>
>
> Acked-by: Olof Johansson <olof@lixom.net>   (pasemi portions)
>
>
> -Olof
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-28  0:23     ` Kyungmin Park
  0 siblings, 0 replies; 20+ messages in thread
From: Kyungmin Park @ 2011-11-28  0:23 UTC (permalink / raw)
  To: Olof Johansson
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Haojian Zhuang, Manuel Lauss, Vimal Singh,
	Ralf Baechle, Jiandong Zheng, Andres Salomon, Jamie Iles,
	Axel Lin, Brian Norris, David Woodhouse, linux-mtd

For OneNAND parts

Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
On 11/28/11, Olof Johansson <olof@lixom.net> wrote:
> On Sun, Nov 27, 2011 at 4:45 AM, Axel Lin <axel.lin@gmail.com> wrote:
>> This patch converts the drivers in drivers/mtd/* to use the
>> module_platform_driver() macro which makes the code smaller and a bit
>> simpler.
>>
>> Cc: David Woodhouse <dwmw2@infradead.org>
>> Cc: Jiandong Zheng <jdzheng@broadcom.com>
>> Cc: Scott Branden <sbranden@broadcom.com>
>> Cc: Wan ZongShun <mcuos.com@gmail.com>
>> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
>> Cc: Jamie Iles <jamie@jamieiles.com>
>> Cc: Olof Johansson <olof@lixom.net>
>> Cc: Kyungmin Park <kyungmin.park@samsung.com>
>> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
>> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
>> Cc: Ralf Baechle <ralf@linux-mips.org>
>> Cc: Brian Norris <computersforpeace@gmail.com>
>> Cc: Lars-Peter Clausen <lars@metafoo.de>
>> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
>> Cc: Vimal Singh <vimal.newwork@gmail.com>
>> Cc: Andres Salomon <dilinger@queued.net>
>> Cc: Sukumar Ghorai <s-ghorai@ti.com>
>> Signed-off-by: Axel Lin <axel.lin@gmail.com>
>
> Acked-by: Olof Johansson <olof@lixom.net>   (pasemi portions)
>
>
> -Olof
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-27 12:45 ` Axel Lin
@ 2011-11-28  0:57   ` Haojian Zhuang
  -1 siblings, 0 replies; 20+ messages in thread
From: Haojian Zhuang @ 2011-11-28  0:57 UTC (permalink / raw)
  To: Axel Lin
  Cc: linux-kernel, David Woodhouse, Jiandong Zheng, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Jamie Iles,
	Olof Johansson, Kyungmin Park, Artem Bityutskiy, Ralf Baechle,
	Brian Norris, Lars-Peter Clausen, Manuel Lauss, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

On Sun, Nov 27, 2011 at 8:45 PM, Axel Lin <axel.lin@gmail.com> wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jiandong Zheng <jdzheng@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wan ZongShun <mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Jamie Iles <jamie@jamieiles.com>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
> Cc: Vimal Singh <vimal.newwork@gmail.com>
> Cc: Andres Salomon <dilinger@queued.net>
> Cc: Sukumar Ghorai <s-ghorai@ti.com>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
>
For pxa nand
Acked by: Haojian Zhuang <haojian.zhuang@gmail.com>

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-28  0:57   ` Haojian Zhuang
  0 siblings, 0 replies; 20+ messages in thread
From: Haojian Zhuang @ 2011-11-28  0:57 UTC (permalink / raw)
  To: Axel Lin
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Ralf Baechle, Kyungmin Park, Vimal Singh,
	Jiandong Zheng, Andres Salomon, Olof Johansson, Jamie Iles,
	Brian Norris, David Woodhouse, linux-mtd, Manuel Lauss

On Sun, Nov 27, 2011 at 8:45 PM, Axel Lin <axel.lin@gmail.com> wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jiandong Zheng <jdzheng@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wan ZongShun <mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Jamie Iles <jamie@jamieiles.com>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
> Cc: Vimal Singh <vimal.newwork@gmail.com>
> Cc: Andres Salomon <dilinger@queued.net>
> Cc: Sukumar Ghorai <s-ghorai@ti.com>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
>
For pxa nand
Acked by: Haojian Zhuang <haojian.zhuang@gmail.com>

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-27 12:45 ` Axel Lin
@ 2011-11-28  2:34   ` Wan ZongShun
  -1 siblings, 0 replies; 20+ messages in thread
From: Wan ZongShun @ 2011-11-28  2:34 UTC (permalink / raw)
  To: Axel Lin
  Cc: linux-kernel, David Woodhouse, Jiandong Zheng, Scott Branden,
	Dmitry Eremin-Solenikov, Jamie Iles, Olof Johansson,
	Kyungmin Park, Haojian Zhuang, Artem Bityutskiy, Ralf Baechle,
	Brian Norris, Lars-Peter Clausen, Manuel Lauss, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

2011/11/27 Axel Lin <axel.lin@gmail.com>:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jiandong Zheng <jdzheng@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wan ZongShun <mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Jamie Iles <jamie@jamieiles.com>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
> Cc: Vimal Singh <vimal.newwork@gmail.com>
> Cc: Andres Salomon <dilinger@queued.net>
> Cc: Sukumar Ghorai <s-ghorai@ti.com>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
> ---
>  drivers/mtd/maps/bcm963xx-flash.c   |   13 +------------
>  drivers/mtd/maps/bfin-async-flash.c |   12 +-----------
>  drivers/mtd/maps/gpio-addr-flash.c  |   12 +-----------
>  drivers/mtd/maps/ixp2000.c          |   12 +-----------
>  drivers/mtd/maps/ixp4xx.c           |   14 +-------------
>  drivers/mtd/maps/latch-addr-flash.c |   12 +-----------
>  drivers/mtd/maps/physmap_of.c       |   13 +------------
>  drivers/mtd/maps/pxa2xx-flash.c     |   13 +------------
>  drivers/mtd/maps/rbtx4939-flash.c   |   13 +------------
>  drivers/mtd/maps/sa1100-flash.c     |   13 +------------
>  drivers/mtd/maps/sun_uflash.c       |   13 +------------
>  drivers/mtd/nand/ams-delta.c        |   12 +-----------
>  drivers/mtd/nand/au1550nd.c         |   12 +-----------
>  drivers/mtd/nand/bcm_umi_nand.c     |   13 +------------
>  drivers/mtd/nand/fsl_elbc_nand.c    |   13 +------------
>  drivers/mtd/nand/fsl_upm.c          |   12 +-----------
>  drivers/mtd/nand/jz4740_nand.c      |   12 +-----------
>  drivers/mtd/nand/mpc5121_nfc.c      |   14 +-------------
>  drivers/mtd/nand/ndfc.c             |   13 +------------
>  drivers/mtd/nand/nomadik_nand.c     |   14 +-------------
>  drivers/mtd/nand/nuc900_nand.c      |   13 +------------
>  drivers/mtd/nand/omap2.c            |   15 +--------------
>  drivers/mtd/nand/pasemi_nand.c      |   12 +-----------
>  drivers/mtd/nand/plat_nand.c        |   13 +------------
>  drivers/mtd/nand/pxa3xx_nand.c      |   12 +-----------
>  drivers/mtd/nand/sharpsl.c          |   12 +-----------
>  drivers/mtd/nand/socrates_nand.c    |   13 +------------
>  drivers/mtd/nand/tmio_nand.c        |   13 +------------
>  drivers/mtd/onenand/generic.c       |   16 ++--------------
>  drivers/mtd/onenand/samsung.c       |   13 +------------
>  30 files changed, 31 insertions(+), 356 deletions(-)
>
> diff --git a/drivers/mtd/maps/bcm963xx-flash.c b/drivers/mtd/maps/bcm963xx-flash.c
> index 736ca10..56927f5 100644
> --- a/drivers/mtd/maps/bcm963xx-flash.c
> +++ b/drivers/mtd/maps/bcm963xx-flash.c
> @@ -257,18 +257,7 @@ static struct platform_driver bcm63xx_mtd_dev = {
>        },
>  };
>
> -static int __init bcm963xx_mtd_init(void)
> -{
> -       return platform_driver_register(&bcm63xx_mtd_dev);
> -}
> -
> -static void __exit bcm963xx_mtd_exit(void)
> -{
> -       platform_driver_unregister(&bcm63xx_mtd_dev);
> -}
> -
> -module_init(bcm963xx_mtd_init);
> -module_exit(bcm963xx_mtd_exit);
> +module_platform_driver(bcm63xx_mtd_dev);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("Broadcom BCM63xx MTD driver for CFE and RedBoot");
> diff --git a/drivers/mtd/maps/bfin-async-flash.c b/drivers/mtd/maps/bfin-async-flash.c
> index 6d6b2b5..650126c 100644
> --- a/drivers/mtd/maps/bfin-async-flash.c
> +++ b/drivers/mtd/maps/bfin-async-flash.c
> @@ -190,17 +190,7 @@ static struct platform_driver bfin_flash_driver = {
>        },
>  };
>
> -static int __init bfin_flash_init(void)
> -{
> -       return platform_driver_register(&bfin_flash_driver);
> -}
> -module_init(bfin_flash_init);
> -
> -static void __exit bfin_flash_exit(void)
> -{
> -       platform_driver_unregister(&bfin_flash_driver);
> -}
> -module_exit(bfin_flash_exit);
> +module_platform_driver(bfin_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("MTD map driver for Blackfins with flash/ethernet on same async bank");
> diff --git a/drivers/mtd/maps/gpio-addr-flash.c b/drivers/mtd/maps/gpio-addr-flash.c
> index 1ec66f0..33cce89 100644
> --- a/drivers/mtd/maps/gpio-addr-flash.c
> +++ b/drivers/mtd/maps/gpio-addr-flash.c
> @@ -279,17 +279,7 @@ static struct platform_driver gpio_flash_driver = {
>        },
>  };
>
> -static int __init gpio_flash_init(void)
> -{
> -       return platform_driver_register(&gpio_flash_driver);
> -}
> -module_init(gpio_flash_init);
> -
> -static void __exit gpio_flash_exit(void)
> -{
> -       platform_driver_unregister(&gpio_flash_driver);
> -}
> -module_exit(gpio_flash_exit);
> +module_platform_driver(gpio_flash_driver);
>
>  MODULE_AUTHOR("Mike Frysinger <vapier@gentoo.org>");
>  MODULE_DESCRIPTION("MTD map driver for flashes addressed physically and with gpios");
> diff --git a/drivers/mtd/maps/ixp2000.c b/drivers/mtd/maps/ixp2000.c
> index 437fcd2..fc7d4d0 100644
> --- a/drivers/mtd/maps/ixp2000.c
> +++ b/drivers/mtd/maps/ixp2000.c
> @@ -246,18 +246,8 @@ static struct platform_driver ixp2000_flash_driver = {
>        },
>  };
>
> -static int __init ixp2000_flash_init(void)
> -{
> -       return platform_driver_register(&ixp2000_flash_driver);
> -}
> -
> -static void __exit ixp2000_flash_exit(void)
> -{
> -       platform_driver_unregister(&ixp2000_flash_driver);
> -}
> +module_platform_driver(ixp2000_flash_driver);
>
> -module_init(ixp2000_flash_init);
> -module_exit(ixp2000_flash_exit);
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Deepak Saxena <dsaxena@plexity.net>");
>  MODULE_ALIAS("platform:IXP2000-Flash");
> diff --git a/drivers/mtd/maps/ixp4xx.c b/drivers/mtd/maps/ixp4xx.c
> index 3040901..8b54101 100644
> --- a/drivers/mtd/maps/ixp4xx.c
> +++ b/drivers/mtd/maps/ixp4xx.c
> @@ -270,19 +270,7 @@ static struct platform_driver ixp4xx_flash_driver = {
>        },
>  };
>
> -static int __init ixp4xx_flash_init(void)
> -{
> -       return platform_driver_register(&ixp4xx_flash_driver);
> -}
> -
> -static void __exit ixp4xx_flash_exit(void)
> -{
> -       platform_driver_unregister(&ixp4xx_flash_driver);
> -}
> -
> -
> -module_init(ixp4xx_flash_init);
> -module_exit(ixp4xx_flash_exit);
> +module_platform_driver(ixp4xx_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("MTD map driver for Intel IXP4xx systems");
> diff --git a/drivers/mtd/maps/latch-addr-flash.c b/drivers/mtd/maps/latch-addr-flash.c
> index 119baa7..8fed58e 100644
> --- a/drivers/mtd/maps/latch-addr-flash.c
> +++ b/drivers/mtd/maps/latch-addr-flash.c
> @@ -223,17 +223,7 @@ static struct platform_driver latch_addr_flash_driver = {
>        },
>  };
>
> -static int __init latch_addr_flash_init(void)
> -{
> -       return platform_driver_register(&latch_addr_flash_driver);
> -}
> -module_init(latch_addr_flash_init);
> -
> -static void __exit latch_addr_flash_exit(void)
> -{
> -       platform_driver_unregister(&latch_addr_flash_driver);
> -}
> -module_exit(latch_addr_flash_exit);
> +module_platform_driver(latch_addr_flash_driver);
>
>  MODULE_AUTHOR("David Griego <dgriego@mvista.com>");
>  MODULE_DESCRIPTION("MTD map driver for flashes addressed physically with upper "
> diff --git a/drivers/mtd/maps/physmap_of.c b/drivers/mtd/maps/physmap_of.c
> index 7d65f9d..2e6fb68 100644
> --- a/drivers/mtd/maps/physmap_of.c
> +++ b/drivers/mtd/maps/physmap_of.c
> @@ -338,18 +338,7 @@ static struct platform_driver of_flash_driver = {
>        .remove         = of_flash_remove,
>  };
>
> -static int __init of_flash_init(void)
> -{
> -       return platform_driver_register(&of_flash_driver);
> -}
> -
> -static void __exit of_flash_exit(void)
> -{
> -       platform_driver_unregister(&of_flash_driver);
> -}
> -
> -module_init(of_flash_init);
> -module_exit(of_flash_exit);
> +module_platform_driver(of_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Vitaly Wool <vwool@ru.mvista.com>");
> diff --git a/drivers/mtd/maps/pxa2xx-flash.c b/drivers/mtd/maps/pxa2xx-flash.c
> index 2a25b67..274e399 100644
> --- a/drivers/mtd/maps/pxa2xx-flash.c
> +++ b/drivers/mtd/maps/pxa2xx-flash.c
> @@ -142,18 +142,7 @@ static struct platform_driver pxa2xx_flash_driver = {
>        .shutdown       = pxa2xx_flash_shutdown,
>  };
>
> -static int __init init_pxa2xx_flash(void)
> -{
> -       return platform_driver_register(&pxa2xx_flash_driver);
> -}
> -
> -static void __exit cleanup_pxa2xx_flash(void)
> -{
> -       platform_driver_unregister(&pxa2xx_flash_driver);
> -}
> -
> -module_init(init_pxa2xx_flash);
> -module_exit(cleanup_pxa2xx_flash);
> +module_platform_driver(pxa2xx_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Nicolas Pitre <nico@fluxnic.net>");
> diff --git a/drivers/mtd/maps/rbtx4939-flash.c b/drivers/mtd/maps/rbtx4939-flash.c
> index 0237f19..bb7d204 100644
> --- a/drivers/mtd/maps/rbtx4939-flash.c
> +++ b/drivers/mtd/maps/rbtx4939-flash.c
> @@ -137,18 +137,7 @@ static struct platform_driver rbtx4939_flash_driver = {
>        },
>  };
>
> -static int __init rbtx4939_flash_init(void)
> -{
> -       return platform_driver_register(&rbtx4939_flash_driver);
> -}
> -
> -static void __exit rbtx4939_flash_exit(void)
> -{
> -       platform_driver_unregister(&rbtx4939_flash_driver);
> -}
> -
> -module_init(rbtx4939_flash_init);
> -module_exit(rbtx4939_flash_exit);
> +module_platform_driver(rbtx4939_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("RBTX4939 MTD map driver");
> diff --git a/drivers/mtd/maps/sa1100-flash.c b/drivers/mtd/maps/sa1100-flash.c
> index fa9c0a9..ac3a290 100644
> --- a/drivers/mtd/maps/sa1100-flash.c
> +++ b/drivers/mtd/maps/sa1100-flash.c
> @@ -394,18 +394,7 @@ static struct platform_driver sa1100_mtd_driver = {
>        },
>  };
>
> -static int __init sa1100_mtd_init(void)
> -{
> -       return platform_driver_register(&sa1100_mtd_driver);
> -}
> -
> -static void __exit sa1100_mtd_exit(void)
> -{
> -       platform_driver_unregister(&sa1100_mtd_driver);
> -}
> -
> -module_init(sa1100_mtd_init);
> -module_exit(sa1100_mtd_exit);
> +module_platform_driver(sa1100_mtd_driver);
>
>  MODULE_AUTHOR("Nicolas Pitre");
>  MODULE_DESCRIPTION("SA1100 CFI map driver");
> diff --git a/drivers/mtd/maps/sun_uflash.c b/drivers/mtd/maps/sun_uflash.c
> index 2d66234..175e537 100644
> --- a/drivers/mtd/maps/sun_uflash.c
> +++ b/drivers/mtd/maps/sun_uflash.c
> @@ -158,15 +158,4 @@ static struct platform_driver uflash_driver = {
>        .remove         = __devexit_p(uflash_remove),
>  };
>
> -static int __init uflash_init(void)
> -{
> -       return platform_driver_register(&uflash_driver);
> -}
> -
> -static void __exit uflash_exit(void)
> -{
> -       platform_driver_unregister(&uflash_driver);
> -}
> -
> -module_init(uflash_init);
> -module_exit(uflash_exit);
> +module_platform_driver(uflash_driver);
> diff --git a/drivers/mtd/nand/ams-delta.c b/drivers/mtd/nand/ams-delta.c
> index 9e6b498..3197e97 100644
> --- a/drivers/mtd/nand/ams-delta.c
> +++ b/drivers/mtd/nand/ams-delta.c
> @@ -280,17 +280,7 @@ static struct platform_driver ams_delta_nand_driver = {
>        },
>  };
>
> -static int __init ams_delta_nand_init(void)
> -{
> -       return platform_driver_register(&ams_delta_nand_driver);
> -}
> -module_init(ams_delta_nand_init);
> -
> -static void __exit ams_delta_nand_exit(void)
> -{
> -       platform_driver_unregister(&ams_delta_nand_driver);
> -}
> -module_exit(ams_delta_nand_exit);
> +module_platform_driver(ams_delta_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Jonathan McDowell <noodles@earth.li>");
> diff --git a/drivers/mtd/nand/au1550nd.c b/drivers/mtd/nand/au1550nd.c
> index 77fb4e6..73abbc3 100644
> --- a/drivers/mtd/nand/au1550nd.c
> +++ b/drivers/mtd/nand/au1550nd.c
> @@ -560,17 +560,7 @@ static struct platform_driver au1550nd_driver = {
>        .remove         = __devexit_p(au1550nd_remove),
>  };
>
> -static int __init au1550nd_load(void)
> -{
> -       return platform_driver_register(&au1550nd_driver);
> -};
> -
> -static void __exit au1550nd_exit(void)
> -{
> -       platform_driver_unregister(&au1550nd_driver);
> -};
> -module_init(au1550nd_load);
> -module_exit(au1550nd_exit);
> +module_platform_driver(au1550nd_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Embedded Edge, LLC");
> diff --git a/drivers/mtd/nand/bcm_umi_nand.c b/drivers/mtd/nand/bcm_umi_nand.c
> index 46b58d6..50387fd 100644
> --- a/drivers/mtd/nand/bcm_umi_nand.c
> +++ b/drivers/mtd/nand/bcm_umi_nand.c
> @@ -546,18 +546,7 @@ static struct platform_driver nand_driver = {
>        .resume = bcm_umi_nand_resume,
>  };
>
> -static int __init nand_init(void)
> -{
> -       return platform_driver_register(&nand_driver);
> -}
> -
> -static void __exit nand_exit(void)
> -{
> -       platform_driver_unregister(&nand_driver);
> -}
> -
> -module_init(nand_init);
> -module_exit(nand_exit);
> +module_platform_driver(nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Broadcom");
> diff --git a/drivers/mtd/nand/fsl_elbc_nand.c b/drivers/mtd/nand/fsl_elbc_nand.c
> index eedd8ee..f2f6e87 100644
> --- a/drivers/mtd/nand/fsl_elbc_nand.c
> +++ b/drivers/mtd/nand/fsl_elbc_nand.c
> @@ -971,18 +971,7 @@ static struct platform_driver fsl_elbc_nand_driver = {
>        .remove = fsl_elbc_nand_remove,
>  };
>
> -static int __init fsl_elbc_nand_init(void)
> -{
> -       return platform_driver_register(&fsl_elbc_nand_driver);
> -}
> -
> -static void __exit fsl_elbc_nand_exit(void)
> -{
> -       platform_driver_unregister(&fsl_elbc_nand_driver);
> -}
> -
> -module_init(fsl_elbc_nand_init);
> -module_exit(fsl_elbc_nand_exit);
> +module_platform_driver(fsl_elbc_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Freescale");
> diff --git a/drivers/mtd/nand/fsl_upm.c b/drivers/mtd/nand/fsl_upm.c
> index b4f3cc9..45df542 100644
> --- a/drivers/mtd/nand/fsl_upm.c
> +++ b/drivers/mtd/nand/fsl_upm.c
> @@ -353,17 +353,7 @@ static struct platform_driver of_fun_driver = {
>        .remove         = __devexit_p(fun_remove),
>  };
>
> -static int __init fun_module_init(void)
> -{
> -       return platform_driver_register(&of_fun_driver);
> -}
> -module_init(fun_module_init);
> -
> -static void __exit fun_module_exit(void)
> -{
> -       platform_driver_unregister(&of_fun_driver);
> -}
> -module_exit(fun_module_exit);
> +module_platform_driver(of_fun_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
> diff --git a/drivers/mtd/nand/jz4740_nand.c b/drivers/mtd/nand/jz4740_nand.c
> index e266407..ac3b9f2 100644
> --- a/drivers/mtd/nand/jz4740_nand.c
> +++ b/drivers/mtd/nand/jz4740_nand.c
> @@ -423,17 +423,7 @@ static struct platform_driver jz_nand_driver = {
>        },
>  };
>
> -static int __init jz_nand_init(void)
> -{
> -       return platform_driver_register(&jz_nand_driver);
> -}
> -module_init(jz_nand_init);
> -
> -static void __exit jz_nand_exit(void)
> -{
> -       platform_driver_unregister(&jz_nand_driver);
> -}
> -module_exit(jz_nand_exit);
> +module_platform_driver(jz_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Lars-Peter Clausen <lars@metafoo.de>");
> diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c
> index 5ede647..c240cf1 100644
> --- a/drivers/mtd/nand/mpc5121_nfc.c
> +++ b/drivers/mtd/nand/mpc5121_nfc.c
> @@ -879,19 +879,7 @@ static struct platform_driver mpc5121_nfc_driver = {
>        },
>  };
>
> -static int __init mpc5121_nfc_init(void)
> -{
> -       return platform_driver_register(&mpc5121_nfc_driver);
> -}
> -
> -module_init(mpc5121_nfc_init);
> -
> -static void __exit mpc5121_nfc_cleanup(void)
> -{
> -       platform_driver_unregister(&mpc5121_nfc_driver);
> -}
> -
> -module_exit(mpc5121_nfc_cleanup);
> +module_platform_driver(mpc5121_nfc_driver);
>
>  MODULE_AUTHOR("Freescale Semiconductor, Inc.");
>  MODULE_DESCRIPTION("MPC5121 NAND MTD driver");
> diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
> index f8aacf4..ec68854 100644
> --- a/drivers/mtd/nand/ndfc.c
> +++ b/drivers/mtd/nand/ndfc.c
> @@ -294,18 +294,7 @@ static struct platform_driver ndfc_driver = {
>        .remove = __devexit_p(ndfc_remove),
>  };
>
> -static int __init ndfc_nand_init(void)
> -{
> -       return platform_driver_register(&ndfc_driver);
> -}
> -
> -static void __exit ndfc_nand_exit(void)
> -{
> -       platform_driver_unregister(&ndfc_driver);
> -}
> -
> -module_init(ndfc_nand_init);
> -module_exit(ndfc_nand_exit);
> +module_platform_driver(ndfc_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Thomas Gleixner <tglx@linutronix.de>");
> diff --git a/drivers/mtd/nand/nomadik_nand.c b/drivers/mtd/nand/nomadik_nand.c
> index b463ecf..673dc6c 100644
> --- a/drivers/mtd/nand/nomadik_nand.c
> +++ b/drivers/mtd/nand/nomadik_nand.c
> @@ -228,19 +228,7 @@ static struct platform_driver nomadik_nand_driver = {
>        },
>  };
>
> -static int __init nand_nomadik_init(void)
> -{
> -       pr_info("Nomadik NAND driver\n");
> -       return platform_driver_register(&nomadik_nand_driver);
> -}
> -
> -static void __exit nand_nomadik_exit(void)
> -{
> -       platform_driver_unregister(&nomadik_nand_driver);
> -}
> -
> -module_init(nand_nomadik_init);
> -module_exit(nand_nomadik_exit);
> +module_platform_driver(nomadik_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("ST Microelectronics (sachin.verma@st.com)");
> diff --git a/drivers/mtd/nand/nuc900_nand.c b/drivers/mtd/nand/nuc900_nand.c
> index fa8faed..8febe46 100644
> --- a/drivers/mtd/nand/nuc900_nand.c
> +++ b/drivers/mtd/nand/nuc900_nand.c
> @@ -364,18 +364,7 @@ static struct platform_driver nuc900_nand_driver = {
>        },
>  };
>
> -static int __init nuc900_nand_init(void)
> -{
> -       return platform_driver_register(&nuc900_nand_driver);
> -}
> -
> -static void __exit nuc900_nand_exit(void)
> -{
> -       platform_driver_unregister(&nuc900_nand_driver);
> -}
> -
> -module_init(nuc900_nand_init);
> -module_exit(nuc900_nand_exit);
> +module_platform_driver(nuc900_nand_driver);
>
>  MODULE_AUTHOR("Wan ZongShun <mcuos.com@gmail.com>");
>  MODULE_DESCRIPTION("w90p910/NUC9xx nand driver!");
> diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c
> index f745f00..b3a883e 100644
> --- a/drivers/mtd/nand/omap2.c
> +++ b/drivers/mtd/nand/omap2.c
> @@ -1145,20 +1145,7 @@ static struct platform_driver omap_nand_driver = {
>        },
>  };
>
> -static int __init omap_nand_init(void)
> -{
> -       pr_info("%s driver initializing\n", DRIVER_NAME);
> -
> -       return platform_driver_register(&omap_nand_driver);
> -}
> -
> -static void __exit omap_nand_exit(void)
> -{
> -       platform_driver_unregister(&omap_nand_driver);
> -}
> -
> -module_init(omap_nand_init);
> -module_exit(omap_nand_exit);
> +module_platform_driver(omap_nand_driver);
>
>  MODULE_ALIAS("platform:" DRIVER_NAME);
>  MODULE_LICENSE("GPL");
> diff --git a/drivers/mtd/nand/pasemi_nand.c b/drivers/mtd/nand/pasemi_nand.c
> index a97264e..974dbf8 100644
> --- a/drivers/mtd/nand/pasemi_nand.c
> +++ b/drivers/mtd/nand/pasemi_nand.c
> @@ -230,17 +230,7 @@ static struct platform_driver pasemi_nand_driver =
>        .remove         = pasemi_nand_remove,
>  };
>
> -static int __init pasemi_nand_init(void)
> -{
> -       return platform_driver_register(&pasemi_nand_driver);
> -}
> -module_init(pasemi_nand_init);
> -
> -static void __exit pasemi_nand_exit(void)
> -{
> -       platform_driver_unregister(&pasemi_nand_driver);
> -}
> -module_exit(pasemi_nand_exit);
> +module_platform_driver(pasemi_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Egor Martovetsky <egor@pasemi.com>");
> diff --git a/drivers/mtd/nand/plat_nand.c b/drivers/mtd/nand/plat_nand.c
> index ea8e123..7f2da69 100644
> --- a/drivers/mtd/nand/plat_nand.c
> +++ b/drivers/mtd/nand/plat_nand.c
> @@ -148,18 +148,7 @@ static struct platform_driver plat_nand_driver = {
>        },
>  };
>
> -static int __init plat_nand_init(void)
> -{
> -       return platform_driver_register(&plat_nand_driver);
> -}
> -
> -static void __exit plat_nand_exit(void)
> -{
> -       platform_driver_unregister(&plat_nand_driver);
> -}
> -
> -module_init(plat_nand_init);
> -module_exit(plat_nand_exit);
> +module_platform_driver(plat_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Vitaly Wool");
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 9eb7f87..90d6016 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -1311,17 +1311,7 @@ static struct platform_driver pxa3xx_nand_driver = {
>        .resume         = pxa3xx_nand_resume,
>  };
>
> -static int __init pxa3xx_nand_init(void)
> -{
> -       return platform_driver_register(&pxa3xx_nand_driver);
> -}
> -module_init(pxa3xx_nand_init);
> -
> -static void __exit pxa3xx_nand_exit(void)
> -{
> -       platform_driver_unregister(&pxa3xx_nand_driver);
> -}
> -module_exit(pxa3xx_nand_exit);
> +module_platform_driver(pxa3xx_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("PXA3xx NAND controller driver");
> diff --git a/drivers/mtd/nand/sharpsl.c b/drivers/mtd/nand/sharpsl.c
> index 619d2a5..b175c0f 100644
> --- a/drivers/mtd/nand/sharpsl.c
> +++ b/drivers/mtd/nand/sharpsl.c
> @@ -230,17 +230,7 @@ static struct platform_driver sharpsl_nand_driver = {
>        .remove         = __devexit_p(sharpsl_nand_remove),
>  };
>
> -static int __init sharpsl_nand_init(void)
> -{
> -       return platform_driver_register(&sharpsl_nand_driver);
> -}
> -module_init(sharpsl_nand_init);
> -
> -static void __exit sharpsl_nand_exit(void)
> -{
> -       platform_driver_unregister(&sharpsl_nand_driver);
> -}
> -module_exit(sharpsl_nand_exit);
> +module_platform_driver(sharpsl_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Richard Purdie <rpurdie@rpsys.net>");
> diff --git a/drivers/mtd/nand/socrates_nand.c b/drivers/mtd/nand/socrates_nand.c
> index 0fb24f9..e02b08b 100644
> --- a/drivers/mtd/nand/socrates_nand.c
> +++ b/drivers/mtd/nand/socrates_nand.c
> @@ -273,18 +273,7 @@ static struct platform_driver socrates_nand_driver = {
>        .remove         = __devexit_p(socrates_nand_remove),
>  };
>
> -static int __init socrates_nand_init(void)
> -{
> -       return platform_driver_register(&socrates_nand_driver);
> -}
> -
> -static void __exit socrates_nand_exit(void)
> -{
> -       platform_driver_unregister(&socrates_nand_driver);
> -}
> -
> -module_init(socrates_nand_init);
> -module_exit(socrates_nand_exit);
> +module_platform_driver(socrates_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Ilya Yanok");
> diff --git a/drivers/mtd/nand/tmio_nand.c b/drivers/mtd/nand/tmio_nand.c
> index beebd95..6caa0cd 100644
> --- a/drivers/mtd/nand/tmio_nand.c
> +++ b/drivers/mtd/nand/tmio_nand.c
> @@ -533,18 +533,7 @@ static struct platform_driver tmio_driver = {
>        .resume         = tmio_resume,
>  };
>
> -static int __init tmio_init(void)
> -{
> -       return platform_driver_register(&tmio_driver);
> -}
> -
> -static void __exit tmio_exit(void)
> -{
> -       platform_driver_unregister(&tmio_driver);
> -}
> -
> -module_init(tmio_init);
> -module_exit(tmio_exit);
> +module_platform_driver(tmio_driver);
>
>  MODULE_LICENSE("GPL v2");
>  MODULE_AUTHOR("Ian Molton, Dirk Opfer, Chris Humbert, Dmitry Baryshkov");
> diff --git a/drivers/mtd/onenand/generic.c b/drivers/mtd/onenand/generic.c
> index 7813095..0ccd5bf 100644
> --- a/drivers/mtd/onenand/generic.c
> +++ b/drivers/mtd/onenand/generic.c
> @@ -115,21 +115,9 @@ static struct platform_driver generic_onenand_driver = {
>        .remove         = __devexit_p(generic_onenand_remove),
>  };
>
> -MODULE_ALIAS("platform:" DRIVER_NAME);
> -
> -static int __init generic_onenand_init(void)
> -{
> -       return platform_driver_register(&generic_onenand_driver);
> -}
> -
> -static void __exit generic_onenand_exit(void)
> -{
> -       platform_driver_unregister(&generic_onenand_driver);
> -}
> -
> -module_init(generic_onenand_init);
> -module_exit(generic_onenand_exit);
> +module_platform_driver(generic_onenand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");
>  MODULE_DESCRIPTION("Glue layer for OneNAND flash on generic boards");
> +MODULE_ALIAS("platform:" DRIVER_NAME);
> diff --git a/drivers/mtd/onenand/samsung.c b/drivers/mtd/onenand/samsung.c
> index 5474547..fa1ee43 100644
> --- a/drivers/mtd/onenand/samsung.c
> +++ b/drivers/mtd/onenand/samsung.c
> @@ -1133,18 +1133,7 @@ static struct platform_driver s3c_onenand_driver = {
>        .remove         = __devexit_p(s3c_onenand_remove),
>  };
>
> -static int __init s3c_onenand_init(void)
> -{
> -       return platform_driver_register(&s3c_onenand_driver);
> -}
> -
> -static void __exit s3c_onenand_exit(void)
> -{
> -       platform_driver_unregister(&s3c_onenand_driver);
> -}
> -
> -module_init(s3c_onenand_init);
> -module_exit(s3c_onenand_exit);
> +module_platform_driver(s3c_onenand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");

Acked-by: Wan ZongShun <mcuos.com@gmail.com>

Thanks!

> --
> 1.7.5.4
>
>
>
>



-- 
Wan ZongShun.
www.mcuos.com

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-28  2:34   ` Wan ZongShun
  0 siblings, 0 replies; 20+ messages in thread
From: Wan ZongShun @ 2011-11-28  2:34 UTC (permalink / raw)
  To: Axel Lin
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Dmitry Eremin-Solenikov, Sukumar Ghorai, linux-kernel,
	Haojian Zhuang, Kyungmin Park, Vimal Singh, Ralf Baechle,
	Jiandong Zheng, Andres Salomon, Olof Johansson, Jamie Iles,
	Brian Norris, David Woodhouse, linux-mtd, Manuel Lauss

2011/11/27 Axel Lin <axel.lin@gmail.com>:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Jiandong Zheng <jdzheng@broadcom.com>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wan ZongShun <mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
> Cc: Jamie Iles <jamie@jamieiles.com>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Brian Norris <computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Manuel Lauss <manuel.lauss@googlemail.com>
> Cc: Vimal Singh <vimal.newwork@gmail.com>
> Cc: Andres Salomon <dilinger@queued.net>
> Cc: Sukumar Ghorai <s-ghorai@ti.com>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
> ---
>  drivers/mtd/maps/bcm963xx-flash.c   |   13 +------------
>  drivers/mtd/maps/bfin-async-flash.c |   12 +-----------
>  drivers/mtd/maps/gpio-addr-flash.c  |   12 +-----------
>  drivers/mtd/maps/ixp2000.c          |   12 +-----------
>  drivers/mtd/maps/ixp4xx.c           |   14 +-------------
>  drivers/mtd/maps/latch-addr-flash.c |   12 +-----------
>  drivers/mtd/maps/physmap_of.c       |   13 +------------
>  drivers/mtd/maps/pxa2xx-flash.c     |   13 +------------
>  drivers/mtd/maps/rbtx4939-flash.c   |   13 +------------
>  drivers/mtd/maps/sa1100-flash.c     |   13 +------------
>  drivers/mtd/maps/sun_uflash.c       |   13 +------------
>  drivers/mtd/nand/ams-delta.c        |   12 +-----------
>  drivers/mtd/nand/au1550nd.c         |   12 +-----------
>  drivers/mtd/nand/bcm_umi_nand.c     |   13 +------------
>  drivers/mtd/nand/fsl_elbc_nand.c    |   13 +------------
>  drivers/mtd/nand/fsl_upm.c          |   12 +-----------
>  drivers/mtd/nand/jz4740_nand.c      |   12 +-----------
>  drivers/mtd/nand/mpc5121_nfc.c      |   14 +-------------
>  drivers/mtd/nand/ndfc.c             |   13 +------------
>  drivers/mtd/nand/nomadik_nand.c     |   14 +-------------
>  drivers/mtd/nand/nuc900_nand.c      |   13 +------------
>  drivers/mtd/nand/omap2.c            |   15 +--------------
>  drivers/mtd/nand/pasemi_nand.c      |   12 +-----------
>  drivers/mtd/nand/plat_nand.c        |   13 +------------
>  drivers/mtd/nand/pxa3xx_nand.c      |   12 +-----------
>  drivers/mtd/nand/sharpsl.c          |   12 +-----------
>  drivers/mtd/nand/socrates_nand.c    |   13 +------------
>  drivers/mtd/nand/tmio_nand.c        |   13 +------------
>  drivers/mtd/onenand/generic.c       |   16 ++--------------
>  drivers/mtd/onenand/samsung.c       |   13 +------------
>  30 files changed, 31 insertions(+), 356 deletions(-)
>
> diff --git a/drivers/mtd/maps/bcm963xx-flash.c b/drivers/mtd/maps/bcm963xx-flash.c
> index 736ca10..56927f5 100644
> --- a/drivers/mtd/maps/bcm963xx-flash.c
> +++ b/drivers/mtd/maps/bcm963xx-flash.c
> @@ -257,18 +257,7 @@ static struct platform_driver bcm63xx_mtd_dev = {
>        },
>  };
>
> -static int __init bcm963xx_mtd_init(void)
> -{
> -       return platform_driver_register(&bcm63xx_mtd_dev);
> -}
> -
> -static void __exit bcm963xx_mtd_exit(void)
> -{
> -       platform_driver_unregister(&bcm63xx_mtd_dev);
> -}
> -
> -module_init(bcm963xx_mtd_init);
> -module_exit(bcm963xx_mtd_exit);
> +module_platform_driver(bcm63xx_mtd_dev);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("Broadcom BCM63xx MTD driver for CFE and RedBoot");
> diff --git a/drivers/mtd/maps/bfin-async-flash.c b/drivers/mtd/maps/bfin-async-flash.c
> index 6d6b2b5..650126c 100644
> --- a/drivers/mtd/maps/bfin-async-flash.c
> +++ b/drivers/mtd/maps/bfin-async-flash.c
> @@ -190,17 +190,7 @@ static struct platform_driver bfin_flash_driver = {
>        },
>  };
>
> -static int __init bfin_flash_init(void)
> -{
> -       return platform_driver_register(&bfin_flash_driver);
> -}
> -module_init(bfin_flash_init);
> -
> -static void __exit bfin_flash_exit(void)
> -{
> -       platform_driver_unregister(&bfin_flash_driver);
> -}
> -module_exit(bfin_flash_exit);
> +module_platform_driver(bfin_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("MTD map driver for Blackfins with flash/ethernet on same async bank");
> diff --git a/drivers/mtd/maps/gpio-addr-flash.c b/drivers/mtd/maps/gpio-addr-flash.c
> index 1ec66f0..33cce89 100644
> --- a/drivers/mtd/maps/gpio-addr-flash.c
> +++ b/drivers/mtd/maps/gpio-addr-flash.c
> @@ -279,17 +279,7 @@ static struct platform_driver gpio_flash_driver = {
>        },
>  };
>
> -static int __init gpio_flash_init(void)
> -{
> -       return platform_driver_register(&gpio_flash_driver);
> -}
> -module_init(gpio_flash_init);
> -
> -static void __exit gpio_flash_exit(void)
> -{
> -       platform_driver_unregister(&gpio_flash_driver);
> -}
> -module_exit(gpio_flash_exit);
> +module_platform_driver(gpio_flash_driver);
>
>  MODULE_AUTHOR("Mike Frysinger <vapier@gentoo.org>");
>  MODULE_DESCRIPTION("MTD map driver for flashes addressed physically and with gpios");
> diff --git a/drivers/mtd/maps/ixp2000.c b/drivers/mtd/maps/ixp2000.c
> index 437fcd2..fc7d4d0 100644
> --- a/drivers/mtd/maps/ixp2000.c
> +++ b/drivers/mtd/maps/ixp2000.c
> @@ -246,18 +246,8 @@ static struct platform_driver ixp2000_flash_driver = {
>        },
>  };
>
> -static int __init ixp2000_flash_init(void)
> -{
> -       return platform_driver_register(&ixp2000_flash_driver);
> -}
> -
> -static void __exit ixp2000_flash_exit(void)
> -{
> -       platform_driver_unregister(&ixp2000_flash_driver);
> -}
> +module_platform_driver(ixp2000_flash_driver);
>
> -module_init(ixp2000_flash_init);
> -module_exit(ixp2000_flash_exit);
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Deepak Saxena <dsaxena@plexity.net>");
>  MODULE_ALIAS("platform:IXP2000-Flash");
> diff --git a/drivers/mtd/maps/ixp4xx.c b/drivers/mtd/maps/ixp4xx.c
> index 3040901..8b54101 100644
> --- a/drivers/mtd/maps/ixp4xx.c
> +++ b/drivers/mtd/maps/ixp4xx.c
> @@ -270,19 +270,7 @@ static struct platform_driver ixp4xx_flash_driver = {
>        },
>  };
>
> -static int __init ixp4xx_flash_init(void)
> -{
> -       return platform_driver_register(&ixp4xx_flash_driver);
> -}
> -
> -static void __exit ixp4xx_flash_exit(void)
> -{
> -       platform_driver_unregister(&ixp4xx_flash_driver);
> -}
> -
> -
> -module_init(ixp4xx_flash_init);
> -module_exit(ixp4xx_flash_exit);
> +module_platform_driver(ixp4xx_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("MTD map driver for Intel IXP4xx systems");
> diff --git a/drivers/mtd/maps/latch-addr-flash.c b/drivers/mtd/maps/latch-addr-flash.c
> index 119baa7..8fed58e 100644
> --- a/drivers/mtd/maps/latch-addr-flash.c
> +++ b/drivers/mtd/maps/latch-addr-flash.c
> @@ -223,17 +223,7 @@ static struct platform_driver latch_addr_flash_driver = {
>        },
>  };
>
> -static int __init latch_addr_flash_init(void)
> -{
> -       return platform_driver_register(&latch_addr_flash_driver);
> -}
> -module_init(latch_addr_flash_init);
> -
> -static void __exit latch_addr_flash_exit(void)
> -{
> -       platform_driver_unregister(&latch_addr_flash_driver);
> -}
> -module_exit(latch_addr_flash_exit);
> +module_platform_driver(latch_addr_flash_driver);
>
>  MODULE_AUTHOR("David Griego <dgriego@mvista.com>");
>  MODULE_DESCRIPTION("MTD map driver for flashes addressed physically with upper "
> diff --git a/drivers/mtd/maps/physmap_of.c b/drivers/mtd/maps/physmap_of.c
> index 7d65f9d..2e6fb68 100644
> --- a/drivers/mtd/maps/physmap_of.c
> +++ b/drivers/mtd/maps/physmap_of.c
> @@ -338,18 +338,7 @@ static struct platform_driver of_flash_driver = {
>        .remove         = of_flash_remove,
>  };
>
> -static int __init of_flash_init(void)
> -{
> -       return platform_driver_register(&of_flash_driver);
> -}
> -
> -static void __exit of_flash_exit(void)
> -{
> -       platform_driver_unregister(&of_flash_driver);
> -}
> -
> -module_init(of_flash_init);
> -module_exit(of_flash_exit);
> +module_platform_driver(of_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Vitaly Wool <vwool@ru.mvista.com>");
> diff --git a/drivers/mtd/maps/pxa2xx-flash.c b/drivers/mtd/maps/pxa2xx-flash.c
> index 2a25b67..274e399 100644
> --- a/drivers/mtd/maps/pxa2xx-flash.c
> +++ b/drivers/mtd/maps/pxa2xx-flash.c
> @@ -142,18 +142,7 @@ static struct platform_driver pxa2xx_flash_driver = {
>        .shutdown       = pxa2xx_flash_shutdown,
>  };
>
> -static int __init init_pxa2xx_flash(void)
> -{
> -       return platform_driver_register(&pxa2xx_flash_driver);
> -}
> -
> -static void __exit cleanup_pxa2xx_flash(void)
> -{
> -       platform_driver_unregister(&pxa2xx_flash_driver);
> -}
> -
> -module_init(init_pxa2xx_flash);
> -module_exit(cleanup_pxa2xx_flash);
> +module_platform_driver(pxa2xx_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Nicolas Pitre <nico@fluxnic.net>");
> diff --git a/drivers/mtd/maps/rbtx4939-flash.c b/drivers/mtd/maps/rbtx4939-flash.c
> index 0237f19..bb7d204 100644
> --- a/drivers/mtd/maps/rbtx4939-flash.c
> +++ b/drivers/mtd/maps/rbtx4939-flash.c
> @@ -137,18 +137,7 @@ static struct platform_driver rbtx4939_flash_driver = {
>        },
>  };
>
> -static int __init rbtx4939_flash_init(void)
> -{
> -       return platform_driver_register(&rbtx4939_flash_driver);
> -}
> -
> -static void __exit rbtx4939_flash_exit(void)
> -{
> -       platform_driver_unregister(&rbtx4939_flash_driver);
> -}
> -
> -module_init(rbtx4939_flash_init);
> -module_exit(rbtx4939_flash_exit);
> +module_platform_driver(rbtx4939_flash_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("RBTX4939 MTD map driver");
> diff --git a/drivers/mtd/maps/sa1100-flash.c b/drivers/mtd/maps/sa1100-flash.c
> index fa9c0a9..ac3a290 100644
> --- a/drivers/mtd/maps/sa1100-flash.c
> +++ b/drivers/mtd/maps/sa1100-flash.c
> @@ -394,18 +394,7 @@ static struct platform_driver sa1100_mtd_driver = {
>        },
>  };
>
> -static int __init sa1100_mtd_init(void)
> -{
> -       return platform_driver_register(&sa1100_mtd_driver);
> -}
> -
> -static void __exit sa1100_mtd_exit(void)
> -{
> -       platform_driver_unregister(&sa1100_mtd_driver);
> -}
> -
> -module_init(sa1100_mtd_init);
> -module_exit(sa1100_mtd_exit);
> +module_platform_driver(sa1100_mtd_driver);
>
>  MODULE_AUTHOR("Nicolas Pitre");
>  MODULE_DESCRIPTION("SA1100 CFI map driver");
> diff --git a/drivers/mtd/maps/sun_uflash.c b/drivers/mtd/maps/sun_uflash.c
> index 2d66234..175e537 100644
> --- a/drivers/mtd/maps/sun_uflash.c
> +++ b/drivers/mtd/maps/sun_uflash.c
> @@ -158,15 +158,4 @@ static struct platform_driver uflash_driver = {
>        .remove         = __devexit_p(uflash_remove),
>  };
>
> -static int __init uflash_init(void)
> -{
> -       return platform_driver_register(&uflash_driver);
> -}
> -
> -static void __exit uflash_exit(void)
> -{
> -       platform_driver_unregister(&uflash_driver);
> -}
> -
> -module_init(uflash_init);
> -module_exit(uflash_exit);
> +module_platform_driver(uflash_driver);
> diff --git a/drivers/mtd/nand/ams-delta.c b/drivers/mtd/nand/ams-delta.c
> index 9e6b498..3197e97 100644
> --- a/drivers/mtd/nand/ams-delta.c
> +++ b/drivers/mtd/nand/ams-delta.c
> @@ -280,17 +280,7 @@ static struct platform_driver ams_delta_nand_driver = {
>        },
>  };
>
> -static int __init ams_delta_nand_init(void)
> -{
> -       return platform_driver_register(&ams_delta_nand_driver);
> -}
> -module_init(ams_delta_nand_init);
> -
> -static void __exit ams_delta_nand_exit(void)
> -{
> -       platform_driver_unregister(&ams_delta_nand_driver);
> -}
> -module_exit(ams_delta_nand_exit);
> +module_platform_driver(ams_delta_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Jonathan McDowell <noodles@earth.li>");
> diff --git a/drivers/mtd/nand/au1550nd.c b/drivers/mtd/nand/au1550nd.c
> index 77fb4e6..73abbc3 100644
> --- a/drivers/mtd/nand/au1550nd.c
> +++ b/drivers/mtd/nand/au1550nd.c
> @@ -560,17 +560,7 @@ static struct platform_driver au1550nd_driver = {
>        .remove         = __devexit_p(au1550nd_remove),
>  };
>
> -static int __init au1550nd_load(void)
> -{
> -       return platform_driver_register(&au1550nd_driver);
> -};
> -
> -static void __exit au1550nd_exit(void)
> -{
> -       platform_driver_unregister(&au1550nd_driver);
> -};
> -module_init(au1550nd_load);
> -module_exit(au1550nd_exit);
> +module_platform_driver(au1550nd_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Embedded Edge, LLC");
> diff --git a/drivers/mtd/nand/bcm_umi_nand.c b/drivers/mtd/nand/bcm_umi_nand.c
> index 46b58d6..50387fd 100644
> --- a/drivers/mtd/nand/bcm_umi_nand.c
> +++ b/drivers/mtd/nand/bcm_umi_nand.c
> @@ -546,18 +546,7 @@ static struct platform_driver nand_driver = {
>        .resume = bcm_umi_nand_resume,
>  };
>
> -static int __init nand_init(void)
> -{
> -       return platform_driver_register(&nand_driver);
> -}
> -
> -static void __exit nand_exit(void)
> -{
> -       platform_driver_unregister(&nand_driver);
> -}
> -
> -module_init(nand_init);
> -module_exit(nand_exit);
> +module_platform_driver(nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Broadcom");
> diff --git a/drivers/mtd/nand/fsl_elbc_nand.c b/drivers/mtd/nand/fsl_elbc_nand.c
> index eedd8ee..f2f6e87 100644
> --- a/drivers/mtd/nand/fsl_elbc_nand.c
> +++ b/drivers/mtd/nand/fsl_elbc_nand.c
> @@ -971,18 +971,7 @@ static struct platform_driver fsl_elbc_nand_driver = {
>        .remove = fsl_elbc_nand_remove,
>  };
>
> -static int __init fsl_elbc_nand_init(void)
> -{
> -       return platform_driver_register(&fsl_elbc_nand_driver);
> -}
> -
> -static void __exit fsl_elbc_nand_exit(void)
> -{
> -       platform_driver_unregister(&fsl_elbc_nand_driver);
> -}
> -
> -module_init(fsl_elbc_nand_init);
> -module_exit(fsl_elbc_nand_exit);
> +module_platform_driver(fsl_elbc_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Freescale");
> diff --git a/drivers/mtd/nand/fsl_upm.c b/drivers/mtd/nand/fsl_upm.c
> index b4f3cc9..45df542 100644
> --- a/drivers/mtd/nand/fsl_upm.c
> +++ b/drivers/mtd/nand/fsl_upm.c
> @@ -353,17 +353,7 @@ static struct platform_driver of_fun_driver = {
>        .remove         = __devexit_p(fun_remove),
>  };
>
> -static int __init fun_module_init(void)
> -{
> -       return platform_driver_register(&of_fun_driver);
> -}
> -module_init(fun_module_init);
> -
> -static void __exit fun_module_exit(void)
> -{
> -       platform_driver_unregister(&of_fun_driver);
> -}
> -module_exit(fun_module_exit);
> +module_platform_driver(of_fun_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
> diff --git a/drivers/mtd/nand/jz4740_nand.c b/drivers/mtd/nand/jz4740_nand.c
> index e266407..ac3b9f2 100644
> --- a/drivers/mtd/nand/jz4740_nand.c
> +++ b/drivers/mtd/nand/jz4740_nand.c
> @@ -423,17 +423,7 @@ static struct platform_driver jz_nand_driver = {
>        },
>  };
>
> -static int __init jz_nand_init(void)
> -{
> -       return platform_driver_register(&jz_nand_driver);
> -}
> -module_init(jz_nand_init);
> -
> -static void __exit jz_nand_exit(void)
> -{
> -       platform_driver_unregister(&jz_nand_driver);
> -}
> -module_exit(jz_nand_exit);
> +module_platform_driver(jz_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Lars-Peter Clausen <lars@metafoo.de>");
> diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c
> index 5ede647..c240cf1 100644
> --- a/drivers/mtd/nand/mpc5121_nfc.c
> +++ b/drivers/mtd/nand/mpc5121_nfc.c
> @@ -879,19 +879,7 @@ static struct platform_driver mpc5121_nfc_driver = {
>        },
>  };
>
> -static int __init mpc5121_nfc_init(void)
> -{
> -       return platform_driver_register(&mpc5121_nfc_driver);
> -}
> -
> -module_init(mpc5121_nfc_init);
> -
> -static void __exit mpc5121_nfc_cleanup(void)
> -{
> -       platform_driver_unregister(&mpc5121_nfc_driver);
> -}
> -
> -module_exit(mpc5121_nfc_cleanup);
> +module_platform_driver(mpc5121_nfc_driver);
>
>  MODULE_AUTHOR("Freescale Semiconductor, Inc.");
>  MODULE_DESCRIPTION("MPC5121 NAND MTD driver");
> diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
> index f8aacf4..ec68854 100644
> --- a/drivers/mtd/nand/ndfc.c
> +++ b/drivers/mtd/nand/ndfc.c
> @@ -294,18 +294,7 @@ static struct platform_driver ndfc_driver = {
>        .remove = __devexit_p(ndfc_remove),
>  };
>
> -static int __init ndfc_nand_init(void)
> -{
> -       return platform_driver_register(&ndfc_driver);
> -}
> -
> -static void __exit ndfc_nand_exit(void)
> -{
> -       platform_driver_unregister(&ndfc_driver);
> -}
> -
> -module_init(ndfc_nand_init);
> -module_exit(ndfc_nand_exit);
> +module_platform_driver(ndfc_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Thomas Gleixner <tglx@linutronix.de>");
> diff --git a/drivers/mtd/nand/nomadik_nand.c b/drivers/mtd/nand/nomadik_nand.c
> index b463ecf..673dc6c 100644
> --- a/drivers/mtd/nand/nomadik_nand.c
> +++ b/drivers/mtd/nand/nomadik_nand.c
> @@ -228,19 +228,7 @@ static struct platform_driver nomadik_nand_driver = {
>        },
>  };
>
> -static int __init nand_nomadik_init(void)
> -{
> -       pr_info("Nomadik NAND driver\n");
> -       return platform_driver_register(&nomadik_nand_driver);
> -}
> -
> -static void __exit nand_nomadik_exit(void)
> -{
> -       platform_driver_unregister(&nomadik_nand_driver);
> -}
> -
> -module_init(nand_nomadik_init);
> -module_exit(nand_nomadik_exit);
> +module_platform_driver(nomadik_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("ST Microelectronics (sachin.verma@st.com)");
> diff --git a/drivers/mtd/nand/nuc900_nand.c b/drivers/mtd/nand/nuc900_nand.c
> index fa8faed..8febe46 100644
> --- a/drivers/mtd/nand/nuc900_nand.c
> +++ b/drivers/mtd/nand/nuc900_nand.c
> @@ -364,18 +364,7 @@ static struct platform_driver nuc900_nand_driver = {
>        },
>  };
>
> -static int __init nuc900_nand_init(void)
> -{
> -       return platform_driver_register(&nuc900_nand_driver);
> -}
> -
> -static void __exit nuc900_nand_exit(void)
> -{
> -       platform_driver_unregister(&nuc900_nand_driver);
> -}
> -
> -module_init(nuc900_nand_init);
> -module_exit(nuc900_nand_exit);
> +module_platform_driver(nuc900_nand_driver);
>
>  MODULE_AUTHOR("Wan ZongShun <mcuos.com@gmail.com>");
>  MODULE_DESCRIPTION("w90p910/NUC9xx nand driver!");
> diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c
> index f745f00..b3a883e 100644
> --- a/drivers/mtd/nand/omap2.c
> +++ b/drivers/mtd/nand/omap2.c
> @@ -1145,20 +1145,7 @@ static struct platform_driver omap_nand_driver = {
>        },
>  };
>
> -static int __init omap_nand_init(void)
> -{
> -       pr_info("%s driver initializing\n", DRIVER_NAME);
> -
> -       return platform_driver_register(&omap_nand_driver);
> -}
> -
> -static void __exit omap_nand_exit(void)
> -{
> -       platform_driver_unregister(&omap_nand_driver);
> -}
> -
> -module_init(omap_nand_init);
> -module_exit(omap_nand_exit);
> +module_platform_driver(omap_nand_driver);
>
>  MODULE_ALIAS("platform:" DRIVER_NAME);
>  MODULE_LICENSE("GPL");
> diff --git a/drivers/mtd/nand/pasemi_nand.c b/drivers/mtd/nand/pasemi_nand.c
> index a97264e..974dbf8 100644
> --- a/drivers/mtd/nand/pasemi_nand.c
> +++ b/drivers/mtd/nand/pasemi_nand.c
> @@ -230,17 +230,7 @@ static struct platform_driver pasemi_nand_driver =
>        .remove         = pasemi_nand_remove,
>  };
>
> -static int __init pasemi_nand_init(void)
> -{
> -       return platform_driver_register(&pasemi_nand_driver);
> -}
> -module_init(pasemi_nand_init);
> -
> -static void __exit pasemi_nand_exit(void)
> -{
> -       platform_driver_unregister(&pasemi_nand_driver);
> -}
> -module_exit(pasemi_nand_exit);
> +module_platform_driver(pasemi_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Egor Martovetsky <egor@pasemi.com>");
> diff --git a/drivers/mtd/nand/plat_nand.c b/drivers/mtd/nand/plat_nand.c
> index ea8e123..7f2da69 100644
> --- a/drivers/mtd/nand/plat_nand.c
> +++ b/drivers/mtd/nand/plat_nand.c
> @@ -148,18 +148,7 @@ static struct platform_driver plat_nand_driver = {
>        },
>  };
>
> -static int __init plat_nand_init(void)
> -{
> -       return platform_driver_register(&plat_nand_driver);
> -}
> -
> -static void __exit plat_nand_exit(void)
> -{
> -       platform_driver_unregister(&plat_nand_driver);
> -}
> -
> -module_init(plat_nand_init);
> -module_exit(plat_nand_exit);
> +module_platform_driver(plat_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Vitaly Wool");
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 9eb7f87..90d6016 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -1311,17 +1311,7 @@ static struct platform_driver pxa3xx_nand_driver = {
>        .resume         = pxa3xx_nand_resume,
>  };
>
> -static int __init pxa3xx_nand_init(void)
> -{
> -       return platform_driver_register(&pxa3xx_nand_driver);
> -}
> -module_init(pxa3xx_nand_init);
> -
> -static void __exit pxa3xx_nand_exit(void)
> -{
> -       platform_driver_unregister(&pxa3xx_nand_driver);
> -}
> -module_exit(pxa3xx_nand_exit);
> +module_platform_driver(pxa3xx_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("PXA3xx NAND controller driver");
> diff --git a/drivers/mtd/nand/sharpsl.c b/drivers/mtd/nand/sharpsl.c
> index 619d2a5..b175c0f 100644
> --- a/drivers/mtd/nand/sharpsl.c
> +++ b/drivers/mtd/nand/sharpsl.c
> @@ -230,17 +230,7 @@ static struct platform_driver sharpsl_nand_driver = {
>        .remove         = __devexit_p(sharpsl_nand_remove),
>  };
>
> -static int __init sharpsl_nand_init(void)
> -{
> -       return platform_driver_register(&sharpsl_nand_driver);
> -}
> -module_init(sharpsl_nand_init);
> -
> -static void __exit sharpsl_nand_exit(void)
> -{
> -       platform_driver_unregister(&sharpsl_nand_driver);
> -}
> -module_exit(sharpsl_nand_exit);
> +module_platform_driver(sharpsl_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Richard Purdie <rpurdie@rpsys.net>");
> diff --git a/drivers/mtd/nand/socrates_nand.c b/drivers/mtd/nand/socrates_nand.c
> index 0fb24f9..e02b08b 100644
> --- a/drivers/mtd/nand/socrates_nand.c
> +++ b/drivers/mtd/nand/socrates_nand.c
> @@ -273,18 +273,7 @@ static struct platform_driver socrates_nand_driver = {
>        .remove         = __devexit_p(socrates_nand_remove),
>  };
>
> -static int __init socrates_nand_init(void)
> -{
> -       return platform_driver_register(&socrates_nand_driver);
> -}
> -
> -static void __exit socrates_nand_exit(void)
> -{
> -       platform_driver_unregister(&socrates_nand_driver);
> -}
> -
> -module_init(socrates_nand_init);
> -module_exit(socrates_nand_exit);
> +module_platform_driver(socrates_nand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Ilya Yanok");
> diff --git a/drivers/mtd/nand/tmio_nand.c b/drivers/mtd/nand/tmio_nand.c
> index beebd95..6caa0cd 100644
> --- a/drivers/mtd/nand/tmio_nand.c
> +++ b/drivers/mtd/nand/tmio_nand.c
> @@ -533,18 +533,7 @@ static struct platform_driver tmio_driver = {
>        .resume         = tmio_resume,
>  };
>
> -static int __init tmio_init(void)
> -{
> -       return platform_driver_register(&tmio_driver);
> -}
> -
> -static void __exit tmio_exit(void)
> -{
> -       platform_driver_unregister(&tmio_driver);
> -}
> -
> -module_init(tmio_init);
> -module_exit(tmio_exit);
> +module_platform_driver(tmio_driver);
>
>  MODULE_LICENSE("GPL v2");
>  MODULE_AUTHOR("Ian Molton, Dirk Opfer, Chris Humbert, Dmitry Baryshkov");
> diff --git a/drivers/mtd/onenand/generic.c b/drivers/mtd/onenand/generic.c
> index 7813095..0ccd5bf 100644
> --- a/drivers/mtd/onenand/generic.c
> +++ b/drivers/mtd/onenand/generic.c
> @@ -115,21 +115,9 @@ static struct platform_driver generic_onenand_driver = {
>        .remove         = __devexit_p(generic_onenand_remove),
>  };
>
> -MODULE_ALIAS("platform:" DRIVER_NAME);
> -
> -static int __init generic_onenand_init(void)
> -{
> -       return platform_driver_register(&generic_onenand_driver);
> -}
> -
> -static void __exit generic_onenand_exit(void)
> -{
> -       platform_driver_unregister(&generic_onenand_driver);
> -}
> -
> -module_init(generic_onenand_init);
> -module_exit(generic_onenand_exit);
> +module_platform_driver(generic_onenand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");
>  MODULE_DESCRIPTION("Glue layer for OneNAND flash on generic boards");
> +MODULE_ALIAS("platform:" DRIVER_NAME);
> diff --git a/drivers/mtd/onenand/samsung.c b/drivers/mtd/onenand/samsung.c
> index 5474547..fa1ee43 100644
> --- a/drivers/mtd/onenand/samsung.c
> +++ b/drivers/mtd/onenand/samsung.c
> @@ -1133,18 +1133,7 @@ static struct platform_driver s3c_onenand_driver = {
>        .remove         = __devexit_p(s3c_onenand_remove),
>  };
>
> -static int __init s3c_onenand_init(void)
> -{
> -       return platform_driver_register(&s3c_onenand_driver);
> -}
> -
> -static void __exit s3c_onenand_exit(void)
> -{
> -       platform_driver_unregister(&s3c_onenand_driver);
> -}
> -
> -module_init(s3c_onenand_init);
> -module_exit(s3c_onenand_exit);
> +module_platform_driver(s3c_onenand_driver);
>
>  MODULE_LICENSE("GPL");
>  MODULE_AUTHOR("Kyungmin Park <kyungmin.park@samsung.com>");

Acked-by: Wan ZongShun <mcuos.com@gmail.com>

Thanks!

> --
> 1.7.5.4
>
>
>
>



-- 
Wan ZongShun.
www.mcuos.com

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-27 12:45 ` Axel Lin
@ 2011-11-29 19:50   ` Jiandong Zheng
  -1 siblings, 0 replies; 20+ messages in thread
From: Jiandong Zheng @ 2011-11-29 19:50 UTC (permalink / raw)
  To: Axel Lin
  Cc: linux-kernel, David Woodhouse, Scott Branden, Wan ZongShun,
	Dmitry Eremin-Solenikov, Jamie Iles, Olof Johansson,
	Kyungmin Park, Haojian Zhuang, Artem Bityutskiy, Ralf Baechle,
	Brian Norris, Lars-Peter Clausen, Manuel Lauss, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

On 11/27/2011 4:45 AM, Axel Lin wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse<dwmw2@infradead.org>
> Cc: Jiandong Zheng<jdzheng@broadcom.com>
> Cc: Scott Branden<sbranden@broadcom.com>
> Cc: Wan ZongShun<mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov<dbaryshkov@gmail.com>
> Cc: Jamie Iles<jamie@jamieiles.com>
> Cc: Olof Johansson<olof@lixom.net>
> Cc: Kyungmin Park<kyungmin.park@samsung.com>
> Cc: Haojian Zhuang<haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy<Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle<ralf@linux-mips.org>
> Cc: Brian Norris<computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen<lars@metafoo.de>
> Cc: Manuel Lauss<manuel.lauss@googlemail.com>
> Cc: Vimal Singh<vimal.newwork@gmail.com>
> Cc: Andres Salomon<dilinger@queued.net>
> Cc: Sukumar Ghorai<s-ghorai@ti.com>
> Signed-off-by: Axel Lin<axel.lin@gmail.com>
> ---

>   drivers/mtd/nand/bcm_umi_nand.c     |   13 +------------

> diff --git a/drivers/mtd/nand/bcm_umi_nand.c b/drivers/mtd/nand/bcm_umi_nand.c
> index 46b58d6..50387fd 100644
> --- a/drivers/mtd/nand/bcm_umi_nand.c
> +++ b/drivers/mtd/nand/bcm_umi_nand.c
> @@ -546,18 +546,7 @@ static struct platform_driver nand_driver = {
>          .resume = bcm_umi_nand_resume,
>   };
>
> -static int __init nand_init(void)
> -{
> -       return platform_driver_register(&nand_driver);
> -}
> -
> -static void __exit nand_exit(void)
> -{
> -       platform_driver_unregister(&nand_driver);
> -}
> -
> -module_init(nand_init);
> -module_exit(nand_exit);
> +module_platform_driver(nand_driver);
>
>   MODULE_LICENSE("GPL");
>   MODULE_AUTHOR("Broadcom");

Acked-by: Jiandong Zheng <jdzheng@broadcom.com>



^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-29 19:50   ` Jiandong Zheng
  0 siblings, 0 replies; 20+ messages in thread
From: Jiandong Zheng @ 2011-11-29 19:50 UTC (permalink / raw)
  To: Axel Lin
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Haojian Zhuang, Kyungmin Park, Vimal Singh,
	Ralf Baechle, linux-mtd, Andres Salomon, Olof Johansson,
	Jamie Iles, Brian Norris, David Woodhouse, Manuel Lauss

On 11/27/2011 4:45 AM, Axel Lin wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.
>
> Cc: David Woodhouse<dwmw2@infradead.org>
> Cc: Jiandong Zheng<jdzheng@broadcom.com>
> Cc: Scott Branden<sbranden@broadcom.com>
> Cc: Wan ZongShun<mcuos.com@gmail.com>
> Cc: Dmitry Eremin-Solenikov<dbaryshkov@gmail.com>
> Cc: Jamie Iles<jamie@jamieiles.com>
> Cc: Olof Johansson<olof@lixom.net>
> Cc: Kyungmin Park<kyungmin.park@samsung.com>
> Cc: Haojian Zhuang<haojian.zhuang@gmail.com>
> Cc: Artem Bityutskiy<Artem.Bityutskiy@nokia.com>
> Cc: Ralf Baechle<ralf@linux-mips.org>
> Cc: Brian Norris<computersforpeace@gmail.com>
> Cc: Lars-Peter Clausen<lars@metafoo.de>
> Cc: Manuel Lauss<manuel.lauss@googlemail.com>
> Cc: Vimal Singh<vimal.newwork@gmail.com>
> Cc: Andres Salomon<dilinger@queued.net>
> Cc: Sukumar Ghorai<s-ghorai@ti.com>
> Signed-off-by: Axel Lin<axel.lin@gmail.com>
> ---

>   drivers/mtd/nand/bcm_umi_nand.c     |   13 +------------

> diff --git a/drivers/mtd/nand/bcm_umi_nand.c b/drivers/mtd/nand/bcm_umi_nand.c
> index 46b58d6..50387fd 100644
> --- a/drivers/mtd/nand/bcm_umi_nand.c
> +++ b/drivers/mtd/nand/bcm_umi_nand.c
> @@ -546,18 +546,7 @@ static struct platform_driver nand_driver = {
>          .resume = bcm_umi_nand_resume,
>   };
>
> -static int __init nand_init(void)
> -{
> -       return platform_driver_register(&nand_driver);
> -}
> -
> -static void __exit nand_exit(void)
> -{
> -       platform_driver_unregister(&nand_driver);
> -}
> -
> -module_init(nand_init);
> -module_exit(nand_exit);
> +module_platform_driver(nand_driver);
>
>   MODULE_LICENSE("GPL");
>   MODULE_AUTHOR("Broadcom");

Acked-by: Jiandong Zheng <jdzheng@broadcom.com>

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-28  2:34   ` Wan ZongShun
@ 2011-11-29 20:46     ` David Woodhouse
  -1 siblings, 0 replies; 20+ messages in thread
From: David Woodhouse @ 2011-11-29 20:46 UTC (permalink / raw)
  To: Wan ZongShun
  Cc: Axel Lin, linux-kernel, Jiandong Zheng, Scott Branden,
	Dmitry Eremin-Solenikov, Jamie Iles, Olof Johansson,
	Kyungmin Park, Haojian Zhuang, Artem Bityutskiy, Ralf Baechle,
	Brian Norris, Lars-Peter Clausen, Manuel Lauss, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

[-- Attachment #1: Type: text/plain, Size: 316 bytes --]

On Mon, 2011-11-28 at 10:34 +0800, Wan ZongShun wrote:
> 
> Acked-by: Wan ZongShun <mcuos.com@gmail.com>
> 
> Thanks! 

Did you really need to quote 771 lines of patch, just to add that?

Please, remember to trim your citations so that you only repeat what is
actually necessary for context.

-- 
dwmw2

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5818 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-11-29 20:46     ` David Woodhouse
  0 siblings, 0 replies; 20+ messages in thread
From: David Woodhouse @ 2011-11-29 20:46 UTC (permalink / raw)
  To: Wan ZongShun
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Dmitry Eremin-Solenikov, Sukumar Ghorai, linux-kernel,
	Haojian Zhuang, Kyungmin Park, Vimal Singh, Ralf Baechle,
	Jiandong Zheng, Andres Salomon, Olof Johansson, Jamie Iles,
	Axel Lin, Brian Norris, linux-mtd, Manuel Lauss

[-- Attachment #1: Type: text/plain, Size: 316 bytes --]

On Mon, 2011-11-28 at 10:34 +0800, Wan ZongShun wrote:
> 
> Acked-by: Wan ZongShun <mcuos.com@gmail.com>
> 
> Thanks! 

Did you really need to quote 771 lines of patch, just to add that?

Please, remember to trim your citations so that you only repeat what is
actually necessary for context.

-- 
dwmw2

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 5818 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-11-27 12:45 ` Axel Lin
@ 2011-12-01  8:17   ` Artem Bityutskiy
  -1 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2011-12-01  8:17 UTC (permalink / raw)
  To: Axel Lin
  Cc: linux-kernel, David Woodhouse, Jiandong Zheng, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Jamie Iles,
	Olof Johansson, Kyungmin Park, Haojian Zhuang, Artem Bityutskiy,
	Ralf Baechle, Brian Norris, Lars-Peter Clausen, Manuel Lauss,
	Vimal Singh, Andres Salomon, Sukumar Ghorai, linux-mtd

On Sun, 2011-11-27 at 20:45 +0800, Axel Lin wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.

Pushed to l2-mtd-2.6.git, thanks. However, I got rejects in the
drivers/mtd/nand/au1550nd.c file, so this part was ignored. Please, take
a look at this and send an incremental change. Thanks!

Artem.



^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-12-01  8:17   ` Artem Bityutskiy
  0 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2011-12-01  8:17 UTC (permalink / raw)
  To: Axel Lin
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Haojian Zhuang, Kyungmin Park, Vimal Singh,
	Ralf Baechle, Jiandong Zheng, Andres Salomon, Olof Johansson,
	Jamie Iles, Brian Norris, David Woodhouse, linux-mtd,
	Manuel Lauss

On Sun, 2011-11-27 at 20:45 +0800, Axel Lin wrote:
> This patch converts the drivers in drivers/mtd/* to use the
> module_platform_driver() macro which makes the code smaller and a bit
> simpler.

Pushed to l2-mtd-2.6.git, thanks. However, I got rejects in the
drivers/mtd/nand/au1550nd.c file, so this part was ignored. Please, take
a look at this and send an incremental change. Thanks!

Artem.

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-12-01  8:17   ` Artem Bityutskiy
@ 2011-12-01  8:21     ` Manuel Lauss
  -1 siblings, 0 replies; 20+ messages in thread
From: Manuel Lauss @ 2011-12-01  8:21 UTC (permalink / raw)
  To: dedekind1
  Cc: Axel Lin, linux-kernel, David Woodhouse, Jiandong Zheng,
	Scott Branden, Wan ZongShun, Dmitry Eremin-Solenikov, Jamie Iles,
	Olof Johansson, Kyungmin Park, Haojian Zhuang, Artem Bityutskiy,
	Ralf Baechle, Brian Norris, Lars-Peter Clausen, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

On Thu, Dec 1, 2011 at 9:17 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> On Sun, 2011-11-27 at 20:45 +0800, Axel Lin wrote:
>> This patch converts the drivers in drivers/mtd/* to use the
>> module_platform_driver() macro which makes the code smaller and a bit
>> simpler.
>
> Pushed to l2-mtd-2.6.git, thanks. However, I got rejects in the
> drivers/mtd/nand/au1550nd.c file, so this part was ignored. Please, take
> a look at this and send an incremental change. Thanks!

Axel's patch is against a bigger change to au1550nd.c (which introduced the
platform driver in the first place) available in linux-next.

Thanks,
        Manuel

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-12-01  8:21     ` Manuel Lauss
  0 siblings, 0 replies; 20+ messages in thread
From: Manuel Lauss @ 2011-12-01  8:21 UTC (permalink / raw)
  To: dedekind1
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Haojian Zhuang, Kyungmin Park, Vimal Singh,
	Ralf Baechle, Jiandong Zheng, Andres Salomon, Olof Johansson,
	Jamie Iles, Axel Lin, Brian Norris, David Woodhouse, linux-mtd

On Thu, Dec 1, 2011 at 9:17 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> On Sun, 2011-11-27 at 20:45 +0800, Axel Lin wrote:
>> This patch converts the drivers in drivers/mtd/* to use the
>> module_platform_driver() macro which makes the code smaller and a bit
>> simpler.
>
> Pushed to l2-mtd-2.6.git, thanks. However, I got rejects in the
> drivers/mtd/nand/au1550nd.c file, so this part was ignored. Please, take
> a look at this and send an incremental change. Thanks!

Axel's patch is against a bigger change to au1550nd.c (which introduced the
platform driver in the first place) available in linux-next.

Thanks,
        Manuel

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
  2011-12-01  8:21     ` Manuel Lauss
@ 2011-12-04 14:14       ` Artem Bityutskiy
  -1 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2011-12-04 14:14 UTC (permalink / raw)
  To: Manuel Lauss
  Cc: Axel Lin, linux-kernel, David Woodhouse, Jiandong Zheng,
	Scott Branden, Wan ZongShun, Dmitry Eremin-Solenikov, Jamie Iles,
	Olof Johansson, Kyungmin Park, Haojian Zhuang, Artem Bityutskiy,
	Ralf Baechle, Brian Norris, Lars-Peter Clausen, Vimal Singh,
	Andres Salomon, Sukumar Ghorai, linux-mtd

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

On Thu, 2011-12-01 at 09:21 +0100, Manuel Lauss wrote:
> On Thu, Dec 1, 2011 at 9:17 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> > On Sun, 2011-11-27 at 20:45 +0800, Axel Lin wrote:
> >> This patch converts the drivers in drivers/mtd/* to use the
> >> module_platform_driver() macro which makes the code smaller and a bit
> >> simpler.
> >
> > Pushed to l2-mtd-2.6.git, thanks. However, I got rejects in the
> > drivers/mtd/nand/au1550nd.c file, so this part was ignored. Please, take
> > a look at this and send an incremental change. Thanks!
> 
> Axel's patch is against a bigger change to au1550nd.c (which introduced the
> platform driver in the first place) available in linux-next.

Then au1550nd.c bits should go in via the MIPS tree as well.

-- 
Best Regards,
Artem Bityutskiy

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread
* Re: [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver()
@ 2011-12-04 14:14       ` Artem Bityutskiy
  0 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2011-12-04 14:14 UTC (permalink / raw)
  To: Manuel Lauss
  Cc: Artem Bityutskiy, Lars-Peter Clausen, Scott Branden,
	Wan ZongShun, Dmitry Eremin-Solenikov, Sukumar Ghorai,
	linux-kernel, Haojian Zhuang, Kyungmin Park, Vimal Singh,
	Ralf Baechle, Jiandong Zheng, Andres Salomon, Olof Johansson,
	Jamie Iles, Axel Lin, Brian Norris, David Woodhouse, linux-mtd

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

On Thu, 2011-12-01 at 09:21 +0100, Manuel Lauss wrote:
> On Thu, Dec 1, 2011 at 9:17 AM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> > On Sun, 2011-11-27 at 20:45 +0800, Axel Lin wrote:
> >> This patch converts the drivers in drivers/mtd/* to use the
> >> module_platform_driver() macro which makes the code smaller and a bit
> >> simpler.
> >
> > Pushed to l2-mtd-2.6.git, thanks. However, I got rejects in the
> > drivers/mtd/nand/au1550nd.c file, so this part was ignored. Please, take
> > a look at this and send an incremental change. Thanks!
> 
> Axel's patch is against a bigger change to au1550nd.c (which introduced the
> platform driver in the first place) available in linux-next.

Then au1550nd.c bits should go in via the MIPS tree as well.

-- 
Best Regards,
Artem Bityutskiy

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread
end of thread, other threads:[~2011-12-04 14:13 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-11-27 12:45 [PATCH] mtd: convert drivers/mtd/* to use module_platform_driver() Axel Lin
2011-11-27 12:45 ` Axel Lin
2011-11-27 18:20 ` Olof Johansson
2011-11-27 18:20   ` Olof Johansson
2011-11-28  0:23   ` Kyungmin Park
2011-11-28  0:23     ` Kyungmin Park
2011-11-28  0:57 ` Haojian Zhuang
2011-11-28  0:57   ` Haojian Zhuang
2011-11-28  2:34 ` Wan ZongShun
2011-11-28  2:34   ` Wan ZongShun
2011-11-29 20:46   ` David Woodhouse
2011-11-29 20:46     ` David Woodhouse
2011-11-29 19:50 ` Jiandong Zheng
2011-11-29 19:50   ` Jiandong Zheng
2011-12-01  8:17 ` Artem Bityutskiy
2011-12-01  8:17   ` Artem Bityutskiy
2011-12-01  8:21   ` Manuel Lauss
2011-12-01  8:21     ` Manuel Lauss
2011-12-04 14:14     ` Artem Bityutskiy
2011-12-04 14:14       ` Artem Bityutskiy

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.