On Thu, Jan 29, 2026 at 11:26 PM Tom Rini <[email protected]> wrote:
>
> On Thu, Jan 29, 2026 at 11:25:03PM +0800, Sune Brian wrote:
> > On Thu, Jan 29, 2026 at 11:19 PM Tom Rini <[email protected]> wrote:
> > >
> > > On Thu, Jan 29, 2026 at 11:06:59PM +0800, Sune Brian wrote:
> > > > On Thu, Jan 29, 2026 at 10:54 PM Tom Rini <[email protected]> wrote:
> > > > >
> > > > > On Thu, Jan 29, 2026 at 10:58:21AM +0800, Sune Brian wrote:
> > > > > > On Wed, Jan 28, 2026 at 10:51 PM Tom Rini <[email protected]> 
> > > > > > wrote:
> > > > > > >
> > > > > > > On Wed, Jan 28, 2026 at 10:49:37PM +0800, Sune Brian wrote:
> > > > > > >
> > > > > > > > Hi Tom,
> > > > > > > >
> > > > > > > > The commits after c08da5d03c2a0b72e81a11ff7ca507e3a6414db3
> > > > > > > > All Cyclone V boards are unable to boot, not sure this patch is 
> > > > > > > > the fix?
> > > > > > >
> > > > > >
> > > > > > Hi Tom,
> > > > > >
> > > > > > > Yes, those commits were unintentional and have been reverted.
> > > > > >
> > > > > > Had tested the last commit 6a1bdb7e952d5841f42742fefa907cae5dc8d50a
> > > > > > Same situation that the boards are unable to enter the SPL phase.
> > > > > > The commit 8de6e8f8a076d2c9b6d38d8563db135c167077ec works properly.
> > > > > > After that things are broken.
> > > > >
> > > > > Please give
> > > > > https://patchwork.ozlabs.org/project/uboot/patch/[email protected]/
> > > > > a try as I think it might have been working before by alignment luck
> > > > > rather than design.
> > > >
> > > > Hi Tom,
> > > >
> > > > It is the revert itself that is broken.
> > > > I tested f4dfa5d3c2680322fd17fcc7c6221876e00a03c2
> > > >
> > > >
> > > > U-Boot SPL 2026.01-00754-gf4dfa5d3c268-dirty (Jan 29 2026 - 22:56:17 
> > > > +0800)
> > > > Trying to boot from MMC1
> > > >
> > > > Which revert is correct.
> > > >
> > > > But once dc2d8423b19d30472b02e02b41504226908a4291
> > > > And revert to 431f1ce46bbffff0db8f03437a34400b11b30175
> > > >
> > > > Then all these broken I don't understand why even file diff cannot
> > > > show any changes.
> > > >
> > > > Do you mean the revert is broken and fixed by alignment?
> > >
> > > I mean that arbitrary changes (as part of testing these things, the
> > > "-dirty" in the version string would lead to failing trees now passing)
> > > can make the difference between pass/fail to boot. Did you try Marek's
> > > patch I linked to?
> >
> > Yes, I had tried. Cannot boot same as I had mentioned failing commit #.
> > I apply that patch on the latest master commit.
>
> The other, maybe, thing is you need a bigger SPL_SYS_MALLOC_F_LEN ?
> Otherwise, you'll need to dig more and see why the device tree isn't
> being found where expected on your platform, and possibly where it is
> instead.

No the failing point is done after

f4dfa5d3c2680322fd17fcc7c6221876e00a03c2
Revert "arm: spl: Correct alignment of .rel.dyn section"
This reverts commit 380ddb4. Signed-off-by: Tom Rini <[email protected]>

This commit I tried to boot and works..

Then this commit

dc2d8423b19d30472b02e02b41504226908a4291
count rel_dyn
Signed-off-by: Tom Rini <[email protected]>

I manually replaced the old Makefile.xpl and also worked it boot.

Then 431f1ce46bbffff0db8f03437a34400b11b30175

Revert a number of incorrect commits
As part of debugging the appended device tree failure, I inadvertently
committed some changes as I was debugging to master, and not a private
branch, and pushed them as part of the release. This reverts commit
dc2d842 through 380ddb4. Signed-off-by: Tom Rini <[email protected]>

This is the point that everything no longer boots.

But between those commits there are no extra codes.

After that all things no longer work.

Thanks

>
> --
> Tom

Reply via email to