Received-SPF: pass (google.com: domain of linux-wireless+bounces-7690-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <98a7ac4e-7ba5-453c-8b27-270ac7236677@quicinc.com>
Date: Wed, 15 May 2024 09:10:08 -0700
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH V4 01/10] wifi: mac80211: Add EHT UL MU-MIMO flag in
 ieee80211_bss_conf
Content-Language: en-US
To: Jeff Johnson <quic_jjohnson@quicinc.com>, <ath12k@lists.infradead.org>
CC: <linux-wireless@vger.kernel.org>
References: <20240508002055.17705-1-quic_pradeepc@quicinc.com>
 <20240508002055.17705-2-quic_pradeepc@quicinc.com>
 <3b7be076-9aa6-4f21-b9dc-7ee04cf26ce7@quicinc.com>
From: Pradeep Kumar Chitrapu <quic_pradeepc@quicinc.com>
In-Reply-To: <3b7be076-9aa6-4f21-b9dc-7ee04cf26ce7@quicinc.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit


On 5/13/2024 1:49 PM, Jeff Johnson wrote:
> On 5/7/2024 5:20 PM, Pradeep Kumar Chitrapu wrote:
>> Add flag for Full Bandwidth UL MU-MIMO for EHT. This is utilized
>> to pass EHT MU-MIMO configurations from user space to driver.
>>
>> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1
>>
>> Signed-off-by: Pradeep Kumar Chitrapu <quic_pradeepc@quicinc.com>
>> ---
>>   include/net/mac80211.h | 4 ++++
>>   net/mac80211/cfg.c     | 5 +++++
>>   2 files changed, 9 insertions(+)
>>
>> diff --git a/include/net/mac80211.h b/include/net/mac80211.h
>> index cafc664ee531..d7b6f75bc920 100644
>> --- a/include/net/mac80211.h
>> +++ b/include/net/mac80211.h
>> @@ -701,6 +701,9 @@ struct ieee80211_fils_discovery {
>>    *	beamformee
>>    * @eht_mu_beamformer: in AP-mode, does this BSS enable operation as an EHT MU
>>    *	beamformer
>> + * @eht_80mhz_full_bw_ul_mumimo: does this BSS support the reception (AP) or transmission
> 
> suggest you reformat to fit in 80 columns
> (we allow code to exceed 80 columns when it improves readability, but comments
> should almost never exceed 80 columns)

Sure..Thanks Jeff, will address in next revision