Commit e8bcb482 authored Aug 18, 2021 by Gustavo A. R. Silva Committed by Greg Kroah-Hartman Aug 19, 2021

staging: r8188eu: Fix fall-through warnings for Clang

Fix the following fallthrough warnings:

drivers/staging/r8188eu/core/rtw_mlme_ext.c:1498:3: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]
drivers/staging/r8188eu/core/rtw_wlan_util.c:1113:4: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]
drivers/staging/r8188eu/core/rtw_wlan_util.c:1147:4: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]
drivers/staging/r8188eu/core/rtw_wlan_util.c:1405:3: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough]

This helps with the ongoing efforts to globally enable
-Wimplicit-fallthrough for Clang.

Link: https://github.com/KSPP/linux/issues/115


Reviewed-by: Nathan Chancellor <nathan@kernel.org>
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Link: https://lore.kernel.org/r/20210818221418.GA311735@embeddedor


Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 92791836

drivers/staging/r8188eu/core/rtw_mlme_ext.c

+1 −0

Original line number	Diff line number	Diff line
		@@ -1495,6 +1495,7 @@ unsigned int OnAssocRsp(struct adapter padapter, struct recv_frame precv_frame
		break;
		case _ERPINFO_IE_:
		ERP_IE_handler(padapter, pIE);
		break;
		default:
		break;
		}

drivers/staging/r8188eu/core/rtw_wlan_util.c

+3 −0

Original line number	Diff line number	Diff line
		@@ -1110,6 +1110,7 @@ unsigned int is_ap_in_tkip(struct adapter *padapter)
		case _RSN_IE_2_:
		if (!memcmp((pIE->data + 8), RSN_TKIP_CIPHER, 4))
		return true;
		break;
		default:
		break;
		}
		@@ -1144,6 +1145,7 @@ unsigned int should_forbid_n_rate(struct adapter *padapter)
		if ((!memcmp((pIE->data + 8), RSN_CIPHER_SUITE_CCMP, 4)) \|\|
		(!memcmp((pIE->data + 12), RSN_CIPHER_SUITE_CCMP, 4)))
		return false;
		break;
		default:
		break;
		}
		@@ -1401,6 +1403,7 @@ unsigned char check_assoc_AP(u8 *pframe, uint len)
		} else {
		break;
		}
		break;

		default:
		break;