2021-07-02 22:33:18

by Justin Forbes

[permalink] [raw]
Subject: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

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 <[email protected]>
---
drivers/net/wireless/intel/iwlwifi/pcie/drv.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/net/wireless/intel/iwlwifi/pcie/drv.c b/drivers/net/wireless/intel/iwlwifi/pcie/drv.c
index 16baee3d52ae..63f5598ebe3f 100644
--- a/drivers/net/wireless/intel/iwlwifi/pcie/drv.c
+++ b/drivers/net/wireless/intel/iwlwifi/pcie/drv.c
@@ -558,6 +558,7 @@ static const struct iwl_dev_info iwl_dev_info_table[] = {
IWL_DEV_INFO(0xA0F0, 0x1652, killer1650i_2ax_cfg_qu_b0_hr_b0, NULL),
IWL_DEV_INFO(0xA0F0, 0x2074, iwl_ax201_cfg_qu_hr, NULL),
IWL_DEV_INFO(0xA0F0, 0x4070, iwl_ax201_cfg_qu_hr, NULL),
+ IWL_DEV_INFO(0xA0F0, 0x6074, iwl_ax201_cfg_qu_hr, NULL),
IWL_DEV_INFO(0x02F0, 0x0070, iwl_ax201_cfg_quz_hr, NULL),
IWL_DEV_INFO(0x02F0, 0x0074, iwl_ax201_cfg_quz_hr, NULL),
IWL_DEV_INFO(0x02F0, 0x6074, iwl_ax201_cfg_quz_hr, NULL),
--
2.31.1


2021-07-09 17:33:29

by Justin Forbes

[permalink] [raw]
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

On Fri, Jul 2, 2021 at 5:32 PM Justin M. Forbes
<[email protected]> 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 <[email protected]>
> ---

Just an update from the user with this hardware that I built a test kernel for:
"Still going strong w/ AX201, speed OK, on par w/ speeds on windows,
no crashes, no weird messages about the driver."

Justin

2021-08-08 23:09:09

by Jaehoon Chung

[permalink] [raw]
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

Hi

On 7/10/21 2:32 AM, Justin Forbes wrote:
> On Fri, Jul 2, 2021 at 5:32 PM Justin M. Forbes
> <[email protected]> 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 <[email protected]>


Before sending patch, I have found same patch to solve the Wifi problem.
Is there any progress about this patch?

I hope that this patch will be applied.

Reviewed-by: Jaehoon Chung <[email protected]>

Best Regards,
Jaehoon Chung


>> ---
>
> Just an update from the user with this hardware that I built a test kernel for:
> "Still going strong w/ AX201, speed OK, on par w/ speeds on windows,
> no crashes, no weird messages about the driver."
>
> Justin
>

2021-08-25 18:08:12

by Justin Forbes

[permalink] [raw]
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

On Sat, Aug 21, 2021 at 8:34 AM Kalle Valo <[email protected]> wrote:
>
> Jaehoon Chung <[email protected]> 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
> >>> <[email protected]> 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 <[email protected]>
> >
> > 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.

Justin

2021-08-31 06:37:45

by Luciano Coelho

[permalink] [raw]
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

On Wed, 2021-08-25 at 13:07 -0500, Justin Forbes wrote:
> On Sat, Aug 21, 2021 at 8:34 AM Kalle Valo <[email protected]> wrote:
> >
> > Jaehoon Chung <[email protected]> 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
> > > > > <[email protected]> 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 <[email protected]>
> > >
> > > 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.

2021-08-31 06:50:58

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

"Coelho, Luciano" <[email protected]> writes:

> On Wed, 2021-08-25 at 13:07 -0500, Justin Forbes wrote:
>> On Sat, Aug 21, 2021 at 8:34 AM Kalle Valo <[email protected]> wrote:
>> >
>> > Jaehoon Chung <[email protected]> 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
>> > > > > <[email protected]> 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 <[email protected]>
>> > >
>> > > 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.

Ok, I'll queue this to v5.15.

> And, if possible, add the cc-stable tag so it gets picked up. :)

Ok.

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

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

2021-09-02 16:40:41

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

"Justin M. Forbes" <[email protected]> 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 <[email protected]>
> Reviewed-by: Jaehoon Chung <[email protected]>

Patch applied to wireless-drivers.git, thanks.

2f32c147a381 iwlwifi Add support for ax201 in Samsung Galaxy Book Flex2 Alpha

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

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