staging: rtl8723bs: core: Fix fall-through warnings for Clang
authorGustavo A. R. Silva <gustavoars@kernel.org>
Fri, 20 Nov 2020 18:27:37 +0000 (12:27 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 23 Nov 2020 16:55:22 +0000 (17:55 +0100)
In preparation to enable -Wimplicit-fallthrough for Clang, fix multiple
warnings by explicitly adding multiple break statements instead of just
letting the code fall through to the next case.

Link: https://github.com/KSPP/linux/issues/115
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Link: https://lore.kernel.org/r/cd07e3ebc11b44a3fe016341438e24cf950994a3.1605896059.git.gustavoars@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_cmd.c
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c
drivers/staging/rtl8723bs/core/rtw_wlan_util.c

index 729eae6e8dd16ec804ccfffa3e7903e3545529ae..4e1301261a01da29413203ba40a968eb7a26db10 100644 (file)
@@ -1480,6 +1480,7 @@ void lps_ctrl_wk_hdl(struct adapter *padapter, u8 lps_ctrl_type)
                break;
        case LPS_CTRL_TRAFFIC_BUSY:
                LPS_Leave(padapter, "LPS_CTRL_TRAFFIC_BUSY");
+               break;
        default:
                break;
        }
index e0b0dd226144f592b54d79e0d82abf0e28098257..8c40ef43ff942e7147e53deb1a5a5a21e1826a13 100644 (file)
@@ -1707,6 +1707,7 @@ unsigned int OnAssocRsp(struct adapter *padapter, union recv_frame *precv_frame)
 
                case _ERPINFO_IE_:
                        ERP_IE_handler(padapter, pIE);
+                       break;
 
                default:
                        break;
index 45e5f06f000492bf1ca752e3570a945c70d015c9..4bc0bea7e9698a10e13e6481e6e10e8445aad4d8 100644 (file)
@@ -1514,6 +1514,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;