mirror of
https://github.com/hardkernel/linux.git
synced 2026-05-31 16:26:41 +09:00
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>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
b41dfdb18e
commit
c43f5f9dc4
@@ -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,
|
||||
|
||||
Reference in New Issue
Block a user