[External] Re: [PATCH] arm64: zynqmp: Support converting pm_cfg_obj.c

[Brandon Maier]

On Tue, Jun 23, 2020 at 4:33 PM Luca Ceresoli <luca at lucaceresoli.net> wrote:
>
> Hi,
>
> On 23/06/20 23:23, Luca Ceresoli wrote:
> > Hi Brandon,
> >
> > On 23/06/20 19:08, Brandon Maier wrote:
> >> On Tue, Jun 23, 2020 at 10:18 AM Luca Ceresoli <luca at lucaceresoli.net> wrote:
> >>>
> >> I'm trying to get it to work with Buildroot. Buildroot does have a
> >> package for uboot-tools, so I could use that to run it. I'd have to
> >> move the config and build commands into Buildroot's uboot package.
> >> Since the script belongs to U-Boot anyway, it seemed cleaner to
> >> integrate it directly into U-Boot so it could be used by other build
> >> systems too.
> >
> > I understand. But I also understand Michal's concern: the U-Boot
> > makefiles are quite complex already, and this feature is a relatively
> > minor improvement.
> >
> > Probably having the conversion in Buildroot could be even simpler as you
> > can support _only_ the C format there and do the conversion
> > unconditionally. Only one workflow, no ifs, simpler code. Don't forget
> > to Cc: me if you send a patch for that, I'll be glad to review it.

Either U-Boot or Buildroot is fine by me. If Yocto alreads does it in
their U-Boot package then I can see that being a precedent for
patching Buildroot. I'll drop this patch to U-Boot then.

>
> I almost forgot: I wrote some notes on how this could be implemented in
> yocto a while back. The principles are pretty much the same so you might
> be interested in reading them:
>
> https://urldefense.proofpoint.com/v2/url?u=https-3A__www.yoctoproject.org_pipermail_meta-2Dxilinx_2019-2DNovember_004578.html&d=DwICaQ&c=ilBQI1lupc9Y65XwNblLtw&r=bIwUnEkCqKFQQ0RVQaaY0gBWY7SIAhmiWLyMS82_mSU&m=RYp4WeqFQ03XpIUkriybKoxAFXTNGqdNLAgzp4S2S8k&s=MswLT0A5A80afFFIfBit21JLH37rUpN4gBC_ACxPw6k&e=
>

Thanks!

>
> --
> Luca