Forgot to CC ---------- Forwarded message --------- From: Brandon Maier brandon.maier@rockwellcollins.com Date: Tue, Jun 23, 2020 at 12:16 PM Subject: Re: [PATCH] arm64: zynqmp: Support converting pm_cfg_obj.c To: Michal Simek michal.simek@xilinx.com

On Tue, Jun 23, 2020 at 11:11 AM Michal Simek michal.simek@xilinx.com wrote:

On 22. 06. 20 22:45, Brandon Maier wrote:

...

Add a config to set the file format, but leave the default as binary type for backwards compatibility.

Based on description I can't see the reason for it. pmu config object should be aligned with pmu not with u-boot itself.

What I mean is, for example if there is a bug in zynqmp_pm_cfg_obj_convert then the pm_cfg binary will automatically be updated the next time U-Boot is built. Compared to if the pm_cfg_obj.bin is stored separately and has to be independently updated for a bug. But it's a minor issue and not that important.

Also using this script is just one way how to get pmufw config object. I personally don't use it but I can't see any issue to be in u-boot project if others want to use it.

We've been trying to piece together the scripts to build pmu config from an HDF using Luca's blog posts and other source code. Out of curiosity, what are these other methods?

Thanks, Michal