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

Reply via email to