ath6kl: wmi: Remove unused variable 'rate'
authorLee Jones <lee.jones@linaro.org>
Wed, 26 Aug 2020 09:33:45 +0000 (10:33 +0100)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 27 Aug 2020 10:15:50 +0000 (13:15 +0300)
Fixes the following W=1 kernel build warning(s):

 drivers/net/wireless/ath/ath6kl/wmi.c: In function ‘ath6kl_wmi_bitrate_reply_rx’:
 drivers/net/wireless/ath/ath6kl/wmi.c:1204:6: warning: variable ‘rate’ set but not used [-Wunused-but-set-variable]

Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: linux-wireless@vger.kernel.org
Cc: netdev@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200826093401.1458456-15-lee.jones@linaro.org
drivers/net/wireless/ath/ath6kl/wmi.c

index 6885d2ded53a85ccb94831dc8e5309eb4b89c9c8..a4339cca661f03832c2e95ed92ab2e9c2ce47730 100644 (file)
@@ -1201,8 +1201,7 @@ static int ath6kl_wmi_pstream_timeout_event_rx(struct wmi *wmi, u8 *datap,
 static int ath6kl_wmi_bitrate_reply_rx(struct wmi *wmi, u8 *datap, int len)
 {
        struct wmi_bit_rate_reply *reply;
-       s32 rate;
-       u32 sgi, index;
+       u32 index;
 
        if (len < sizeof(struct wmi_bit_rate_reply))
                return -EINVAL;
@@ -1211,15 +1210,10 @@ static int ath6kl_wmi_bitrate_reply_rx(struct wmi *wmi, u8 *datap, int len)
 
        ath6kl_dbg(ATH6KL_DBG_WMI, "rateindex %d\n", reply->rate_index);
 
-       if (reply->rate_index == (s8) RATE_AUTO) {
-               rate = RATE_AUTO;
-       } else {
+       if (reply->rate_index != (s8) RATE_AUTO) {
                index = reply->rate_index & 0x7f;
                if (WARN_ON_ONCE(index > (RATE_MCS_7_40 + 1)))
                        return -EINVAL;
-
-               sgi = (reply->rate_index & 0x80) ? 1 : 0;
-               rate = wmi_rate_tbl[index][sgi];
        }
 
        ath6kl_wakeup_event(wmi->parent_dev);