public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Francesco Dolcini <francesco@dolcini.it>
To: Bryan Brattlof <bb@ti.com>, Anshul Dalal <anshuld@ti.com>,
	Devarsh Thakkar <devarsht@ti.com>,
	Francesco Dolcini <francesco@dolcini.it>,
	trini@konsulko.com
Cc: Suhaas Joshi <s-joshi@ti.com>,
	u-boot@lists.denx.de, vigneshr@ti.com, n-francis@ti.com,
	s-tripathi1@ti.com, k-malarvizhi@ti.com, kamlesh@ti.com,
	vishalm@ti.com, d.schultz@phytec.de, w.egorov@phytec.de,
	ggiordano@phytec.com
Subject: Re: [REGRESSION] Verdin AM62/62P not booting with 42b3ee7fa524
Date: Tue, 24 Feb 2026 15:38:06 +0100	[thread overview]
Message-ID: <20260224143806.GC340942@francesco-nb> (raw)
In-Reply-To: <20260224102121.GB340942@francesco-nb>

On Tue, Feb 24, 2026 at 11:21:23AM +0100, Francesco Dolcini wrote:
> On Mon, Feb 23, 2026 at 04:58:21PM +0100, Francesco Dolcini wrote:
> > On Fri, Feb 20, 2026 at 07:23:42AM -0600, Bryan Brattlof wrote:
> > > On February 20, 2026 thus sayeth Francesco Dolcini:
> > 
> > I was not able to verify this yet, but it seems that since commit
> > ba20b2443c29 ("arm: mach-k3: common: Reserve video memory from end of
> > the RAM") the ram top is overwritten and our
> > verdin-am62.c:board_get_usable_ram_top() is ignored :-(
> > 
> > So, the change from devarsh broke it, but before the addition of the
> > memory firewall the memory corruption was not leading to a crash ...
> 
> I can confirm that the issue is around the board_init_f() that is
> overridden on the k3 architecture and spl_enable_cache(). It messes up
> with gd->*ram* variables ignoring various config options, including
> verdin-am62.c:board_get_usable_ram_top(). Any comment on this?
> 
> Trying to get out of this regression, one solution would be to move the
> whole memory map as it was done on the pocket beagle2, as it was
> suggested a few email ago.
> 
> Or fix the spl_enable_cache() implementation.
> 
> Bryan, and TI folks, what's your suggestion on the next steps here?
> 
> In addition to that, no matter if I do a quick and dirty hack on
> spl_enable_cache(), or if I move the optee at the beginning of the ram,
> I get this error once every few resets
> 
> ```
> ...
> Authentication passed
> Authentication passed
> Loading Environment from nowhere... OK
> init_env from device 9 not supported!
> Warning: Did not detect image signing certificate. Skipping authentication to prevent boot failure.
> This will fail on Security Enforcing(HS-SE) devices
> ```
> 
> and after that a freeze. I am starting to think that this is another,
> unrelated issue.

Yes, I can confirm that this is another, but related issue :-/

There is a memory corruption, something not ok in get_ram_size(), cache
related, that is leading to a memory corruption and therefore these
random crashes.

For some reason the memory corruption is happening calling dram_init()
twice and with 512MB module and with commit cb41b7c83444 ("board:
toradex: Make A53 get RAM size from DT in K3 boards"), I did debug this
with plain v2026.01 + cb41b7c83444.

This https://lore.kernel.org/all/20250314100734.23777-1-eichest@gmail.com/
solves the issue, but it was not applied when it was first sent months
ago.

Tom: you were the one with some concern, from my point of view we
should just apply it, any concern?

On the other topic I am waiting for TI advise if they plan to fix
spl_enable_cache() or I would need to work on revising the memory map
for the various firmwares + U-Boot.

Francesco


  parent reply	other threads:[~2026-02-24 14:38 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-09  8:38 [REGRESSION] Verdin AM62/62P not booting with 42b3ee7fa524 Francesco Dolcini
2026-02-09  8:45 ` Francesco Dolcini
2026-02-09 10:08   ` Suhaas Joshi
2026-02-09 10:16     ` Francesco Dolcini
2026-02-09 17:25       ` Francesco Dolcini
2026-02-09 17:43         ` Tom Rini
2026-02-09 18:07           ` Francesco Dolcini
2026-02-09 18:13             ` Tom Rini
2026-02-10  9:26               ` Suhaas Joshi
2026-02-10 10:02                 ` Wadim Egorov
2026-02-10 10:22                   ` Wadim Egorov
2026-02-10 10:34                     ` Francesco Dolcini
2026-02-09 20:20         ` Bryan Brattlof
2026-02-10 10:50           ` Francesco Dolcini
2026-02-11 10:11             ` Francesco Dolcini
2026-02-11 11:21               ` Suhaas Joshi
2026-02-13 10:20                 ` Francesco Dolcini
2026-02-17 13:21                   ` Suhaas Joshi
2026-02-18  6:59                     ` Francesco Dolcini
2026-02-18 14:52                       ` Francesco Dolcini
2026-02-19  1:46                         ` Bryan Brattlof
2026-02-19 10:30                           ` Wadim Egorov
2026-02-19 10:40                           ` Suhaas Joshi
2026-02-19 19:30                             ` Francesco Dolcini
2026-02-20  1:05                               ` Bryan Brattlof
2026-02-20 10:46                                 ` Francesco Dolcini
2026-02-20 13:23                                   ` Bryan Brattlof
2026-02-23 15:58                                     ` Francesco Dolcini
2026-02-24 10:21                                       ` Francesco Dolcini
2026-02-24 13:53                                         ` devarsh
2026-02-24 14:38                                         ` Francesco Dolcini [this message]
2026-02-24 15:24                                           ` Francesco Dolcini

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=20260224143806.GC340942@francesco-nb \
    --to=francesco@dolcini.it \
    --cc=anshuld@ti.com \
    --cc=bb@ti.com \
    --cc=d.schultz@phytec.de \
    --cc=devarsht@ti.com \
    --cc=ggiordano@phytec.com \
    --cc=k-malarvizhi@ti.com \
    --cc=kamlesh@ti.com \
    --cc=n-francis@ti.com \
    --cc=s-joshi@ti.com \
    --cc=s-tripathi1@ti.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vigneshr@ti.com \
    --cc=vishalm@ti.com \
    --cc=w.egorov@phytec.de \
    /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