Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
MIME-Version: 1.0
References: <20231101095533.45258-1-tom@baremetal.dev> <ZUJoFAcKMrVD5Glo@lore-desk>
 <CANjHDnBkCW_zQxyCcKcyrZOYnnLNVKT1Z2AJr6x6DT55UZ+A7Q@mail.gmail.com> <ZUJwJ+bimXFVcXps@lore-desk>
In-Reply-To: <ZUJwJ+bimXFVcXps@lore-desk>
From:   Tom Waldron <tom@baremetal.dev>
Date:   Wed, 1 Nov 2023 15:48:21 +0000
Message-ID: <CANjHDnBsTaqPsMBtN1EU8rUdQwPu3riXcy66BTFBcJcy45HcDw@mail.gmail.com>
Subject: Re: [PATCH 1/1] wifi: mt76: mt7921: Add a new VID/PID 0b48/7922 for MT7922
To:     Lorenzo Bianconi <lorenzo@kernel.org>
Cc:     linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Just realised that I had based my changes on the wrong repo(linux
stable). Also that this change has already been made in the
wireless-next repo. It seems the ITTIM vendor was used in this case.

Please ignore this submission and please accept my apologies for
taking up your time.

Regards

Tom

On Wed, 1 Nov 2023 at 15:35, Lorenzo Bianconi <lorenzo@kernel.org> wrote:
>
> > PCI VID 0x0b48 doesn't appear to be listed anywhere I can find, though
> > a website named pcilookup.com suggests it belongs to TechnoTrend AG. I
> > think it's more likely to be part of AMD RZ600 series - rebranded
> > mediatek devices. Not sure how any of this might be verified.
>
> I have just found it here: include/linux/pci_ids.h
> but I am not sure if it is related. Anyway:
>
> Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>
>
> >
> > On Wed, 1 Nov 2023 at 15:00, Lorenzo Bianconi <lorenzo@kernel.org> wrote:
> > >
> > > > Add VID/PID 0b48/7922 for MediaTek MT7922 wifi chip.
> > > > Change tested on Redmi Book Pro 15 2023 (Fedora 38).
> > > >
> > > > Signed-off-by: Tom Waldron <tom@baremetal.dev>
> > > > ---
> > > >  drivers/net/wireless/mediatek/mt76/mt7921/pci.c | 2 ++
> > > >  1 file changed, 2 insertions(+)
> > > >
> > > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > > > index 3dda84a93..ba805cb02 100644
> > > > --- a/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > > > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci.c
> > > > @@ -21,6 +21,8 @@ static const struct pci_device_id mt7921_pci_device_table[] = {
> > > >               .driver_data = (kernel_ulong_t)MT7921_FIRMWARE_WM },
> > > >       { PCI_DEVICE(PCI_VENDOR_ID_MEDIATEK, 0x0616),
> > > >               .driver_data = (kernel_ulong_t)MT7922_FIRMWARE_WM },
> > > > +     { PCI_DEVICE(0x0b48, 0x7922),
> > > > +             .driver_data = (kernel_ulong_t)MT7922_FIRMWARE_WM },
> > >
> > > is PCI VID 0x0b48 PCI_VENDOR_ID_ITTIM?
> > >
> > > Regards,
> > > Lorenzo
> > >
> > > >       { },
> > > >  };
> > > >
> > > > --
> > > > 2.41.0
> > > >