Messages in this thread Patch in this message | | | From | Michael Straube <> | Subject | [PATCH 2/5] staging: r8188eu: convert switch to if statement in mgt_dispatcher() | Date | Sat, 8 Jan 2022 09:27:33 +0100 |
| |
The 'switch (GetFrameSubType(pframe))' in mgt_dispatcher() has only one case that does something different than the default case. Convert the switch to an if statement to improve readability.
Signed-off-by: Michael Straube <straube.linux@gmail.com> --- drivers/staging/r8188eu/core/rtw_mlme_ext.c | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c index cdd928c37a3e..a169a7a39ebc 100644 --- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c +++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c @@ -443,24 +443,13 @@ void mgt_dispatcher(struct adapter *padapter, struct recv_frame *precv_frame) psta->RxMgmtFrameSeqNum = precv_frame->attrib.seq_num; } - switch (GetFrameSubType(pframe)) { - case WIFI_AUTH: + if (GetFrameSubType(pframe) == WIFI_AUTH) { if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) ptable->func = &OnAuth; else ptable->func = &OnAuthClient; - fallthrough; - case WIFI_ASSOCREQ: - case WIFI_REASSOCREQ: - case WIFI_PROBEREQ: - case WIFI_BEACON: - case WIFI_ACTION: - _mgt_dispatcher(padapter, ptable, precv_frame); - break; - default: - _mgt_dispatcher(padapter, ptable, precv_frame); - break; } + _mgt_dispatcher(padapter, ptable, precv_frame); } static u32 p2p_listen_state_process(struct adapter *padapter, unsigned char *da) -- 2.34.1
| |