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: <6604842b607f7e2000c53aa52970c29480fd54b6.camel@sipsolutions.net>
Subject: Re: [PATCH 06/14] wifi: iwlwifi: mvm: add support for Extra EHT LTF
From:   Johannes Berg <johannes@sipsolutions.net>
To:     Wen Gong <quic_wgong@quicinc.com>, gregory.greenman@intel.com
Cc:     linux-wireless@vger.kernel.org
Date:   Fri, 11 Aug 2023 11:12:18 +0200
In-Reply-To: <2027bcac-cfd9-3858-3438-9a7297600350@quicinc.com>
References: <20230613125727.300445-1-gregory.greenman@intel.com>
         <20230613155501.de019d7cc174.I806f0f6042b89274192701a60b4f7900822db666@changeid>
         <2027bcac-cfd9-3858-3438-9a7297600350@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 Thu, 2023-07-20 at 12:02 +0800, Wen Gong wrote:
> On 6/13/2023 8:57 PM, gregory.greenman@intel.com wrote:
> > From: Gregory Greenman <gregory.greenman@intel.com>
> >=20
> > Add support for Extra EHT LTF defined in 9.4.2.313
> > EHT Capabilities element.
> >=20
> > Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
> > ---
> >   drivers/net/wireless/intel/iwlwifi/fw/api/rs.h    |  2 ++
> >   .../net/wireless/intel/iwlwifi/iwl-nvm-parse.c    |  2 ++
> >   drivers/net/wireless/intel/iwlwifi/mvm/rs-fw.c    | 15 ++++++++++++++=
+
> >   include/linux/ieee80211.h                         |  1 +
> >   4 files changed, 20 insertions(+)
> >=20
> This patch changed both ieee80211.h and iwlwifi, if separate the change=
=20
> of ieee80211.h
> to another patch such as commit 4fdeb8471302(wifi: ieee80211: clean up=
=20
> public action codes),

Well, we've not always been so picky about that, and this is literally a
single line addition. So there have always been changes like that, and I
don't think we'll ever stop.

> then it will be easy for someone to pick the patch if he/she only need=
=20
> the change of ieee80211.h.

No. Don't even go there. I don't care about non-upstream stuff you do
the slightest.

johannes