mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-05 06:18:45 -04:00
staging: r8188eu: replace switch-case with if
OnAction_p2p has a switch-case statement where only a single case is handled. Use if instead, this makes the code shorter and easier to read. Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150 Signed-off-by: Martin Kaiser <martin@kaiser.cx> Link: https://lore.kernel.org/r/20221030173326.1588647-8-martin@kaiser.cx Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
b31b29788f
commit
e246bf42ef
@@ -3852,19 +3852,8 @@ static void OnAction_p2p(struct adapter *padapter, struct recv_frame *precv_fram
|
||||
len -= sizeof(struct ieee80211_hdr_3addr);
|
||||
OUI_Subtype = frame_body[5];
|
||||
|
||||
switch (OUI_Subtype) {
|
||||
case P2P_NOTICE_OF_ABSENCE:
|
||||
break;
|
||||
case P2P_PRESENCE_REQUEST:
|
||||
if (OUI_Subtype == P2P_PRESENCE_REQUEST)
|
||||
process_p2p_presence_req(pwdinfo, pframe, len);
|
||||
break;
|
||||
case P2P_PRESENCE_RESPONSE:
|
||||
break;
|
||||
case P2P_GO_DISC_REQUEST:
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
static void OnAction(struct adapter *padapter, struct recv_frame *precv_frame)
|
||||
|
||||
Reference in New Issue
Block a user