Re: [PATCH v2 0/9] brcmfmac: add support for BCM4359 SDIO chipset
From: Soeren Moch
Date: Thu Dec 12 2019 - 05:59:34 EST
On 12.12.19 10:42, Kalle Valo wrote:
> Soeren Moch <smoch@xxxxxx> writes:
>
>> Add support for the BCM4359 chipset with SDIO interface and RSDB support
>> to the brcmfmac wireless network driver in patches 1-7.
>>
>> Enhance devicetree of the RockPro64 arm64/rockchip board to use an
>> AP6359SA based wifi/bt combo module with this chipset in patches 8-9.
>>
>>
>> Chung-Hsien Hsu (1):
>> brcmfmac: set F2 blocksize and watermark for 4359
>>
>> Soeren Moch (5):
>> brcmfmac: fix rambase for 4359/9
>> brcmfmac: make errors when setting roaming parameters non-fatal
>> brcmfmac: add support for BCM4359 SDIO chipset
>> arm64: dts: rockchip: RockPro64: enable wifi module at sdio0
>> arm64: dts: rockchip: RockPro64: hook up bluetooth at uart0
>>
>> Wright Feng (3):
>> brcmfmac: reset two D11 cores if chip has two D11 cores
>> brcmfmac: add RSDB condition when setting interface combinations
>> brcmfmac: not set mbss in vif if firmware does not support MBSS
>>
>> .../boot/dts/rockchip/rk3399-rockpro64.dts | 50 +++++++++++---
>> .../broadcom/brcm80211/brcmfmac/bcmsdh.c | 8 ++-
>> .../broadcom/brcm80211/brcmfmac/cfg80211.c | 68 +++++++++++++++----
>> .../broadcom/brcm80211/brcmfmac/chip.c | 54 ++++++++++++++-
>> .../broadcom/brcm80211/brcmfmac/chip.h | 1 +
>> .../broadcom/brcm80211/brcmfmac/pcie.c | 2 +-
>> .../broadcom/brcm80211/brcmfmac/sdio.c | 17 +++++
>> include/linux/mmc/sdio_ids.h | 2 +
>> 8 files changed, 176 insertions(+), 26 deletions(-)
> Just to make sure we are on the same page, I will apply patches 1-7 to
> wireless-drivers-next and patches 8-9 go to some other tree? And there
> are no dependencies between the brcmfmac patches and dts patches?
>
Yes, this also is my understanding. I'm glad if you are fine with
patches 1-7.
Heiko will pick up patches 8-9 later for linux-rockchip independently.
And if we need another round of review for patches 8-9, I think we don't
need to bother linux-wireless with this.
Thanks,
Soeren