brcmfmac: support BCM4365E with 43666 ChipCommon chip ID

From: Rafał Miłecki <rafal@milecki.pl>

This adds support for the BCM43666/4 which seems to be using the same
firmware as BCM4366 (4366c0). I found it in the Netgear R8000P router.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
 drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c       | 1 +
 drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c       | 1 +
 drivers/net/wireless/broadcom/brcm80211/include/brcm_hw_ids.h | 1 +
 3 files changed, 3 insertions(+)

On 10-12-2020 20:08, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>

> 

> This adds support for the BCM43666/4 which seems to be using the same

> firmware as BCM4366 (4366c0). I found it in the Netgear R8000P router.


minor nit below...

Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>

> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>

> ---

>   drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c       | 1 +

>   drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c       | 1 +

>   drivers/net/wireless/broadcom/brcm80211/include/brcm_hw_ids.h | 1 +

>   3 files changed, 3 insertions(+)


[...]

> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

> index 45bc502fcb34..ff3721b20a9f 100644

> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

> @@ -77,6 +77,7 @@ static const struct brcmf_firmware_mapping brcmf_pcie_fwnames[] = {

>   	BRCMF_FW_ENTRY(BRCM_CC_4366_CHIP_ID, 0x0000000F, 4366B),

>   	BRCMF_FW_ENTRY(BRCM_CC_4366_CHIP_ID, 0xFFFFFFF0, 4366C),

>   	BRCMF_FW_ENTRY(BRCM_CC_43664_CHIP_ID, 0xFFFFFFF0, 4366C),

> +	BRCMF_FW_ENTRY(BRCM_CC_43666_CHIP_ID, 0x00000010, 4366C),


Our default approach is to enable it for all future revisions as was 
done for the two entries before this one.

>   	BRCMF_FW_ENTRY(BRCM_CC_4371_CHIP_ID, 0xFFFFFFFF, 4371),

>   };


-- 
This electronic communication and the information and any files transmitted 
with it, or attached to it, are confidential and are intended solely for 
the use of the individual or entity to whom it is addressed and may contain 
information that is confidential, legally privileged, protected by privacy 
laws, or otherwise restricted from disclosure to anyone else. If you are 
not the intended recipient or the person responsible for delivering the 
e-mail to the intended recipient, you are hereby notified that any use, 
copying, distributing, dissemination, forwarding, printing, or copying of 
this e-mail is strictly prohibited. If you received this e-mail in error, 
please return the e-mail to the sender, delete it from your computer, and 
destroy any printed copy of it.

On 14.12.2020 10:30, Arend van Spriel wrote:
> On 10-12-2020 20:08, Rafał Miłecki wrote:

>> From: Rafał Miłecki <rafal@milecki.pl>

>>

>> This adds support for the BCM43666/4 which seems to be using the same

>> firmware as BCM4366 (4366c0). I found it in the Netgear R8000P router.

> 

> minor nit below...

> 

> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>

>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>

>> ---

>>   drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c       | 1 +

>>   drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c       | 1 +

>>   drivers/net/wireless/broadcom/brcm80211/include/brcm_hw_ids.h | 1 +

>>   3 files changed, 3 insertions(+)

> 

> [...]

> 

>> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

>> index 45bc502fcb34..ff3721b20a9f 100644

>> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

>> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

>> @@ -77,6 +77,7 @@ static const struct brcmf_firmware_mapping brcmf_pcie_fwnames[] = {

>>       BRCMF_FW_ENTRY(BRCM_CC_4366_CHIP_ID, 0x0000000F, 4366B),

>>       BRCMF_FW_ENTRY(BRCM_CC_4366_CHIP_ID, 0xFFFFFFF0, 4366C),

>>       BRCMF_FW_ENTRY(BRCM_CC_43664_CHIP_ID, 0xFFFFFFF0, 4366C),

>> +    BRCMF_FW_ENTRY(BRCM_CC_43666_CHIP_ID, 0x00000010, 4366C),

> 

> Our default approach is to enable it for all future revisions as was done for the two entries before this one.

> 

>>       BRCMF_FW_ENTRY(BRCM_CC_4371_CHIP_ID, 0xFFFFFFFF, 4371),

>>   };


Thanks!



 > This electronic communication and the information and any files transmitted with it, or attached to it, are confidential and are intended solely for the use of the individual or entity to whom it is addressed and may contain information that is confidential, legally privileged, protected by privacy laws, or otherwise restricted from disclosure to anyone else. If you are not the intended recipient or the person responsible for delivering the e-mail to the intended recipient, you are hereby 

notified that any use, copying, distributing, dissemination, forwarding, printing, or copying of this e-mail is strictly prohibited. If you received this e-mail in error, please return the e-mail to the sender, delete it from your computer, and destroy any printed copy of it.

This is probably unintended for public mailing lists :)