staging: wilc1000: refactor wilc_wlan_parse_info_frame() function

Return 'void' from wilc_wlan_parse_info_frame() as same constant value
is returned always. Also removed the 'size' from input parameter as its
not used in the function.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Ajay Singh 2018-09-25 11:53:26 +05:30 committed by Greg Kroah-Hartman
parent 5d482d2dc9
commit 1a46d545e9
1 changed files with 4 additions and 8 deletions

View File

@ -350,22 +350,17 @@ static void wilc_wlan_parse_response_frame(u8 *info, int size)
} }
} }
static int wilc_wlan_parse_info_frame(u8 *info, int size) static void wilc_wlan_parse_info_frame(u8 *info)
{ {
struct wilc_mac_cfg *pd = &g_mac; struct wilc_mac_cfg *pd = &g_mac;
u32 wid, len; u32 wid, len;
int type = WILC_CFG_RSP_STATUS;
wid = info[0] | (info[1] << 8); wid = info[0] | (info[1] << 8);
len = info[2]; len = info[2];
if (len == 1 && wid == WID_STATUS) { if (len == 1 && wid == WID_STATUS)
pd->mac_status = info[3]; pd->mac_status = info[3];
type = WILC_CFG_RSP_STATUS;
}
return type;
} }
/******************************************** /********************************************
@ -528,7 +523,8 @@ void wilc_wlan_cfg_indicate_rx(struct wilc *wilc, u8 *frame, int size,
break; break;
case 'I': case 'I':
rsp->type = wilc_wlan_parse_info_frame(frame, size); wilc_wlan_parse_info_frame(frame);
rsp->type = WILC_CFG_RSP_STATUS;
rsp->seq_no = msg_id; rsp->seq_no = msg_id;
/*call host interface info parse as well*/ /*call host interface info parse as well*/
wilc_gnrl_async_info_received(wilc, frame - 4, size + 4); wilc_gnrl_async_info_received(wilc, frame - 4, size + 4);