linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] backlight: Convert backlight i2c drivers to module_i2c_driver
@ 2012-01-13  8:35 Axel Lin
  2012-01-13  8:38 ` [PATCH 2/2] backlight: Convert backlight spi drivers to module_spi_driver Axel Lin
  0 siblings, 1 reply; 5+ messages in thread
From: Axel Lin @ 2012-01-13  8:35 UTC (permalink / raw)
  To: linux-kernel
  Cc: Michael Hennerich, Dmitry Baryshkov, Richard Purdie, Andrew Morton

Factor out some boilerplate code for i2c driver registration into
module_i2c_driver.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Cc: Michael Hennerich <hennerich@blackfin.uclinux.org>
Cc: Dmitry Baryshkov <dbaryshkov@gmail.com>
Cc: Richard Purdie <rpurdie@rpsys.net>
---
 drivers/video/backlight/adp8860_bl.c |   12 +-----------
 drivers/video/backlight/adp8870_bl.c |   12 +-----------
 drivers/video/backlight/tosa_bl.c    |   13 +------------
 3 files changed, 3 insertions(+), 34 deletions(-)

diff --git a/drivers/video/backlight/adp8860_bl.c b/drivers/video/backlight/adp8860_bl.c
index 378276c..550dbf0 100644
--- a/drivers/video/backlight/adp8860_bl.c
+++ b/drivers/video/backlight/adp8860_bl.c
@@ -819,17 +819,7 @@ static struct i2c_driver adp8860_driver = {
 	.id_table = adp8860_id,
 };
 
-static int __init adp8860_init(void)
-{
-	return i2c_add_driver(&adp8860_driver);
-}
-module_init(adp8860_init);
-
-static void __exit adp8860_exit(void)
-{
-	i2c_del_driver(&adp8860_driver);
-}
-module_exit(adp8860_exit);
+module_i2c_driver(adp8860_driver);
 
 MODULE_LICENSE("GPL v2");
 MODULE_AUTHOR("Michael Hennerich <hennerich@blackfin.uclinux.org>");
diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c
index 6735059..9be58c6 100644
--- a/drivers/video/backlight/adp8870_bl.c
+++ b/drivers/video/backlight/adp8870_bl.c
@@ -991,17 +991,7 @@ static struct i2c_driver adp8870_driver = {
 	.id_table = adp8870_id,
 };
 
-static int __init adp8870_init(void)
-{
-	return i2c_add_driver(&adp8870_driver);
-}
-module_init(adp8870_init);
-
-static void __exit adp8870_exit(void)
-{
-	i2c_del_driver(&adp8870_driver);
-}
-module_exit(adp8870_exit);
+module_i2c_driver(adp8870_driver);
 
 MODULE_LICENSE("GPL v2");
 MODULE_AUTHOR("Michael Hennerich <hennerich@blackfin.uclinux.org>");
diff --git a/drivers/video/backlight/tosa_bl.c b/drivers/video/backlight/tosa_bl.c
index 425a736..2b241ab 100644
--- a/drivers/video/backlight/tosa_bl.c
+++ b/drivers/video/backlight/tosa_bl.c
@@ -181,18 +181,7 @@ static struct i2c_driver tosa_bl_driver = {
 	.id_table	= tosa_bl_id,
 };
 
-static int __init tosa_bl_init(void)
-{
-	return i2c_add_driver(&tosa_bl_driver);
-}
-
-static void __exit tosa_bl_exit(void)
-{
-	i2c_del_driver(&tosa_bl_driver);
-}
-
-module_init(tosa_bl_init);
-module_exit(tosa_bl_exit);
+module_i2c_driver(tosa_bl_driver);
 
 MODULE_AUTHOR("Dmitry Baryshkov");
 MODULE_LICENSE("GPL v2");
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] backlight: Convert backlight spi drivers to module_spi_driver
@ 2012-01-13  8:45 Jingoo Han
  0 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2012-01-13  8:45 UTC (permalink / raw)
  To: Axel Lin, linux-kernel
  Cc: Jingoo Han, Eric Miao, Alberto Panizzo, Donghwa Lee, Marek Vasut,
	Haavard Skinnemoen, In-Ki Dae, Dmitry Baryshkov, Ben Dooks,
	Richard Purdie, Andrew Morton

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=euc-kr, Size: 9271 bytes --]

Hi, Axel Lin.
> -----Original Message-----
> From: Axel Lin [mailto:axel.lin@gmail.com]
> Sent: Friday, January 13, 2012 5:38 PM
> To: linux-kernel@vger.kernel.org
> Cc: Jingoo Han; Eric Miao; Alberto Panizzo; Donghwa Lee; Marek Vasut; Haavard Skinnemoen; InKi Dae;
> Dmitry Baryshkov; Ben Dooks; Richard Purdie; Andrew Morton
> Subject: [PATCH 2/2] backlight: Convert backlight spi drivers to module_spi_driver
> 
> Factor out some boilerplate code for spi driver registration into
> module_spi_driver.
> 
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
> Cc: Jingoo Han <jg1.han@samsung.com>

Acked-by: Jingoo Han <jg1.han@samsung.com>
Thank you.

> Cc: Eric Miao <eric.y.miao@gmail.com>
> Cc: Alberto Panizzo <maramaopercheseimorto@gmail.com>
> Cc: Donghwa Lee <dh09.lee@samsung.com>
> Cc: Marek Vasut <marek.vasut@gmail.com>
> Cc: Haavard Skinnemoen <hskinnemoen@atmel.com>
> Cc: InKi Dae <inki.dae@samsung.com>
> Cc: Dmitry Baryshkov <dbaryshkov@gmail.com>
> Cc: Ben Dooks <ben-linux@fluff.org>
> Cc: Richard Purdie <rpurdie@rpsys.net>
> ---
>  drivers/video/backlight/ams369fg06.c  |   13 +------------
>  drivers/video/backlight/corgi_lcd.c   |   12 +-----------
>  drivers/video/backlight/l4f00242t03.c |   13 +------------
>  drivers/video/backlight/ld9040.c      |   13 +------------
>  drivers/video/backlight/lms283gf05.c  |   13 +------------
>  drivers/video/backlight/ltv350qv.c    |   12 +-----------
>  drivers/video/backlight/s6e63m0.c     |   13 +------------
>  drivers/video/backlight/tdo24m.c      |   12 +-----------
>  drivers/video/backlight/tosa_lcd.c    |   13 +------------
>  drivers/video/backlight/vgg2432a4.c   |   15 +--------------
>  10 files changed, 10 insertions(+), 119 deletions(-)
> 
> diff --git a/drivers/video/backlight/ams369fg06.c b/drivers/video/backlight/ams369fg06.c
> index 7838a23..7bdadc7 100644
> --- a/drivers/video/backlight/ams369fg06.c
> +++ b/drivers/video/backlight/ams369fg06.c
> @@ -629,18 +629,7 @@ static struct spi_driver ams369fg06_driver = {
>  	.resume		= ams369fg06_resume,
>  };
> 
> -static int __init ams369fg06_init(void)
> -{
> -	return spi_register_driver(&ams369fg06_driver);
> -}
> -
> -static void __exit ams369fg06_exit(void)
> -{
> -	spi_unregister_driver(&ams369fg06_driver);
> -}
> -
> -module_init(ams369fg06_init);
> -module_exit(ams369fg06_exit);
> +module_spi_driver(ams369fg06_driver);
> 
>  MODULE_AUTHOR("Jingoo Han <jg1.han@samsung.com>");
>  MODULE_DESCRIPTION("ams369fg06 LCD Driver");
> diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
> index c6533ba..6dab13f 100644
> --- a/drivers/video/backlight/corgi_lcd.c
> +++ b/drivers/video/backlight/corgi_lcd.c
> @@ -629,17 +629,7 @@ static struct spi_driver corgi_lcd_driver = {
>  	.resume		= corgi_lcd_resume,
>  };
> 
> -static int __init corgi_lcd_init(void)
> -{
> -	return spi_register_driver(&corgi_lcd_driver);
> -}
> -module_init(corgi_lcd_init);
> -
> -static void __exit corgi_lcd_exit(void)
> -{
> -	spi_unregister_driver(&corgi_lcd_driver);
> -}
> -module_exit(corgi_lcd_exit);
> +module_spi_driver(corgi_lcd_driver);
> 
>  MODULE_DESCRIPTION("LCD and backlight driver for SHARP C7x0/Cxx00");
>  MODULE_AUTHOR("Eric Miao <eric.miao@marvell.com>");
> diff --git a/drivers/video/backlight/l4f00242t03.c b/drivers/video/backlight/l4f00242t03.c
> index 4f5d1c4..fae421a 100644
> --- a/drivers/video/backlight/l4f00242t03.c
> +++ b/drivers/video/backlight/l4f00242t03.c
> @@ -272,18 +272,7 @@ static struct spi_driver l4f00242t03_driver = {
>  	.shutdown	= l4f00242t03_shutdown,
>  };
> 
> -static __init int l4f00242t03_init(void)
> -{
> -	return spi_register_driver(&l4f00242t03_driver);
> -}
> -
> -static __exit void l4f00242t03_exit(void)
> -{
> -	spi_unregister_driver(&l4f00242t03_driver);
> -}
> -
> -module_init(l4f00242t03_init);
> -module_exit(l4f00242t03_exit);
> +module_spi_driver(l4f00242t03_driver);
> 
>  MODULE_AUTHOR("Alberto Panizzo <maramaopercheseimorto@gmail.com>");
>  MODULE_DESCRIPTION("EPSON L4F00242T03 LCD");
> diff --git a/drivers/video/backlight/ld9040.c b/drivers/video/backlight/ld9040.c
> index 78dafc0..efd352b 100644
> --- a/drivers/video/backlight/ld9040.c
> +++ b/drivers/video/backlight/ld9040.c
> @@ -856,18 +856,7 @@ static struct spi_driver ld9040_driver = {
>  	.resume		= ld9040_resume,
>  };
> 
> -static int __init ld9040_init(void)
> -{
> -	return spi_register_driver(&ld9040_driver);
> -}
> -
> -static void __exit ld9040_exit(void)
> -{
> -	spi_unregister_driver(&ld9040_driver);
> -}
> -
> -module_init(ld9040_init);
> -module_exit(ld9040_exit);
> +module_spi_driver(ld9040_driver);
> 
>  MODULE_AUTHOR("Donghwa Lee <dh09.lee@samsung.com>");
>  MODULE_DESCRIPTION("ld9040 LCD Driver");
> diff --git a/drivers/video/backlight/lms283gf05.c b/drivers/video/backlight/lms283gf05.c
> index 4ec78cf..4161f9e 100644
> --- a/drivers/video/backlight/lms283gf05.c
> +++ b/drivers/video/backlight/lms283gf05.c
> @@ -226,18 +226,7 @@ static struct spi_driver lms283gf05_driver = {
>  	.remove		= __devexit_p(lms283gf05_remove),
>  };
> 
> -static __init int lms283gf05_init(void)
> -{
> -	return spi_register_driver(&lms283gf05_driver);
> -}
> -
> -static __exit void lms283gf05_exit(void)
> -{
> -	spi_unregister_driver(&lms283gf05_driver);
> -}
> -
> -module_init(lms283gf05_init);
> -module_exit(lms283gf05_exit);
> +module_spi_driver(lms283gf05_driver);
> 
>  MODULE_AUTHOR("Marek Vasut <marek.vasut@gmail.com>");
>  MODULE_DESCRIPTION("LCD283GF05 LCD");
> diff --git a/drivers/video/backlight/ltv350qv.c b/drivers/video/backlight/ltv350qv.c
> index cca43c0..333949f 100644
> --- a/drivers/video/backlight/ltv350qv.c
> +++ b/drivers/video/backlight/ltv350qv.c
> @@ -321,17 +321,7 @@ static struct spi_driver ltv350qv_driver = {
>  	.resume		= ltv350qv_resume,
>  };
> 
> -static int __init ltv350qv_init(void)
> -{
> -	return spi_register_driver(&ltv350qv_driver);
> -}
> -
> -static void __exit ltv350qv_exit(void)
> -{
> -	spi_unregister_driver(&ltv350qv_driver);
> -}
> -module_init(ltv350qv_init);
> -module_exit(ltv350qv_exit);
> +module_spi_driver(ltv350qv_driver);
> 
>  MODULE_AUTHOR("Haavard Skinnemoen (Atmel)");
>  MODULE_DESCRIPTION("Samsung LTV350QV LCD Driver");
> diff --git a/drivers/video/backlight/s6e63m0.c b/drivers/video/backlight/s6e63m0.c
> index e132157..9b0092e 100644
> --- a/drivers/video/backlight/s6e63m0.c
> +++ b/drivers/video/backlight/s6e63m0.c
> @@ -909,18 +909,7 @@ static struct spi_driver s6e63m0_driver = {
>  	.resume		= s6e63m0_resume,
>  };
> 
> -static int __init s6e63m0_init(void)
> -{
> -	return spi_register_driver(&s6e63m0_driver);
> -}
> -
> -static void __exit s6e63m0_exit(void)
> -{
> -	spi_unregister_driver(&s6e63m0_driver);
> -}
> -
> -module_init(s6e63m0_init);
> -module_exit(s6e63m0_exit);
> +module_spi_driver(s6e63m0_driver);
> 
>  MODULE_AUTHOR("InKi Dae <inki.dae@samsung.com>");
>  MODULE_DESCRIPTION("S6E63M0 LCD Driver");
> diff --git a/drivers/video/backlight/tdo24m.c b/drivers/video/backlight/tdo24m.c
> index 1997e12..2368b8e 100644
> --- a/drivers/video/backlight/tdo24m.c
> +++ b/drivers/video/backlight/tdo24m.c
> @@ -459,17 +459,7 @@ static struct spi_driver tdo24m_driver = {
>  	.resume		= tdo24m_resume,
>  };
> 
> -static int __init tdo24m_init(void)
> -{
> -	return spi_register_driver(&tdo24m_driver);
> -}
> -module_init(tdo24m_init);
> -
> -static void __exit tdo24m_exit(void)
> -{
> -	spi_unregister_driver(&tdo24m_driver);
> -}
> -module_exit(tdo24m_exit);
> +module_spi_driver(tdo24m_driver);
> 
>  MODULE_AUTHOR("Eric Miao <eric.miao@marvell.com>");
>  MODULE_DESCRIPTION("Driver for Toppoly TDO24M LCD Panel");
> diff --git a/drivers/video/backlight/tosa_lcd.c b/drivers/video/backlight/tosa_lcd.c
> index 772f601..a2161f6 100644
> --- a/drivers/video/backlight/tosa_lcd.c
> +++ b/drivers/video/backlight/tosa_lcd.c
> @@ -285,18 +285,7 @@ static struct spi_driver tosa_lcd_driver = {
>  	.resume		= tosa_lcd_resume,
>  };
> 
> -static int __init tosa_lcd_init(void)
> -{
> -	return spi_register_driver(&tosa_lcd_driver);
> -}
> -
> -static void __exit tosa_lcd_exit(void)
> -{
> -	spi_unregister_driver(&tosa_lcd_driver);
> -}
> -
> -module_init(tosa_lcd_init);
> -module_exit(tosa_lcd_exit);
> +module_spi_driver(tosa_lcd_driver);
> 
>  MODULE_AUTHOR("Dmitry Baryshkov");
>  MODULE_LICENSE("GPL v2");
> diff --git a/drivers/video/backlight/vgg2432a4.c b/drivers/video/backlight/vgg2432a4.c
> index b49063c..b617fae 100644
> --- a/drivers/video/backlight/vgg2432a4.c
> +++ b/drivers/video/backlight/vgg2432a4.c
> @@ -262,20 +262,7 @@ static struct spi_driver vgg2432a4_driver = {
>  	.resume		= vgg2432a4_resume,
>  };
> 
> -/* Device driver initialisation */
> -
> -static int __init vgg2432a4_init(void)
> -{
> -	return spi_register_driver(&vgg2432a4_driver);
> -}
> -
> -static void __exit vgg2432a4_exit(void)
> -{
> -	spi_unregister_driver(&vgg2432a4_driver);
> -}
> -
> -module_init(vgg2432a4_init);
> -module_exit(vgg2432a4_exit);
> +module_spi_driver(vgg2432a4_driver);
> 
>  MODULE_AUTHOR("Ben Dooks <ben-linux@fluff.org>");
>  MODULE_DESCRIPTION("VGG2432A4 LCD Driver");
> --
> 1.7.5.4
> 

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-01-13 12:15 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-13  8:35 [PATCH 1/2] backlight: Convert backlight i2c drivers to module_i2c_driver Axel Lin
2012-01-13  8:38 ` [PATCH 2/2] backlight: Convert backlight spi drivers to module_spi_driver Axel Lin
2012-01-13  9:57   ` Alberto Panizzo
2012-01-13 12:15   ` Marek Vasut
2012-01-13  8:45 Jingoo Han

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).