staging: unisys: visorbus: keep the success path on the left

The code was indenting for the successful path and then combining the
error and success path for the rest of the function. Correct it so the
success path is not indented.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: Tim Sell <timothy.sell@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b/drivers/staging/unisys/visorbus/visorbus_main.c
index ea124991cbdd954166b0317b3c386f4adc04d944..308372e55ffdf5d2113404d230ff868c18ac20e8 100644 (file)
--- a/drivers/staging/unisys/visorbus/visorbus_main.c
+++ b/drivers/staging/unisys/visorbus/visorbus_main.c
@@ -888,7 +888,7 @@ static void publish_vbus_dev_info(struct visor_device *visordev)
  */
 static int visordriver_probe_device(struct device *xdev)
 {
-       int res;
+       int err;
        struct visor_driver *drv;
        struct visor_device *dev;
 
@@ -898,15 +898,17 @@ static int visordriver_probe_device(struct device *xdev)
        mutex_lock(&dev->visordriver_callback_lock);
        dev->being_removed = false;
 
-       res = drv->probe(dev);
-       if (res >= 0) {
-               /* success: reference kept via unmatched get_device() */
-               get_device(&dev->device);
-               publish_vbus_dev_info(dev);
+       err = drv->probe(dev);
+       if (err) {
+               mutex_unlock(&dev->visordriver_callback_lock);
+               return err;
        }
 
+       /* success: reference kept via unmatched get_device() */
+       get_device(&dev->device);
+       publish_vbus_dev_info(dev);
        mutex_unlock(&dev->visordriver_callback_lock);
-       return res;
+       return 0;
 }
 
 /*