greybus: use PTR_ERR_OR_ZERO()
authorAlex Elder <elder@linaro.org>
Tue, 24 May 2016 18:34:51 +0000 (13:34 -0500)
committerGreg Kroah-Hartman <gregkh@google.com>
Fri, 27 May 2016 18:28:16 +0000 (11:28 -0700)
Coccinelle points out that the macro PTR_ERR_OR_ZERO() handles the
frequent case of converting a pointer into either error code (if its
value is an encoded error value) or 0 (otherwise).  Switch some code
in gb_power_supply_register() to use that macro.  I have verified
this is true of the kernel we're now working with (arche-6.0).

Signed-off-by: Alex Elder <elder@linaro.org>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
drivers/staging/greybus/power_supply.c

index 9cae396..8d6570d 100644 (file)
@@ -547,10 +547,7 @@ static int gb_power_supply_register(struct gb_power_supply *gbpsy)
 
        gbpsy->psy = power_supply_register(&connection->bundle->dev,
                                           &gbpsy->desc, &cfg);
-       if (IS_ERR(gbpsy->psy))
-               return PTR_ERR(gbpsy->psy);
-
-       return 0;
+       return PTR_ERR_OR_ZERO(gbpsy->psy);
 }
 #endif