LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: "Coelho, Luciano" <luciano.coelho@intel.com>
To: "kvalo@codeaurora.org" <kvalo@codeaurora.org>,
	"jmforbes@linuxtx.org" <jmforbes@linuxtx.org>
Cc: "yj99.shin@samsung.com" <yj99.shin@samsung.com>,
	"davem@davemloft.net" <davem@davemloft.net>,
	"Berg, Johannes" <johannes.berg@intel.com>,
	"Baruch, Yaara" <yaara.baruch@intel.com>,
	"ihab.zhaika@intel.com" <ihab.zhaika@intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"kuba@kernel.org" <kuba@kernel.org>,
	"Gottlieb, Matti" <matti.gottlieb@intel.com>,
	"Grumbach, Emmanuel" <emmanuel.grumbach@intel.com>,
	"jh80.chung@samsung.com" <jh80.chung@samsung.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha
Date: Tue, 31 Aug 2021 06:36:15 +0000	[thread overview]
Message-ID: <ddcb88a3f6614ef6138b68375a22fbba1b068ff3.camel@intel.com> (raw)
In-Reply-To: <CAFxkdArjsp4YxYWYZ_qW7UsNobzodKOaNJqKTHpPf5RmtT+Rww@mail.gmail.com>

On Wed, 2021-08-25 at 13:07 -0500, Justin Forbes wrote:
> On Sat, Aug 21, 2021 at 8:34 AM Kalle Valo <kvalo@codeaurora.org> wrote:
> > 
> > Jaehoon Chung <jh80.chung@samsung.com> writes:
> > 
> > > Hi
> > > 
> > > On 8/9/21 8:09 AM, Jaehoon Chung wrote:
> > > > Hi
> > > > 
> > > > On 7/10/21 2:32 AM, Justin Forbes wrote:
> > > > > On Fri, Jul 2, 2021 at 5:32 PM Justin M. Forbes
> > > > > <jforbes@fedoraproject.org> wrote:
> > > > > > 
> > > > > > The Samsung Galaxy Book Flex2 Alpha uses an ax201 with the ID a0f0/6074.
> > > > > > This works fine with the existing driver once it knows to claim it.
> > > > > > Simple patch to add the device.
> > > > > > 
> > > > > > Signed-off-by: Justin M. Forbes <jforbes@fedoraproject.org>
> > > 
> > > If this patch is merged, can this patch be also applied on stable tree?
> > 
> > Luca, what should we do with this patch?
> > 
> > --
> > https://patchwork.kernel.org/project/linux-wireless/list/
> > 
> > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
> 
> 
> Is that to imply that there is an issue with the submission?  Happy to
> fix any problems, but it would nice to get this in soon.  I know the
> 5.14 merge window was already opened when I sent it, but the 5.15 MR
> is opening soon.  Hardware is definitely shipping and in users hands.

Sorry for the delay here.  This fell between the cracks.

Kalle can you apply this directly to your tree? I'll assign it to you.
And, if possible, add the cc-stable tag so it gets picked up. :)

Longer reasoning: generally we detect the hardware in a more
programmatic way, by checking the type from registers (and not relying
entirely on the PCI IDs), but for some reason this type of device is
still using the legacy way of matching the exact PCI ID to a device
type.  Thus, this patch is needed, at least for now.

Thanks!

--
Cheers,
Luca.

  reply	other threads:[~2021-08-31  6:36 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-02 22:31 Justin M. Forbes
2021-07-09 17:32 ` Justin Forbes
2021-08-08 23:09   ` Jaehoon Chung
2021-08-12 22:26     ` Jaehoon Chung
2021-08-21 13:34       ` Kalle Valo
2021-08-25 18:07         ` Justin Forbes
2021-08-31  6:36           ` Coelho, Luciano [this message]
2021-08-31  6:49             ` Kalle Valo
2021-09-02 16:39 ` Kalle Valo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ddcb88a3f6614ef6138b68375a22fbba1b068ff3.camel@intel.com \
    --to=luciano.coelho@intel.com \
    --cc=davem@davemloft.net \
    --cc=emmanuel.grumbach@intel.com \
    --cc=ihab.zhaika@intel.com \
    --cc=jh80.chung@samsung.com \
    --cc=jmforbes@linuxtx.org \
    --cc=johannes.berg@intel.com \
    --cc=kuba@kernel.org \
    --cc=kvalo@codeaurora.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=matti.gottlieb@intel.com \
    --cc=netdev@vger.kernel.org \
    --cc=yaara.baruch@intel.com \
    --cc=yj99.shin@samsung.com \
    --subject='Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).