Messages in this thread | | | Date | Fri, 1 Jul 2022 22:58:58 +0300 | Subject | Re: [PATCH -next] staging: r8188eu: remove unneeded semicolon | From | Pavel Skripkin <> |
| |
Hi Martin,
Martin Kaiser <lists@kaiser.cx> says: > Thus wrote Yang Li (yang.lee@linux.alibaba.com): > >> Eliminate the following coccicheck warning: >> ./drivers/staging/r8188eu/hal/HalPwrSeqCmd.c:70:2-3: Unneeded semicolon > >> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> >> --- >> drivers/staging/r8188eu/hal/HalPwrSeqCmd.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> diff --git a/drivers/staging/r8188eu/hal/HalPwrSeqCmd.c b/drivers/staging/r8188eu/hal/HalPwrSeqCmd.c >> index b5f6d41464db..5b0f66573d94 100644 >> --- a/drivers/staging/r8188eu/hal/HalPwrSeqCmd.c >> +++ b/drivers/staging/r8188eu/hal/HalPwrSeqCmd.c >> @@ -67,7 +67,7 @@ u8 HalPwrSeqCmdParsing(struct adapter *padapter, enum r8188eu_pwr_seq seq) >> break; >> default: >> return false; >> - }; >> + } > >> do { >> pwrcfgcmd = pwrseqcmd[aryidx]; >> -- >> 2.20.1.7.g153144c > > Thanks for spotting this. > > Fixes: df8d0a55047b ("staging: r8188eu: make power sequences static")
^^^^^^^^^^^^^^^^^^^^^^^^^^
No need to backport code-style warning, so Fixes: is not needed here.
Thanks, --Pavel Skripkin
| |