diff options
| author | Phillip Potter <[email protected]> | 2021-08-17 00:44:57 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2021-08-17 20:01:34 +0200 |
| commit | 25bcf747bdfdb413154a3d47985113474dc8133d (patch) | |
| tree | dfdeebf6ee999327bba7d7ea1ecb0799be5e14c7 | |
| parent | 10e13123973bcde7c5d1d2a37a38e4c69a973e98 (diff) | |
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 <[email protected]>
Acked-by: Michael Straube <[email protected]>
Signed-off-by: Phillip Potter <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/r8188eu/hal/rtl8188e_cmd.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/staging/r8188eu/hal/rtl8188e_cmd.c b/drivers/staging/r8188eu/hal/rtl8188e_cmd.c index 6afdebd4f367..43ce92434b7c 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; |