On Thu, 03 Sep 2026 12:58:03 +0530, Gokul Praveen <[email protected]> wrote: > This fragment enables a boot mode for K3 platforms where the R5 SPL > directly loads U-Boot proper, bypassing the A Core SPL stage > > The R5 config modifies the payload filename to load tinospl.bin > instead of tispl.bin. The A Core config updates the U-Boot load > address to match the A Core SPL load address, allowing R5 SPL to > load U-Boot directly from that location since A Core SPL is not > present in this boot flow. > > This reduces boot time by eliminating the A Core SPL stage.
I'm not a huge fan of adding single line config fragments, if the change really is that trivial it should better go in the documentation or just be easily exposed via Kconfig. > > > diff --git a/configs/k3_a72_nospl.config b/configs/k3_a72_nospl.config > new file mode 100644 > index 000000000000..7522acab950a > --- /dev/null > +++ b/configs/k3_a72_nospl.config > @@ -0,0 +1 @@ > +CONFIG_TEXT_BASE=0x80080000 If I understand correctly this is done to match TFA's PRELOADED_BL33_BASE. This is a valid approach but we could just use CONFIG_SPL_TEXT_BASE instead while building tinospl.bin as the load address for U-Boot, check my comment on the next patch. > > diff --git a/configs/k3_r5_nospl.config b/configs/k3_r5_nospl.config > new file mode 100644 > index 000000000000..94931bf87988 > --- /dev/null > +++ b/configs/k3_r5_nospl.config > @@ -0,0 +1 @@ > +CONFIG_SPL_FS_LOAD_PAYLOAD_NAME="tinospl.bin" You could also just ask the user to rename 'tinospl.bin' to 'tispl.bin' which would allow us to have Non-SPL boot without changing the R5 SPL. NACKed-by: Anshul Dalal <[email protected]> -- Anshul Dalal <[email protected]>
