staging: wilc1000: Merge assignment with return
PATCH v2 1/1 staging: wilc1000: Merge assignment with return
Instead of storing the return value into a variable and then returning it, we can club the two into a single return statement. This change was made using the following semantic patch by Coccinelle:
@@
local idexpression ret;
expression e;
@@
-ret =
+return
e;
-return ret;
Signed-off-by Rehas Sachdeva <[email protected]>
---
Change in v2:
Make the commit message clearer.
drivers/staging/wilc1000/host_interface.c | 5 +---- drivers/staging/wilc1000/wilc_spi.c | 3 +-- drivers/staging/wilc1000/wilc_wlan.c | 3 +-- 3 files changed, 3 insertions(+), 8 deletions(-)
drivers/staging/wilc1000/host_interface.c +1 -4
@@ -3279,7 +3279,6 @@ int wilc_scan(struct wilc_vif *vif, u8 scan_source, u8 scan_type,int wilc_hif_set_cfg (struct wilc_vif *vif,struct cfg_param_attr *cfg_param) { -int result =0 ;struct host_if_msg msg;struct host_if_drv *hif_drv = vif->hif_drv;
@@ -3293,9 +3292,7 @@ int wilc_hif_set_cfg(struct wilc_vif *vif, msg.body.cfg_info = *cfg_param; msg.vif = vif; - result =wilc_enqueue_cmd (&msg); - -return result; +return wilc_enqueue_cmd (&msg); }static void GetPeriodicRSSI (unsigned long arg)
drivers/staging/wilc1000/wilc_spi.c +1 -2
@@ -410,8 +410,7 @@ static int spi_cmd_complete(struct wilc *wilc, u8 cmd, u32 adr, u8 *b, u32 sz,if (len2 >ARRAY_SIZE (wb)) {dev_err (&spi->dev,"spi buffer size too small (%d) (%zu)\n" , len2,ARRAY_SIZE (wb)); - result =N_FAIL ; -return result; +return N_FAIL ; }/* zero spi write buffers. */ for (wix = len; wix < len2; wix++)
drivers/staging/wilc1000/wilc_wlan.c +1 -2
@@ -1001,8 +1001,7 @@ int wilc_wlan_start(struct wilc *wilc) ret = wilc->hif_func->hif_write_reg (wilc,WILC_VMM_CORE_CFG , reg);if (!ret) {release_bus (wilc,RELEASE_ONLY ); - ret = -EIO ; -return ret; +return -EIO ; } reg =0 ;if (wilc->io_type ==HIF_SDIO && wilc->dev_irq_num)
Signature · 1 line
2.7.4