On Tue, Jun 23, 2020 at 4:33 PM Luca Ceresoli luca@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@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_pi...
Thanks!
-- Luca