public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Vitor Soares <ivitro@gmail.com>
To: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de, vitor.soares@toradex.com, n-francis@ti.com,
	 b-padhi@ti.com, afd@ti.com, anshuld@ti.com, festevam@denx.de
Subject: Re: Toradex Verdin AM62 boot failure on recent master
Date: Sat, 24 Jan 2026 16:36:17 +0000	[thread overview]
Message-ID: <06bd1e1cee6a2c3d8fc408330279e093c734f487.camel@gmail.com> (raw)
In-Reply-To: <20260123150129.GK3416603@bill-the-cat>

On Fri, 2026-01-23 at 09:01 -0600, Tom Rini wrote:
> On Thu, Jan 22, 2026 at 11:58:54PM +0000, Vitor Soares wrote:
> 
> > 
> > Hi,
> > 
> > I'm experiencing a boot failure on Toradex Verdin AM62 with the latest
> > master.
> > The last known good commit is a4dc1c3b7d56 and it fails to boot at
> > 64662b99c0eb
> > (current HEAD).
> > 
> > Looking at the 35 commits in this range, I noticed that the first commit
> > after
> > my last known good is 85f586035d75 ("ARM: OMAP2+: Pad SPL binary to 8-byte
> > alignment before DTB"), which directly modifies the K3/AM62 SPL linker
> > script.
> > 
> > Related commits:
> >    85f586035d75 ARM: OMAP2+: Pad SPL binary to 8-byte alignment before DTB
> >    8b0ebe054bb3 arm: Update linker scripts to ensure appended device tree is
> > aligned
> > 
> > Could these changes be related to the boot failure? Any insights would
> > be appreciated.
> 
> You likely need
> https://patchwork.ozlabs.org/project/uboot/patch/20260122172247.2995437-1-b-padhi@ti.com/
> 

The v3 patch alone didn't fix the boot issue on Verdin AM62.

However, Marek's suggested approach [1] that only aligns flat_dt type
images to 8 bytes does fix it:

diff --git a/tools/fit_image.c b/tools/fit_image.c
--- a/tools/fit_image.c
+++ b/tools/fit_image.c
@@ -685,6 +685,13 @@ static int fit_extract_data(...)
              const char *data, *type;
              int len;

+             /* Ensure flat_dt images are 8-byte aligned per DT spec */
+             if (align_size < 8) {
+                     type = fdt_getprop(fdt, node, FIT_TYPE_PROP, &len);
+                     if (type && len > 0 && !strncmp(type, "flat_dt", len < 8 ?
len : 7))
+                             align_size = 8;
+             }
+
              data = fdt_getprop(fdt, node, FIT_DATA_PROP, &len);

+Cc Fabio - this should resolve your boot failure as well.

[1]
https://lore.kernel.org/u-boot/1e1f2e09-bfb7-4c7d-abd6-25a79271f256@mailbox.org/

Best regards,
Vitor Soares




  reply	other threads:[~2026-01-24 18:33 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-22 23:58 Toradex Verdin AM62 boot failure on recent master Vitor Soares
2026-01-23 13:08 ` Fabio Estevam
2026-01-23 15:01 ` Tom Rini
2026-01-24 16:36   ` Vitor Soares [this message]
2026-01-24 18:35     ` Fabio Estevam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=06bd1e1cee6a2c3d8fc408330279e093c734f487.camel@gmail.com \
    --to=ivitro@gmail.com \
    --cc=afd@ti.com \
    --cc=anshuld@ti.com \
    --cc=b-padhi@ti.com \
    --cc=festevam@denx.de \
    --cc=n-francis@ti.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vitor.soares@toradex.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox