diff options
author | Heena Sirwani <heenasirwani@gmail.com> | 2014-09-22 15:44:01 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-23 13:48:06 -0700 |
commit | 38bb617449060ae5301c1b0802bbe4c0fb1252e3 (patch) | |
tree | 660ca109469fc5ee70313dc369ca40c8378542d3 /drivers/staging/bcm/Bcmchar.c | |
parent | 522c15ee3759a94907bdb500ff920d0b857d80ef (diff) | |
download | linux-38bb617449060ae5301c1b0802bbe4c0fb1252e3.tar.bz2 |
Staging: bcm: Remove unneeded else after return.
The following patch fixes the checkpatch.pl warning:
WARNING: else is generally not useful after a break or return.
Signed-off-by: Heena Sirwani <heenasirwani@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/bcm/Bcmchar.c')
-rw-r--r-- | drivers/staging/bcm/Bcmchar.c | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/drivers/staging/bcm/Bcmchar.c b/drivers/staging/bcm/Bcmchar.c index 963a64fd482a..88ce2da531c6 100644 --- a/drivers/staging/bcm/Bcmchar.c +++ b/drivers/staging/bcm/Bcmchar.c @@ -564,9 +564,8 @@ static int bcm_char_ioctl_gpio_set_request(void __user *argp, BCM_DEBUG_PRINT(ad, DBG_TYPE_OTHERS, OSAL_DBG, DBG_LVL_ALL, "GPIO_MODE_REGISTER read failed"); return status; - } else { - status = STATUS_SUCCESS; } + status = STATUS_SUCCESS; /* Set the gpio mode register to output */ *(UINT *)reset_val |= (1<<bit); @@ -673,9 +672,8 @@ static int bcm_char_ioctl_gpio_status_request(void __user *argp, BCM_DEBUG_PRINT(ad, DBG_TYPE_PRINTK, 0, 0, "RDM Failed\n"); return status; - } else { - status = STATUS_SUCCESS; } + status = STATUS_SUCCESS; return status; } @@ -766,9 +764,8 @@ static int bcm_char_ioctl_gpio_multi_request(void __user *argp, BCM_DEBUG_PRINT(ad, DBG_TYPE_PRINTK, 0, 0, "RDM to GPIO_PIN_STATE_REGISTER Failed."); return status; - } else { - status = STATUS_SUCCESS; } + status = STATUS_SUCCESS; pgpio_multi_info[WIMAX_IDX].uiGPIOValue = (*(UINT *)reset_val & @@ -822,9 +819,8 @@ static int bcm_char_ioctl_gpio_mode_request(void __user *argp, BCM_DEBUG_PRINT(ad, DBG_TYPE_PRINTK, 0, 0, "Read of GPIO_MODE_REGISTER failed"); return status; - } else { - status = STATUS_SUCCESS; } + status = STATUS_SUCCESS; /* Validating the request */ if (IsReqGpioIsLedInNVM(ad, pgpio_multi_mode[WIMAX_IDX].uiGPIOMask) @@ -1079,10 +1075,9 @@ static int bcm_char_ioctl_buffer_download_stop(void __user *argp, up(&ad->fw_download_sema); up(&ad->NVMRdmWrmLock); return status; - } else { - BCM_DEBUG_PRINT(ad, DBG_TYPE_OTHERS, OSAL_DBG, - DBG_LVL_ALL, "Firm Download Over...\n"); } + BCM_DEBUG_PRINT(ad, DBG_TYPE_OTHERS, OSAL_DBG, + DBG_LVL_ALL, "Firm Download Over...\n"); mdelay(10); |