From c43f5f9dc4135cdb5c252ae5d35e8ec95740c679 Mon Sep 17 00:00:00 2001 From: Chaehyun Lim Date: Fri, 12 Feb 2016 23:04:44 +0900 Subject: [PATCH] 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 Signed-off-by: Greg Kroah-Hartman --- drivers/staging/wilc1000/host_interface.c | 10 +++------- 1 file 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, -- 2.20.1