MIME-Version: 1.0
References: <20230123090555.21415-1-git@qrsnap.io>
In-Reply-To: <20230123090555.21415-1-git@qrsnap.io>
From:   Nick Morrow <morrownr@gmail.com>
Date:   Fri, 17 Mar 2023 14:49:56 -0500
Message-ID: <CAFktD2eFdaCAdE=zxVx05QYWPRcr5StompKr+ehn7piYpQHjzA@mail.gmail.com>
Subject: Re: [PATCH] Added Netgear AXE3000 (A8000) usb_device_id to mt7921u_device_table[]
To:     Reese Russell <git@qrsnap.io>
Cc:     Felix Fietkau <nbd@nbd.name>,
        Lorenzo Bianconi <lorenzo@kernel.org>,
        Ryder Lee <ryder.lee@mediatek.com>,
        Shayne Chen <shayne.chen@mediatek.com>,
        Sean Wang <sean.wang@mediatek.com>,
        Kalle Valo <kvalo@kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Deren Wu <deren.wu@mediatek.com>,
        YN Chen <YN.Chen@mediatek.com>,
        Ben Greear <greearb@candelatech.com>,
        linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

> Issue: Though the Netgear AXE3000 (A8000) is based on the mt7921
> chipset because of the unique USB VID:PID combination this device
> does not initialize/register. Thus making it not plug and play.
>
> Fix: Adds support for the Netgear AXE3000 (A8000) based on the Mediatek
> mt7921au chipset. The method of action is adding the USD VID/PID
> pair to the mt7921u_device_table[] array.
>
> Notes: A retail sample of the Netgear AXE3000 (A8000) yeilds the following
> from lsusb D 0846:9060 NetGear, Inc. Wireless_Device. This pair
> 0846:9060 VID:PID has been reported by other users on Github.
>
> Signed-off-by: Reese Russell <git@qrsnap.io>
> ---
>  drivers/net/wireless/mediatek/mt76/mt7921/usb.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/usb.c b/drivers/net/wireless/mediatek/mt76/mt7921/usb.c
> index 5321d20dcdcb..62e9728588f8 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/usb.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/usb.c
> @@ -15,6 +15,8 @@
>  static const struct usb_device_id mt7921u_device_table[] = {
>         { USB_DEVICE_AND_INTERFACE_INFO(0x0e8d, 0x7961, 0xff, 0xff, 0xff),
>                 .driver_info = (kernel_ulong_t)MT7921_FIRMWARE_WM },
> +       { USB_DEVICE_AND_INTERFACE_INFO(0x0846, 0x9060, 0xff, 0xff, 0xff),
> +               .driver_info = (kernel_ulong_t)MT7921_FIRMWARE_WM },
>         { },
>  };
>
> --
> 2.37.2


I can confirm this VID/PID needs to go into 6.1 LTS and the current
testing version of the kernel as I am getting an increasing amount of
traffic from users that have purchased the Netgear A8000.

My site is github.com/morrownr/USB-WiFi

Helping Linux users with USB WiFi is what we do.

The OP could have added a comment to the patch showing the adapter
that is causing this patch to be submitted. Maybe he can submit a v2
that can be expedited?

Guidance?

Nick
github.com/morrownr/USB-WiFi