watchdog: geodewdt: Use platform_driver_probe
authorJean Delvare <jdelvare@suse.de>
Fri, 14 Mar 2014 12:16:47 +0000 (13:16 +0100)
committerWim Van Sebroeck <wim@iguana.be>
Mon, 31 Mar 2014 11:35:05 +0000 (13:35 +0200)
Using platform_driver_probe instead of platform_driver_register has
two benefits:
* The driver will fail to load if device probing fails.
* The probe function can be marked __init.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
drivers/watchdog/geodewdt.c

index 4a6ae84b42bcc5b2e829a8bec9a51b017d2a63d5..4c43e3fa8bd2e30d4989dc99c76e3250cafcf25d 100644 (file)
@@ -215,7 +215,7 @@ static struct miscdevice geodewdt_miscdev = {
        .fops = &geodewdt_fops,
 };
 
-static int geodewdt_probe(struct platform_device *dev)
+static int __init geodewdt_probe(struct platform_device *dev)
 {
        int ret;
 
@@ -255,7 +255,6 @@ static void geodewdt_shutdown(struct platform_device *dev)
 }
 
 static struct platform_driver geodewdt_driver = {
-       .probe          = geodewdt_probe,
        .remove         = geodewdt_remove,
        .shutdown       = geodewdt_shutdown,
        .driver         = {
@@ -268,20 +267,18 @@ static int __init geodewdt_init(void)
 {
        int ret;
 
-       ret = platform_driver_register(&geodewdt_driver);
-       if (ret)
-               return ret;
-
        geodewdt_platform_device = platform_device_register_simple(DRV_NAME,
                                                                -1, NULL, 0);
-       if (IS_ERR(geodewdt_platform_device)) {
-               ret = PTR_ERR(geodewdt_platform_device);
+       if (IS_ERR(geodewdt_platform_device))
+               return PTR_ERR(geodewdt_platform_device);
+
+       ret = platform_driver_probe(&geodewdt_driver, geodewdt_probe);
+       if (ret)
                goto err;
-       }
 
        return 0;
 err:
-       platform_driver_unregister(&geodewdt_driver);
+       platform_device_unregister(geodewdt_platform_device);
        return ret;
 }