2021-04-12 08:37:24

by Arend van Spriel

[permalink] [raw]
Subject: Re: [PATCH] brcmfmac: Add support for BCM43596 PCIe Wi-Fi

On 07-03-2021 12:35, Konrad Dybcio wrote:
> Add support for BCM43596 dual-band AC chip, found in
> SONY Xperia X Performance, XZ and XZs smartphones (and
> *possibly* other devices from other manufacturers).
> The chip doesn't require any special handling and seems to work
> just fine OOTB.
>
> PCIe IDs taken from: https://github.com/sonyxperiadev/kernel/commit/9e43fefbac8e43c3d7792e73ca52a052dd86d7e3.patch

I don't see 4359 firmware in linux-firmware repo so what are you using?

Regards,
Arend


Subject: Re: [PATCH] brcmfmac: Add support for BCM43596 PCIe Wi-Fi

Il 12/04/21 10:36, Arend van Spriel ha scritto:
> On 07-03-2021 12:35, Konrad Dybcio wrote:
>> Add support for BCM43596 dual-band AC chip, found in
>> SONY Xperia X Performance, XZ and XZs smartphones (and
>> *possibly* other devices from other manufacturers).
>> The chip doesn't require any special handling and seems to work
>> just fine OOTB.
>>
>> PCIe IDs taken from:
>> https://github.com/sonyxperiadev/kernel/commit/9e43fefbac8e43c3d7792e73ca52a052dd86d7e3.patch
>>
>
> I don't see 4359 firmware in linux-firmware repo so what are you using?
>
> Regards,
> Arend

Hi Arend,

we are using firmwares that come with our specific Sony devices, as we
couldn't find any generic one.
Pushing firmwares around is something that we tend to be careful about
because, as you know, they are usually covered with proprietary licenses
and such.

If anyone from Broadcom can help us by pushing "generic" firmwares for
this chip on linux-firmware, we would largely appreciate that.

Yours,
- Angelo

2021-05-21 07:21:35

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] brcmfmac: Add support for BCM43596 PCIe Wi-Fi

Hi,


sorry for the lack of answer from me (I've had a lot going on..), but the one Angelo has given you is right. We have to use the files that SONY ships with their stock Android firmware. Could you look into this patch once again?


Konrad