Re: [PATCH wireless-next v2] wifi: brcmfmac: remove 43752 SDIO incorrectly labelled as Cypress chip
Gokul Sivakumar <[email protected]> Thu, 24 Jul 2025 15:31:54 +0530
| Newsgroups | dev.linux.lists.brcm80211,org.kernel.vger.linux-wireless |
|---|---|
| Message-ID | <[email protected]> |
On 07/24, Arend van Spriel wrote:
> On 7/24/2025 11:15 AM, Johannes Berg wrote:
> > On Thu, 2025-07-24 at 14:16 +0530, Gokul Sivakumar wrote:
> > > Cypress(Infineon) is not the vendor for this 43752 SDIO WLAN chip, and so
> > > has not officially released any firmware binary for it. It is incorrect to
> > > maintain this WLAN chip with firmware vendor ID as "CYW". So relabel the
> > > chip as "WCC" as suggested by the maintainer.
> > >
> > > Fixes: d2587c57ffd8 ("brcmfmac: add 43752 SDIO ids and initialization")
> > > Fixes: f74f1ec22dc2 ("wifi: brcmfmac: add support for Cypress firmware api")
> > > Signed-off-by: Gokul Sivakumar <[email protected]>
> > > ---
> > >
> > > v2:
> > > * Relabelled the 43752-SDIO chip from CYW to WCC instead of dropping the
> > > support for this chip as suggested by Arend.
> > >
> >
> > Maybe update the subject now?
>
> That would be nice. Please not that the vendor ID is more about the
> firmware API that should be used for the device and not so much about
> maintainers.
>
Will send the patch again after rephrasing the subject to indicate that the
incorrect firmware Vendor ID (FWVID) is fixed by this patch.
Gokul