[PATCH 1/2] ds2781_battery: convert to module_platform_driver

[PATCH 1/2] ds2781_battery: convert to module_platform_driver

[Devendra Naga]

module_platform_driver can be used to replace the
platform_driver register and unregister functions,
with the calls to module_init and module_exit,

i.e. all the code that is doing like the below

static int __init mymod_init(void)
{
	return platform_driver_register(&drv_operations);
}

static void __exit mymod_exit(void)
{
	platform_driver_unregister(&drv_operations);
}

module_init(mymod_init);
module_exit(mymod_exit);

can be replaced with
module_platform_driver(drv_operations)...

Signed-off-by: Devendra Naga <develkernel412222@gmail.com>
---
 drivers/power/ds2781_battery.c |   14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/drivers/power/ds2781_battery.c b/drivers/power/ds2781_battery.c
index 7a1ff4e..8f84ab7 100644
--- a/drivers/power/ds2781_battery.c
+++ b/drivers/power/ds2781_battery.c
@@ -835,19 +835,7 @@ static struct platform_driver ds2781_battery_driver = {
 	.remove   = __devexit_p(ds2781_battery_remove),
 };
 
-static int __init ds2781_battery_init(void)
-{
-	return platform_driver_register(&ds2781_battery_driver);
-}
-
-static void __exit ds2781_battery_exit(void)
-{
-	platform_driver_unregister(&ds2781_battery_driver);
-}
-
-module_init(ds2781_battery_init);
-module_exit(ds2781_battery_exit);
-
+module_platform_driver(ds2781_battery_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Renata Sayakhova <renata@oktetlabs.ru>");
-- 
1.7.9.5

Re: [PATCH 1/2] ds2781_battery: convert to module_platform_driver

[Anton Vorontsov]

On Mon, Jul 30, 2012 at 04:29:10PM +0545, Devendra Naga wrote:
> module_platform_driver can be used to replace the
> platform_driver register and unregister functions,
> with the calls to module_init and module_exit,
> 
> i.e. all the code that is doing like the below
> 
> static int __init mymod_init(void)
> {
> 	return platform_driver_register(&drv_operations);
> }
> 
> static void __exit mymod_exit(void)
> {
> 	platform_driver_unregister(&drv_operations);
> }
> 
> module_init(mymod_init);
> module_exit(mymod_exit);
> 
> can be replaced with
> module_platform_driver(drv_operations)...
> 
> Signed-off-by: Devendra Naga <develkernel412222@gmail.com>

Nice! Both patches applied, thank you!