From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brandon Maier Date: Tue, 23 Jun 2020 16:23:40 -0500 Subject: Fwd: [PATCH] arm64: zynqmp: Support converting pm_cfg_obj.c In-Reply-To: References: <20200622204531.11301-1-brandon.maier@rockwellcollins.com> <94593a88-5cd5-059f-7749-f3f9ae99728e@xilinx.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Forgot to CC ---------- Forwarded message --------- From: Brandon Maier Date: Tue, Jun 23, 2020 at 12:16 PM Subject: Re: [PATCH] arm64: zynqmp: Support converting pm_cfg_obj.c To: Michal Simek On Tue, Jun 23, 2020 at 11:11 AM Michal Simek 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