From: Phillip Potter Date: Mon, 16 Aug 2021 23:44:57 +0000 (+0100) Subject: staging: r8188eu: remove set but unused variable from rtl8188e_Add_RateATid X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=25bcf747bdfdb413154a3d47985113474dc8133d;p=linux.git staging: r8188eu: remove set but unused variable from rtl8188e_Add_RateATid Remove set but unused variable init_rate from the rtl8188e_Add_RateATid function in hal/rtl8188eu_cmd.c, as this fixes a kernel test robot warning. Removing the call to get_highest_rate_idx has no side effects here so is safe. Reported-by: kernel test robot Acked-by: Michael Straube Signed-off-by: Phillip Potter Link: https://lore.kernel.org/r/20210816234459.132239-2-phil@philpotter.co.uk Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/r8188eu/hal/rtl8188e_cmd.c b/drivers/staging/r8188eu/hal/rtl8188e_cmd.c index 6afdebd4f367b..43ce92434b7c5 100644 --- a/drivers/staging/r8188eu/hal/rtl8188e_cmd.c +++ b/drivers/staging/r8188eu/hal/rtl8188e_cmd.c @@ -155,7 +155,7 @@ void rtl8188e_Add_RateATid(struct adapter *pAdapter, u32 bitmap, u8 arg, u8 rssi { struct hal_data_8188e *haldata = GET_HAL_DATA(pAdapter); - u8 macid, init_rate, raid, shortGIrate = false; + u8 macid, raid, shortGIrate = false; macid = arg & 0x1f; @@ -167,13 +167,8 @@ void rtl8188e_Add_RateATid(struct adapter *pAdapter, u32 bitmap, u8 arg, u8 rssi bitmap |= ((raid << 28) & 0xf0000000); - init_rate = get_highest_rate_idx(bitmap & 0x0fffffff) & 0x3f; - shortGIrate = (arg & BIT(5)) ? true : false; - if (shortGIrate) - init_rate |= BIT(6); - raid = (bitmap >> 28) & 0x0f; bitmap &= 0x0fffffff;