[Buildroot] [PATCH v1] package/rpi-wifi-firmware: provide convenience links for RPi3A+, RPi3B+ and RPi4B firmware configuration files

[Peter Seiderer <ps.report@gmx.net>]

Hello Andreas,

thanks for the patch iteration and improvement ;-), but some comments...

First as it is a new iteration 'PATCH v1' is no longer valid and should be 'PATCH v2'....,
same for patch subject as it is now missing the added RPi0W and RPi3B...

On Tue, 30 Mar 2021 10:01:45 +0200, Andreas Ziegler <br015@umbiko.net> wrote:

> Provide convenience links for BCM43455/CYW43455 based RPi3A+, RPi3B+ and RPi4B
> WIFI firmware configuration files to avoid runtime firmware loading warning in
> the kernel logs, e.g. (RPi3B+):
>
>   brcmfmac mmc1:0001:1: Direct firmware load for brcm/brcmfmac43455-sdio.raspberrypi,3-model-b-plus.txt failed with error -2
>
> or (RP4B
>
>   brcmfmac mmc1:0001:1: Direct firmware load for brcm/brcmfmac43455-sdio.raspberrypi,4-model-b.txt failed with error -2
>
> Changes:
>   added links for brcmfmac43430 based boards RPi [0w, 3b]

As you added some changes this should go to...

>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>

...here as follows:

[Added links for brcmfmac43430 based RPi0W and RPi3B

> Signed-off-by: Andreas Ziegler <br015@umbiko.net>

---
Changes v1 -> v2:
  - added links for brcmfmac43430 based RPi0W and RPi3B

> ---
>  package/rpi-wifi-firmware/rpi-wifi-firmware.mk | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/package/rpi-wifi-firmware/rpi-wifi-firmware.mk b/package/rpi-wifi-firmware/rpi-wifi-firmware.mk
> index 24ba5941b0..c3ab9be857 100644
> --- a/package/rpi-wifi-firmware/rpi-wifi-firmware.mk
> +++ b/package/rpi-wifi-firmware/rpi-wifi-firmware.mk
> @@ -12,6 +12,11 @@ RPI_WIFI_FIRMWARE_LICENSE_FILES = LICENCE.broadcom_bcm43xx
>  define RPI_WIFI_FIRMWARE_INSTALL_TARGET_CMDS
>  	$(INSTALL) -d $(TARGET_DIR)/lib/firmware/brcm
>  	$(INSTALL) -m 0644 $(@D)/firmware/brcm/brcmfmac* $(TARGET_DIR)/lib/firmware/brcm
> -endef
> +	ln -sf brcmfmac43430-sdio.txt $(TARGET_DIR)/lib/firmware/brcm/brcmfmac43430-sdio.raspberrypi-rpi.txt
> +	ln -sf brcmfmac43430-sdio.txt $(TARGET_DIR)/lib/firmware/brcm/brcmfmac43430-sdio.raspberrypi,3-model-b.txt

Did not add this two ones as I was not sure about it, because [1] states
RPi0W and RPi3B uses BCM43143, did not know (or test) it maps to 43430...,
I see ([2]) contains a brcmfmac43430-sdio.raspberrypi,3-model-b.txt file...,
but according to [3] I would expect 'raspberrypi,model-zero-w' for the RPi0W...

At least some commit log explanation/reference would be help full..., all in
all (as already stated in the other mail thread) an additional patch (based on
the real v1 of this one) would be easier to review/handle...

Regards,
Peter

[1] https://de.wikipedia.org/wiki/Raspberry_Pi
[2] https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/tree/brcm
[3] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/arm/bcm/bcm2835.yaml


> +	ln -sf brcmfmac43455-sdio.txt $(TARGET_DIR)/lib/firmware/brcm/brcmfmac43455-sdio.raspberrypi,3-model-a-plus.txt
> +	ln -sf brcmfmac43455-sdio.txt $(TARGET_DIR)/lib/firmware/brcm/brcmfmac43455-sdio.raspberrypi,3-model-b-plus.txt
> +	ln -sf brcmfmac43455-sdio.txt $(TARGET_DIR)/lib/firmware/brcm/brcmfmac43455-sdio.raspberrypi,4-model-b.txt
> +	endef
>
>  $(eval $(generic-package))