Re: [Buildroot] [PATCH 1/1] boot/uboot/uboot.mk: support binary pm_cfg_obj

[Luca Ceresoli via buildroot <buildroot@buildroot.org>]

Hello Brandon,

thanks for your patch.

On Fri,  9 Dec 2022 13:40:43 -0600
Brandon Maier <brandon.maier@collins.com> wrote:

> BR2_TARGET_UBOOT_ZYNQMP_PM_CFG only works with C files, as it always
> tries to process them through tools/zynqmp_pm_cfg_obj_convert.py. Rework
> the logic so if the pm_cfg_obj isn't a C file, it is provided directly
> to U-Boot.

Can you describe a use case where this would be useful? While I
intuitively understand there _might_ be some, I cannot find any good
example based on my experience.

Building the pm_cfg_obj from source code is very easily done by
Buildroot, unlike building the PMUFW. Additionally one PMUFW binary
can be used on multiple zynqmp hardware, while the cfg obj is very
likely to change across different projects, and a somewhat readable
text file is way more understandable than a binary blob.

> This mimics changes done to the ZYNQMP_PMUFW which had a similar issue.

That is different: we now support two different binary formats, not
binary-vs-source, and detection is based on the file extension.

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot