wifi: wcn36xx: main: fix sparse warnings
authorKalle Valo <quic_kvalo@quicinc.com>
Wed, 20 Mar 2024 18:24:48 +0000 (20:24 +0200)
committerKalle Valo <quic_kvalo@quicinc.com>
Mon, 25 Mar 2024 10:50:41 +0000 (12:50 +0200)
Sparse warns:

drivers/net/wireless/ath/wcn36xx/main.c:758:58: warning: incorrect type in assignment (different base types)
drivers/net/wireless/ath/wcn36xx/main.c:758:58:    expected unsigned short [usertype] vht_rx_mcs_map
drivers/net/wireless/ath/wcn36xx/main.c:758:58:    got restricted __le16 [usertype] rx_mcs_map
drivers/net/wireless/ath/wcn36xx/main.c:760:58: warning: incorrect type in assignment (different base types)
drivers/net/wireless/ath/wcn36xx/main.c:760:58:    expected unsigned short [usertype] vht_tx_mcs_map
drivers/net/wireless/ath/wcn36xx/main.c:760:58:    got restricted __le16 [usertype] tx_mcs_map

le16_to_cpu() was just missing. Compile tested only.

Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/20240320182449.3757215-4-kvalo@kernel.org
drivers/net/wireless/ath/wcn36xx/main.c

index bfbd3c7a70b37dce1d2a66fb534b4858e0f59190..e760d8002e09d9f5811506d253e6512bc337d0c5 100644 (file)
@@ -756,9 +756,9 @@ static void wcn36xx_update_allowed_rates(struct ieee80211_sta *sta,
        if (sta->deflink.vht_cap.vht_supported) {
                sta_priv->supported_rates.op_rate_mode = STA_11ac;
                sta_priv->supported_rates.vht_rx_mcs_map =
-                               sta->deflink.vht_cap.vht_mcs.rx_mcs_map;
+                       le16_to_cpu(sta->deflink.vht_cap.vht_mcs.rx_mcs_map);
                sta_priv->supported_rates.vht_tx_mcs_map =
-                               sta->deflink.vht_cap.vht_mcs.tx_mcs_map;
+                       le16_to_cpu(sta->deflink.vht_cap.vht_mcs.tx_mcs_map);
        }
 }