Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <f39df1cac6519e9f2037d26ea8a75e39cf869ed3.camel@sipsolutions.net>
Subject: Re: [PATCH] mac80211 : fix bw change to 40 MHz during channel switch
From:   Johannes Berg <johannes@sipsolutions.net>
To:     quic_haric@quicinc.com
Cc:     linux-wireless@vger.kernel.org
Date:   Mon, 11 Apr 2022 12:40:35 +0200
In-Reply-To: <1649393452-11704-1-git-send-email-quic_haric@quicinc.com>
References: <1649393452-11704-1-git-send-email-quic_haric@quicinc.com>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.42.4 (3.42.4-2.fc35) 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Precedence: bulk


> +++ b/net/mac80211/spectmgmt.c
> @@ -148,6 +148,11 @@ int ieee80211_parse_ch_switch_ie(struct ieee80211_sub_if_data *sdata,
>  					    IEEE80211_HT_OP_MODE_CCFS2_SHIFT),
>  		};
>  
> +		csa_ie->chandef.width = NL80211_CHAN_WIDTH_40;
> +		csa_ie->chandef.center_freq1 =
> +			ieee80211_channel_to_frequency(wide_bw_chansw_ie->new_center_freq_seg0,
> +						       new_chan->band);
> +
>  		/* default, for the case of IEEE80211_VHT_CHANWIDTH_USE_HT,
>  		 * to the previously parsed chandef
>  		 */

It seems like this should go under the comment, and change
new_vht_chandef, not csa_ie->chandef?

johannes