On Tue, Aug 22, 2023 at 8:27 AM Nathan Chancellor <nathan@xxxxxxxxxx> wrote: > > clang warns (or errors with CONFIG_WERROR=y): > > drivers/net/wireless/realtek/rtw89/sar.c:216:3: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough] > 216 | case RTW89_TAS_STATE_DPR_FORBID: > | ^ > drivers/net/wireless/realtek/rtw89/sar.c:216:3: note: insert 'break;' to avoid fall-through > 216 | case RTW89_TAS_STATE_DPR_FORBID: > | ^ > | break; > 1 error generated. > > Clang is a little more pedantic than GCC, which does not warn when > falling through to a case that is just break or return. Clang's version > is more in line with the kernel's own stance in deprecated.rst, which > states that all switch/case blocks must end in either break, > fallthrough, continue, goto, or return. Add the missing break to silence > the warning. > > Closes: https://github.com/ClangBuiltLinux/linux/issues/1921 > Fixes: eb2624f55ad1 ("wifi: rtw89: Introduce Time Averaged SAR (TAS) feature") > Signed-off-by: Nathan Chancellor <nathan@xxxxxxxxxx> Thanks for the patch! Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > --- > drivers/net/wireless/realtek/rtw89/sar.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/wireless/realtek/rtw89/sar.c b/drivers/net/wireless/realtek/rtw89/sar.c > index fafc7a0cfe97..715bf5c2f56a 100644 > --- a/drivers/net/wireless/realtek/rtw89/sar.c > +++ b/drivers/net/wireless/realtek/rtw89/sar.c > @@ -213,6 +213,7 @@ s8 rtw89_query_sar(struct rtw89_dev *rtwdev) > case RTW89_TAS_STATE_DPR_ON: > delta = rtw89_txpwr_tas_to_sar(sar_hdl, tas->delta); > cfg -= delta; > + break; > case RTW89_TAS_STATE_DPR_FORBID: > default: > break; > > --- > base-commit: f585f4ab0b998578b4ef3610ccfc08e207fc3499 > change-id: 20230822-rtw89-tas-clang-implicit-fallthrough-aafed8e950fc > > Best regards, > -- > Nathan Chancellor <nathan@xxxxxxxxxx> > -- Thanks, ~Nick Desaulniers