On Fri, Jan 23, 2026 at 09:13:27PM -0300, Fabio Estevam wrote: > Hi Tom, > > On Fri, Jan 23, 2026 at 5:12 PM Tom Rini <[email protected]> wrote: > > > I've been talking with Ilias for the last few hours. This still isn't > > right, it just happens to work in some cases, but not others. What's > > going on is that while we always have u-boot-nodtb.bin 8-byte aligned > > because while the BSS itself may end 8 byte aligned, we actually care > > about the size of the BSS being 8 byte aligned. Can you please try: > > > > diff --git a/arch/arm/cpu/u-boot-spl.lds b/arch/arm/cpu/u-boot-spl.lds > > index d780a5060774..dae8a1ec665b 100644 > > --- a/arch/arm/cpu/u-boot-spl.lds > > +++ b/arch/arm/cpu/u-boot-spl.lds > > @@ -58,6 +58,7 @@ SECTIONS > > _end = .; > > > > .bss __rel_dyn_start (OVERLAY) : { > > + . = ALIGN(8); > > __bss_start = .; > > *(.bss*) > > . = ALIGN(8); > > > > And see if it works at run time? hexdump'ing looks good. If so, I'll do > > something more broad to catch the SPL_SEPARATE_BSS=n case, and add the > > ASSERTS that are missing. > > This fixes the boot on RV1103, thanks! > > When you send a formal patch, feel free to add: > > Reported-by: Fabio Estevam <[email protected]> > Tested-by: Fabio Estevam <[email protected]>
After talking more with Ilias, we're going to handle this in two-steps. Step one gets more platforms booting today. It's *slightly* different than what I posted here, however it's binary identical (I just checked and got the same checksum as this patch produces, build-testing sonoff-ihost-rv1126). So I'll add your tags to the commit. -- Tom
signature.asc
Description: PGP signature
