Hi Massimo,
On 9/8/26 7:04 PM, Massimo Pegorer wrote:
Just check that CONFIG_XPL_BUILD is not set to select files for U-Boot
proper build. Indeed, CONFIG_TPL_BUILD being set implies by design that
CONFIG_XPL_BUILD is set too, therefore CONFIG_XPL_BUILD not set implies
that CONFIG_TPL_BUILD is not set either, making its check needless.
I suspect we have a lot more than this one.
A simple git grep "CONFIG_XPL_BUILD.*CONFIG_[TS]PL_BUILD" returned a few
matches. And this is only for them being on the same line. It's not
necessarily a direct replacement, as we have CONFIG_VPL_BUILD which
could match too, but nothing (except sandbox) makes use of that stage at
the moment.
Also git grep "CONFIG_[TS]PL_BUILD.*CONFIG_XPL_BUILD" returns another match.
Fixes: bef9fdbed2e5 ("arch: Use CONFIG_XPL_BUILD instead of CONFIG_SPL_BUILD")
Signed-off-by: Massimo Pegorer <[email protected]>
Reviewed-by: Quentin Schulz <[email protected]>
Thanks!
Quentin