[Buildroot] [PATCH v2 1/8] package/freescale-imx/firmware-imx: bump version to 8.10

[Romain Naour <romain.naour@gmail.com>]

Hi Fabio,

Le 13/01/2021 ? 01:43, Fabio Estevam a ?crit?:
> Hi Romain,
> 
> On Tue, Jan 12, 2021 at 9:27 PM Fabio Estevam <festevam@gmail.com> wrote:
> 
>> I notice the same here on a imx53.
> 
> It works for me if I change it like this:
> 
> --- a/package/freescale-imx/firmware-imx/firmware-imx.mk
> +++ b/package/freescale-imx/firmware-imx/firmware-imx.mk
> @@ -116,9 +116,8 @@ endif
>  FIRMWARE_IMX_VPU_FW_NAME = $(call
> qstrip,$(BR2_PACKAGE_FIRMWARE_IMX_VPU_FW_NAME))
>  ifneq ($(FIRMWARE_IMX_VPU_FW_NAME),)
>  define FIRMWARE_IMX_INSTALL_TARGET_VPU_FW
> -       mkdir -p $(TARGET_DIR)/lib/firmware/imx/vpu
>         cp $(@D)/firmware/vpu/vpu_fw_$(FIRMWARE_IMX_VPU_FW_NAME)*.bin \
> -               $(TARGET_DIR)/lib/firmware/imx/vpu/
> +               $(TARGET_DIR)/lib/firmware/

Here you're installing the firmware in the first supported firmware path, this
allow to load the firmware on the first try.

>  endef
>  endif
> 
> 
> [    9.727308] coda 63ff4000.vpu: IRQ bit not found
> [    9.848458] coda 63ff4000.vpu: Initialized CODA7541.
> [    9.853967] coda 63ff4000.vpu: Firmware version: 1.4.50
> [    9.937682] coda 63ff4000.vpu: encoder registered as video5
> [    9.962135] coda 63ff4000.vpu: decoder registered as video6
> [    9.985276] coda 63ff4000.vpu: encoder registered as video7
> [   10.018420] coda 63ff4000.vpu: decoder registered as video8
> 
> Would the change above be acceptable? If so, I can submit it.

I'm not sure if we should install the vpu firmware directly in "/lib/firmware/"
or "/lib/firmware/vpu". Feel free to submit a patch to discuss about this change.

Thanks!

Best regards,
Romain

> 
> Thanks
>