On Thu, Dec 19, 2024 at 08:07:22AM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 17 Dec 2024 at 16:54, Tom Rini <[email protected]> wrote: > > > > On Tue, Dec 17, 2024 at 04:42:30PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Tue, 17 Dec 2024 at 12:53, Tom Rini <[email protected]> wrote: > > > > > > > > On Tue, Dec 17, 2024 at 12:45:29PM -0700, Simon Glass wrote: > > > > > Hi Tom, > > > > > > > > > > On Tue, 17 Dec 2024 at 07:20, Tom Rini <[email protected]> wrote: > > > > > > > > > > > > On Tue, Dec 17, 2024 at 06:26:16AM -0700, Simon Glass wrote: > > > > > > > > > > > > > At present buildman only supports building the default target. > > > > > > > Generally > > > > > > > this is what is wanted, but in some cases boards erroneously have > > > > > > > a > > > > > > > different target for product extra files. > > > > > > > > > > > > > > Add a --target option to help. Also add a comment indicating which > > > > > > > letters are free for new options. > > > > > > > > > > > > > > Signed-off-by: Simon Glass <[email protected]> > > > > > > > > > > > > How do you use this? It would be nice to be able to do "--target > > > > > > u-boot-initial-env" for example for converting and verifying the > > > > > > conversion of boards to plain text environment. Except the list of > > > > > > artifacts that are kept isn't updated with that string as well. And > > > > > > FWIW, is this for a case not solved by CONFIG_BUILD_TARGET? > > > > > > > > > > It's related to the problem where colibri-imx8 doesn't actually build > > > > > an image with the default boot target. > > > > > > > > I don't recall if there's a good reason for imx8 not setting > > > > CONFIG_BUILD_TARGET to something, but that sounds like how to solve this > > > > problem. "make all" should work, normally. > > > > > > Even if it did, the build fails the first time. It just needs to be fixed/ > > > > Yes, there's both the problem of CONFIG_BUILD_TARGET not being set and > > then whatever issues there are with configuring access to the blobs > > especially when they exist and aren't faked via BINMAN_ALLOW_MISSING. > > IIRC Toradex does weekly builds of top of tree, via OpenEmbedded / Yocto > > so there's not some bitrot here to be fixed, just smoothing out > > integration. > > To me, BUILD_TARGET is a pre-Binman construction and we should just > use binman to make sure that the default target builds a bootable > image.
Well, no. Assuming that doc/board/toradex/colibri-imx8x.rst is correct then CONFIG_BUILD_TARGET should be set to "u-boot-dtb.imx" so that "make" just works like other platforms. If there's some other target actually in use, that should be what's put in CONFIG_BUILD_TARGETS and in turn binman will get invoked as needed. -- Tom
signature.asc
Description: PGP signature
