staging: greybus: arche-platform: compress return logic into one line
authorEva Rachel Retuya <eraretuya@gmail.com>
Wed, 21 Sep 2016 05:07:04 +0000 (13:07 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 22 Sep 2016 09:54:47 +0000 (11:54 +0200)
Modify return statement to use the value being returned directly instead of
assigning it first to 'ret' and returning this variable. Coccinelle semantic
patch used:

@@
expression e;
local idexpression ret;
@@

-ret =
+return
e;
-return ret;

Signed-off-by: Eva Rachel Retuya <eraretuya@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/greybus/arche-platform.c

index 9d9048e6aed31bc24e9b601e0aced8d0b73249de..d6b36099d8ea6b190ae3cd9d882057e2cfe4d2cb 100644 (file)
@@ -634,8 +634,7 @@ static int arche_platform_probe(struct platform_device *pdev)
        arche_pdata->wake_detect_gpio = of_get_named_gpio(np, "svc,wake-detect-gpio", 0);
        if (arche_pdata->wake_detect_gpio < 0) {
                dev_err(dev, "failed to get wake detect gpio\n");
-               ret = arche_pdata->wake_detect_gpio;
-               return ret;
+               return arche_pdata->wake_detect_gpio;
        }
 
        ret = devm_gpio_request(dev, arche_pdata->wake_detect_gpio, "wake detect");