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: <5b809506-7fea-4598-b6ae-39ea94cc2849@quicinc.com>
Date:   Tue, 29 Aug 2023 09:21:12 -0700
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 1/2] wifi: cfg80211: remove scan_width support
Content-Language: en-US
To:     Johannes Berg <johannes@sipsolutions.net>,
        <linux-wireless@vger.kernel.org>
CC:     Johannes Berg <johannes.berg@intel.com>,
        Xinyue Ling <quic_xinyling@quicinc.com>
References: <20230829121742.634a0631eaf3.I38b5dcce8b130f93743a2d43ab9d89269b937b5d@changeid>
From:   Jeff Johnson <quic_jjohnson@quicinc.com>
In-Reply-To: <20230829121742.634a0631eaf3.I38b5dcce8b130f93743a2d43ab9d89269b937b5d@changeid>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 8/29/2023 3:17 AM, Johannes Berg wrote:
> From: Johannes Berg <johannes.berg@intel.com>
> 
> There really isn't any support for scanning at different
> channel widths than 20 MHz since there's no way to set it.
> Remove this support for now, if somebody wants to maintain
> this whole thing later we can revisit how it should work.
> 
> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
> ---
>   drivers/net/wireless/ath/wil6210/wmi.c        |  2 -
>   .../broadcom/brcm80211/brcmfmac/cfg80211.c    |  1 -
>   include/net/cfg80211.h                        | 63 +---------------
>   include/uapi/linux/nl80211.h                  |  2 +-
>   net/mac80211/ibss.c                           | 30 ++------
>   net/mac80211/ieee80211_i.h                    |  3 +-
>   net/mac80211/ocb.c                            |  5 +-
>   net/mac80211/scan.c                           | 71 ++++---------------
>   net/wireless/mesh.c                           |  5 +-
>   net/wireless/nl80211.c                        |  1 -
>   net/wireless/scan.c                           | 23 +-----
>   net/wireless/trace.h                          |  6 +-
>   net/wireless/util.c                           | 14 ++--
>   13 files changed, 35 insertions(+), 191 deletions(-)

Just want to remind that there were two proposals given to allow setting 
this from upstream:
<https://lore.kernel.org/all/20230301090242.1613672-1-quic_xinyling@quicinc.com/>

In the thread you asked for additional information:
<https://lore.kernel.org/all/6796c3c209a071983d330ca145d7ab8b929d53c6.camel@sipsolutions.net/>

So to see if there is any update on that request Cc: Xinyue Ling 
<quic_xinyling@quicinc.com>

Would hate to see this ripped out just to need to replace it later.

/jeff