RE: [PATCH -next] rtw89: remove unneeded semicolon

Pkshih
2021-10-19T09:14:13Z

> -----Original Message-----
> From: Yang Li <yang.lee@linux.alibaba.com>
> Sent: Tuesday, October 19, 2021 3:55 PM
> To: davem@davemloft.net
> Cc: kuba@kernel.org; Pkshih <pkshih@realtek.com>; kvalo@codeaurora.org; linux-wireless@vger.kernel.org;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Yang Li <yang.lee@linux.alibaba.com>
> Subject: [PATCH -next] rtw89: remove unneeded semicolon
>
> Eliminate the following coccicheck warning:
> ./drivers/net/wireless/realtek/rtw89/pci.c:1348:2-3: Unneeded semicolon
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Fixes: e3ec7017f6a2 ("rtw89: add Realtek 802.11ax driver")
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>

Acked-by: Ping-Ke Shih <pkshih@realtek.com>

> ---
> drivers/net/wireless/realtek/rtw89/pci.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/realtek/rtw89/pci.c b/drivers/net/wireless/realtek/rtw89/pci.c
> index e973101..2c94762 100644
> --- a/drivers/net/wireless/realtek/rtw89/pci.c
> +++ b/drivers/net/wireless/realtek/rtw89/pci.c
> @@ -1345,7 +1345,7 @@ static int rtw89_pci_check_mdio(struct rtw89_dev *rtwdev, u8 addr, u8 speed, u16
> default:
> rtw89_err(rtwdev, "[ERR]Error Speed %d!\n", speed);
> return -EINVAL;
> - };
> + }
> rtw89_write16(rtwdev, R_AX_MDIO_CFG, val);
> rtw89_write16_set(rtwdev, R_AX_MDIO_CFG, rw_bit);
>
> --
> 1.8.3.1