linux-kernel - Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit property to limit channels

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Mon, 15 Apr 2024 19:06:28 +0300
From: Kalle Valo <kvalo@...nel.org>
To: Jeff Johnson <quic_jjohnson@...cinc.com>
Cc: Christian Lamparter <christian.lamparter@....uni-stuttgart.de>,  Robert
 Marko <robimarko@...il.com>,  <davem@...emloft.net>,
  <edumazet@...gle.com>,  <kuba@...nel.org>,  <pabeni@...hat.com>,
  <robh@...nel.org>,  <krzysztof.kozlowski+dt@...aro.org>,
  <conor+dt@...nel.org>,  <jjohnson@...nel.org>,
  <linux-wireless@...r.kernel.org>,  <netdev@...r.kernel.org>,
  <devicetree@...r.kernel.org>,  <ath11k@...ts.infradead.org>,
  <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 2/2] wifi: ath11k: support DT ieee80211-freq-limit
 property to limit channels

Jeff Johnson <quic_jjohnson@...cinc.com> writes:

> On 4/12/2024 12:52 PM, Christian Lamparter wrote:
>> On 4/12/24 6:24 PM, Robert Marko wrote:
>>> The common DT property can be used to limit the available channels
>>> but ath11k has to manually call wiphy_read_of_freq_limits().
>>>
>>> Signed-off-by: Robert Marko <robimarko@...il.com>
>> 
>> I've seen this before.
>> 
>> https://patchwork.kernel.org/project/linux-wireless/patch/ed266944c721de8dbf0fe35f387a3a71b2c84037.1686486468.git.chunkeey@gmail.com/
>> 
>> (dt-binding too. it has/had an ack)
>> https://patchwork.kernel.org/project/linux-wireless/patch/fc606d2550d047a53b4289235dd3c0fe23d5daac.1686486468.git.chunkeey@gmail.com/
>> 
>> sooo.... this is awkward.
>
> Patchwork indicates Changes Requested
> Any idea what changes Kalle is looking for?

I can't remember anymore but most likely I assumed based on Krzysztof's
comments there will be v3 and missed that Conor already had acked it.
Sorry about that, I set Christian's patches to New state now so that
they are back in queue. And I'll drop Robert patches. Does this sound ok
to everyone?

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches