diff options
author | Chaehyun Lim <chaehyun.lim@gmail.com> | 2016-02-12 23:04:44 +0900 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-02-14 16:25:38 -0800 |
commit | c43f5f9dc4135cdb5c252ae5d35e8ec95740c679 (patch) | |
tree | 30a28ad07df927872869687be92da70b68282872 | |
parent | b41dfdb18e2ad1320ffea58fb7c3c8915e772660 (diff) |
staging: wilc1000: change handle_set_channel's return type to void
When handle_set_channel is called in hostIFthread that is a kernel
thread, it is not checked return type of this function. This patch
changes return type to void and removes braces if statement due
to have a single statement.
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/wilc1000/host_interface.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index e9a6aa7f4afb..eecd697c3f02 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drivers/staging/wilc1000/host_interface.c @@ -299,8 +299,8 @@ static struct wilc_vif *wilc_get_vif_from_idx(struct wilc *wilc, int idx) return wilc->vif[index]; } -static s32 handle_set_channel(struct wilc_vif *vif, - struct channel_attr *hif_set_ch) +static void handle_set_channel(struct wilc_vif *vif, + struct channel_attr *hif_set_ch) { s32 result = 0; struct wid wid; @@ -313,12 +313,8 @@ static s32 handle_set_channel(struct wilc_vif *vif, result = wilc_send_config_pkt(vif, SET_CFG, &wid, 1, wilc_get_vif_idx(vif)); - if (result) { + if (result) PRINT_ER("Failed to set channel\n"); - return -EINVAL; - } - - return result; } static s32 handle_set_wfi_drv_handler(struct wilc_vif *vif, |