Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3;
Message-ID: <51d4c1a8fe45366b4c3f2baf490ba64d8cced56f.camel@sipsolutions.net>
Subject: Re: [PATCH v3 0/9] wifi: cfg80211/mac80211: extend 6 GHz support
 for all power modes
From:   Johannes Berg <johannes@sipsolutions.net>
To:     Wen Gong <quic_wgong@quicinc.com>,
        Aditya Kumar Singh <quic_adisi@quicinc.com>
Cc:     linux-wireless@vger.kernel.org
Date:   Wed, 13 Sep 2023 16:55:49 +0200
In-Reply-To: <efa43e0a-9ef1-4989-0a45-dc6425d9b96b@quicinc.com>
References: <20230315132904.31779-1-quic_adisi@quicinc.com>
         <319ecb67faac8a2e50408f2bfa28f2431a6e6b9a.camel@sipsolutions.net>
         <efa43e0a-9ef1-4989-0a45-dc6425d9b96b@quicinc.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

On Wed, 2023-09-13 at 18:35 +0800, Wen Gong wrote:
>=20
> This patch "[PATCH v3 2/9] wifi: cfg80211: save Power Spectral Density
> (PSD) of the regulatory rule" does not have relation with concurrency.
> Because it only add a field "s8 psd" into struct ieee80211_reg_rule
> and ieee80211_channel. The psd value is same with other field such as
> max_reg_power.
>=20
> max_reg_power is also different value for AP and station mode in db.txt=
=20
> here:
> for example:
> "country TW: DFS-FCC" and "country US: DFS-FCC"
> # 5.15 ~ 5.25 GHz: 30 dBm for master mode, 23 dBm for clients
> https://git.kernel.org/pub/scm/linux/kernel/git/sforshee/wireless-regdb.g=
it/tree/db.txt#n1785
>=20
> So could you merge the "[PATCH v3 2/9] wifi: cfg80211: save Power Spectra=
l
> Density (PSD) of the regulatory rule" firstly?


Well, arguing with the regdb is kind of a bad thing because it reminds
me that nobody ever cares about it anymore ... maybe we should just give
up on it?

I mean, clearly, if you actually cared your patch here would come with
additions to the regdb format (which aren't even that hard now) which
then the wireless-regdb could actually use rather than the comment,
right?

So really all you've achieved right now is again reminding me how little
you really care about upstream?

johannes