Re: [PATCH mt76 07/12] wifi: mt76: mt7996: set link_valid field when initializing wcid

[Lorenzo Bianconi <lorenzo@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1437 bytes --]

> This ensures the upper layer uses the correct link ID during packet
> processing.
> 
> Fixes: dd82a9e02c05 ("wifi: mt76: mt7996: Rely on mt7996_sta_link in sta_add/sta_remove callbacks")
> Signed-off-by: Shayne Chen <shayne.chen@mediatek.com>

Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>

> ---
>  drivers/net/wireless/mediatek/mt76/mt7996/main.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7996/main.c b/drivers/net/wireless/mediatek/mt76/mt7996/main.c
> index 685be98b9f27..c26f8f49ce44 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7996/main.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7996/main.c
> @@ -343,6 +343,7 @@ int mt7996_vif_link_add(struct mt76_phy *mphy, struct ieee80211_vif *vif,
>  	INIT_LIST_HEAD(&msta_link->rc_list);
>  	msta_link->wcid.idx = idx;
>  	msta_link->wcid.link_id = link_conf->link_id;
> +	msta_link->wcid.link_valid = ieee80211_vif_is_mld(vif);
>  	msta_link->wcid.tx_info |= MT_WCID_TX_INFO_SET;
>  	mt76_wcid_init(&msta_link->wcid, band_idx);
>  
> @@ -984,6 +985,7 @@ mt7996_mac_sta_init_link(struct mt7996_dev *dev,
>  	msta_link->wcid.sta = 1;
>  	msta_link->wcid.idx = idx;
>  	msta_link->wcid.link_id = link_id;
> +	msta_link->wcid.link_valid = !!sta->valid_links;
>  	msta_link->wcid.def_wcid = &msta->deflink.wcid;
>  
>  	ewma_avg_signal_init(&msta_link->avg_ack_signal);
> -- 
> 2.51.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]