Re: [PATCH] SATA/PATA: convert drivers/ata/* to use module_platform_driver()

[Viresh Kumar <viresh.kumar@st.com>]

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 <viresh.kumar@st.com>
> Cc: Jeff Garzik <jgarzik@pobox.com>
> Cc: Arnaud Patard <arnaud.patard@rtp-net.org>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Sylvain Munaut <tnt@246tNt.com>
> Cc: Anton Vorontsov <avorontsov@ru.mvista.com>
> Cc: Marek Vasut <marek.vasut@gmail.com>
> Cc: Florian Fainelli <florian@openwrt.org>
> Cc: Mark Miesfeld <mmiesfeld@amcc.com>
> Cc: Ashish Kalra <ashish.kalra@freescale.com>
> Cc: Paul Mundt <lethal@linux-sh.org>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
> ---
>  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 <viresh.kumar@st.com>");
>  MODULE_DESCRIPTION("Arasan ATA Compact Flash driver");

For SPEAr:

Acked-by: Viresh Kumar <viresh.kumar@st.com>

-- 
viresh