From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH] SATA/PATA: convert drivers/ata/* to use module_platform_driver() Date: Mon, 28 Nov 2011 09:13:13 +0530 Message-ID: <4ED30351.7090809@st.com> References: <1322376266.2226.3.camel@phoenix> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from eu1sys200aog107.obsmtp.com ([207.126.144.123]:60162 "EHLO eu1sys200aog107.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753141Ab1K1Dr2 (ORCPT ); Sun, 27 Nov 2011 22:47:28 -0500 In-Reply-To: <1322376266.2226.3.camel@phoenix> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Axel Lin Cc: "linux-kernel@vger.kernel.org" , Arnaud Patard , Alessandro Zummo , Sylvain Munaut , Anton Vorontsov , Marek Vasut , Florian Fainelli , Mark Miesfeld , Ashish Kalra , Paul Mundt , Jeff Garzik , "linux-ide@vger.kernel.org" , Shiraz HASHIM , Armando VISCONTI On 11/27/2011 12:14 PM, Axel Lin wrote: > This patch converts the drivers in drivers/ata/* to use the > module_platform_driver() macro which makes the code smaller and a bit > simpler. > > Cc: Viresh Kumar > Cc: Jeff Garzik > Cc: Arnaud Patard > Cc: Alessandro Zummo > Cc: Sylvain Munaut > Cc: Anton Vorontsov > Cc: Marek Vasut > Cc: Florian Fainelli > Cc: Mark Miesfeld > Cc: Ashish Kalra > Cc: Paul Mundt > Signed-off-by: Axel Lin > --- > drivers/ata/pata_arasan_cf.c | 12 +----------- > drivers/ata/pata_at91.c | 15 +-------------- > drivers/ata/pata_imx.c | 12 +----------- > drivers/ata/pata_ixp4xx_cf.c | 13 +------------ > drivers/ata/pata_mpc52xx.c | 21 +-------------------- > drivers/ata/pata_of_platform.c | 12 +----------- > drivers/ata/pata_palmld.c | 13 +------------ > drivers/ata/pata_platform.c | 12 +----------- > drivers/ata/pata_pxa.c | 13 +------------ > drivers/ata/pata_rb532_cf.c | 21 ++------------------- > drivers/ata/sata_dwc_460ex.c | 13 +------------ > drivers/ata/sata_fsl.c | 14 +------------- > 12 files changed, 13 insertions(+), 158 deletions(-) > > diff --git a/drivers/ata/pata_arasan_cf.c b/drivers/ata/pata_arasan_cf.c > index e8574bb..048589f 100644 > --- a/drivers/ata/pata_arasan_cf.c > +++ b/drivers/ata/pata_arasan_cf.c > @@ -963,17 +963,7 @@ static struct platform_driver arasan_cf_driver = { > }, > }; > > -static int __init arasan_cf_init(void) > -{ > - return platform_driver_register(&arasan_cf_driver); > -} > -module_init(arasan_cf_init); > - > -static void __exit arasan_cf_exit(void) > -{ > - platform_driver_unregister(&arasan_cf_driver); > -} > -module_exit(arasan_cf_exit); > +module_platform_driver(arasan_cf_driver); > > MODULE_AUTHOR("Viresh Kumar "); > MODULE_DESCRIPTION("Arasan ATA Compact Flash driver"); For SPEAr: Acked-by: Viresh Kumar -- viresh