[Buildroot] [PATCH] package/linux-firmware: Add option to install cadence MHDP8546 DPI/DP bridge firmware

[Julien Olivain]

Hi Andy,

Thanks for the patch.  I have one comment.  See below.

On 01/09/2025 10:31, Andy Yan wrote:
> Cadence MHDP8546 DPTX IP used in TI J721E SoC needs the firmware to
> work.
> 
> Signed-off-by: Andy Yan <andyshrk at 163.com>
> ---
>  package/linux-firmware/Config.in         | 5 +++++
>  package/linux-firmware/linux-firmware.mk | 5 +++++
>  2 files changed, 10 insertions(+)
> 
> diff --git a/package/linux-firmware/Config.in 
> b/package/linux-firmware/Config.in
> index 778dfef8a0..21dff9768e 100644
> --- a/package/linux-firmware/Config.in
> +++ b/package/linux-firmware/Config.in
> @@ -205,6 +205,11 @@ config BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM4366C0
>  	help
>  	  Firmware file for the Broadcom 4366c0 FullMAC chip.
> 
> +config BR2_PACKAGE_LINUX_FIRMWARE_CDNS_MHDP8546
> +	bool "Cadence MHDP8546"
> +	help
> +	  Firmware file for the Cadence MHDP8546 DPI/DP bridge
> +
>  config BR2_PACKAGE_LINUX_FIRMWARE_CYPRESS_CYW43XX
>  	bool "Cypress CY cyw43xx"
>  	help
> diff --git a/package/linux-firmware/linux-firmware.mk 
> b/package/linux-firmware/linux-firmware.mk
> index 40758054e5..869e2d668f 100644
> --- a/package/linux-firmware/linux-firmware.mk
> +++ b/package/linux-firmware/linux-firmware.mk
> @@ -913,6 +913,11 @@ LINUX_FIRMWARE_FILES += ti_3410.fw ti_5052.fw \
>  LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.moxa
>  endif
> 
> +ifeq ($(BR2_PACKAGE_LINUX_FIRMWARE_CDNS_MHDP8546),y)
> +LINUX_FIRMWARE_FILES += cadence/mhdp8546.bin
> +LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.cadence

Adding a new licence file here also requires to add the new
file hash in "package/linux-firmware/linux-firmware.hash".

Running the command "make linux-firmware-legal-info"
with this firmware selected show the message:

     ERROR: No hash found for LICENCE.cadence

> +endif
> +
>  ifeq ($(BR2_PACKAGE_LINUX_FIRMWARE_CX231XX),y)
>  LINUX_FIRMWARE_FILES += v4l-cx231xx-avcore-01.fw
>  # No license file; the license is in the file WHENCE
> --
> 2.43.0

Could you send an updated patch adding the license hash, please?

Best regards,

Julien.