staging: wfx: Remove unnecessary return variable
authorSumera Priyadarsini <sylphrenadin@gmail.com>
Sun, 26 Jul 2020 13:07:20 +0000 (18:37 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 28 Jul 2020 08:17:30 +0000 (10:17 +0200)
Remove redundant variable in file fwio.c used for returning value.

Issue was found using Coccinelle:

@@
local idexpression ret;
expression e;
@@

-ret =
+return
     e;
-return ret;

Signed-off-by: Sumera Priyadarsini <sylphrenadin@gmail.com>
Link: https://lore.kernel.org/r/20200726130720.12993-1-sylphrenadin@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wfx/fwio.c

index 206c6cf..22d3b68 100644 (file)
@@ -397,10 +397,9 @@ int wfx_init_device(struct wfx_dev *wdev)
        ret = load_firmware_secure(wdev);
        if (ret < 0)
                return ret;
-       ret = config_reg_write_bits(wdev,
-                                   CFG_DIRECT_ACCESS_MODE |
-                                   CFG_IRQ_ENABLE_DATA |
-                                   CFG_IRQ_ENABLE_WRDY,
-                                   CFG_IRQ_ENABLE_DATA);
-       return ret;
+       return config_reg_write_bits(wdev,
+                                    CFG_DIRECT_ACCESS_MODE |
+                                    CFG_IRQ_ENABLE_DATA |
+                                    CFG_IRQ_ENABLE_WRDY,
+                                    CFG_IRQ_ENABLE_DATA);
 }