Commit 9bad1d0a authored by Chaehyun Lim's avatar Chaehyun Lim Committed by Greg Kroah-Hartman

staging: wilc1000: use switch statement instead of multiple if statement

It is more readable than multiple if-else statement.
Signed-off-by: default avatarChaehyun Lim <chaehyun.lim@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 9e58c36c
...@@ -378,19 +378,31 @@ int wilc_wlan_cfg_set_wid(u8 *frame, u32 offset, u16 id, u8 *buf, int size) ...@@ -378,19 +378,31 @@ int wilc_wlan_cfg_set_wid(u8 *frame, u32 offset, u16 id, u8 *buf, int size)
u8 type = (id >> 12) & 0xf; u8 type = (id >> 12) & 0xf;
int ret = 0; int ret = 0;
if (type == CFG_BYTE_CMD) { switch (type) {
case CFG_BYTE_CMD:
if (size >= 1) if (size >= 1)
ret = wilc_wlan_cfg_set_byte(frame, offset, id, *buf); ret = wilc_wlan_cfg_set_byte(frame, offset, id, *buf);
} else if (type == CFG_HWORD_CMD) { break;
case CFG_HWORD_CMD:
if (size >= 2) if (size >= 2)
ret = wilc_wlan_cfg_set_hword(frame, offset, id, *((u16 *)buf)); ret = wilc_wlan_cfg_set_hword(frame, offset, id,
} else if (type == CFG_WORD_CMD) { *((u16 *)buf));
break;
case CFG_WORD_CMD:
if (size >= 4) if (size >= 4)
ret = wilc_wlan_cfg_set_word(frame, offset, id, *((u32 *)buf)); ret = wilc_wlan_cfg_set_word(frame, offset, id,
} else if (type == CFG_STR_CMD) { *((u32 *)buf));
break;
case CFG_STR_CMD:
ret = wilc_wlan_cfg_set_str(frame, offset, id, buf, size); ret = wilc_wlan_cfg_set_str(frame, offset, id, buf, size);
} else if (type == CFG_BIN_CMD) { break;
case CFG_BIN_CMD:
ret = wilc_wlan_cfg_set_bin(frame, offset, id, buf, size); ret = wilc_wlan_cfg_set_bin(frame, offset, id, buf, size);
break;
} }
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment