rsta2 / circle

A C++ bare metal environment for Raspberry Pi with USB (32 and 64 bit)
https://circle-rpi.readthedocs.io
GNU General Public License v3.0
1.8k stars 238 forks source link

Pi Zero 2 W: Different revisions, different WLAN firmwares #313

Open hpingel opened 1 year ago

hpingel commented 1 year ago

Hi Rene, Yesterday I noticed that two different PI Zero 2 W PCBs can have different WLAN hardware: These logger lines exist on different Pis:

wlan: ether4330: chip 43430 rev2 type 1 (my Pi) wlan: ether4330: chip 43430 rev1 type 1 (the Pi of a colleague) In the next line my colleague gets this: wlan: ether 4330: can't enable HT clock: csr 50 wlan: I/O error

Now, I was wondering if this error could be related to the WLAN firmware files. But it could also be something else like a power supply or a hardware fault. (EDIT: The colleague told me he didn't copy all the firmware files in the meantime. So his problem seems to be fixed.)

Today I came accross this ticket: https://github.com/RPi-Distro/firmware-nonfree/issues/23

I tried to understand if this is of any relevance to the issue above and am not convinced. But anyway, I wanted to let you know that there are some firmware files for the Zero 2 W in https://github.com/RPi-Distro/firmware-nonfree/tree/bullseye/debian/config/brcm80211/brcm that are not yet grabbed by wget from Circle's side.

The missing files are: brcmfmac43436s-sdio.bin brcmfmac43436s-sdio.txt

And there were some changes to some files after October 2021.

Cheers, Henning

P.S.: Also see: https://github.com/RPi-Distro/firmware-nonfree/commit/b3eec612566ca08913f0830d299f4df70297428f

hpingel commented 1 year ago

I just grasped that a firmware brcmfmac43436s-sdio.* will only be necessary for a future revision of the Pi Zero 2 W that is probably not yet available due to the global chip crisis anyway.

I just tested all Pi Zero 2 W boards I have here and they all have either rev1 or rev2.

rsta2 commented 1 year ago

The WLAN device driver has separate entries for revision 1 and 2 of the 43430 chip, so this difference should be handled well. As I understand it, there is currently no need to change something. But I will keep an eye on this and this issue will remain open. Thanks for this info!