From: Chaehyun Lim Date: Fri, 12 Feb 2016 14:04:44 +0000 (+0900) Subject: staging: wilc1000: change handle_set_channel's return type to void X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=c43f5f9dc4135cdb5c252ae5d35e8ec95740c679;p=openwrt%2Fstaging%2Fblogic.git 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 --- 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,