Hi Arend, Thanks for your patch. It works well. > +enum { > + BRCMF_DRVDATA_CYW, > + BRCMF_DRVDATA_BCA, > + BRCMF_DRVDATA_WCC, > + BRCMF_DRVDATA_WCC_SEED, > +}; I found that in drivers/net/wirless/broadcom/brcm80211/brcmfmac/bus.h has already prepared an enumeration like these. enum brcmf_fwvendor { BRCMF_FWVENDOR_WCC, BRCMF_FWVENDOR_CYW, BRCMF_FWVENDOR_BCA, /* keep last */ BRCMF_FWVENDOR_NUM, BRCMF_FWVENDOR_INVALID }; So should we delete them? Because you also changed BRCMF_FWVENDOR_ to BRCMF_DRVDATA_ --- Best Regards Jacobe