On 6/10/2024 8:10 PM, Aaradhana Sahu wrote: > Smatch throws following errors > > drivers/net/wireless/ath/ath12k/mac.c:1922 ath12k_peer_assoc_h_he() error: uninitialized symbol 'rx_mcs_80'. > drivers/net/wireless/ath/ath12k/mac.c:1922 ath12k_peer_assoc_h_he() error: uninitialized symbol 'rx_mcs_160'. > drivers/net/wireless/ath/ath12k/mac.c:1924 ath12k_peer_assoc_h_he() error: uninitialized symbol 'rx_mcs_80'. > > In ath12k_peer_assoc_h_he() rx_mcs_80 and rx_mcs_160 variables > remain uninitialized in the following conditions: > 1. Whenever the value of mcs_80 become equal to > IEEE80211_HE_MCS_NOT_SUPPORTED then rx_mcs_80 remains uninitialized. > 2. Whenever phy capability is not supported 160 channel width and > value of mcs_160 become equal to IEEE80211_HE_MCS_NOT_SUPPORTED > then rx_mcs_160 remains uninitialized. > > Initialize these variables during declaration. > > Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.1.1-00188-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Aaradhana Sahu <quic_aarasahu@xxxxxxxxxxx> Acked-by: Jeff Johnson <quic_jjohnson@xxxxxxxxxxx>
