All of lore.kernel.org
 help / color / mirror / Atom feed
From: Francesco Dolcini <francesco@dolcini.it>
To: Fabio Estevam <festevam@gmail.com>
Cc: "Francesco Dolcini" <francesco@dolcini.it>,
	"Tim Harvey" <tharvey@gateworks.com>,
	"João Paulo Gonçalves" <jpaulo.silvagoncalves@gmail.com>,
	"Peng Fan" <peng.fan@oss.nxp.com>,
	"Stefano Babic" <sbabic@nabladev.com>,
	"NXP i.MX U-Boot Team" <uboot-imx@nxp.com>,
	"Tom Rini" <trini@konsulko.com>,
	u-boot@lists.denx.de, "Alice Guo" <alice.guo@nxp.com>,
	"Ye Li" <ye.li@nxp.com>, "Peng Fan" <peng.fan@nxp.com>,
	"João Paulo Gonçalves" <joao.goncalves@toradex.com>
Subject: Re: [PATCH] arm: imx: imx9: scmi: Fix booting from USB
Date: Mon, 28 Jul 2025 18:44:41 +0200	[thread overview]
Message-ID: <20250728164441.GA15074@francesco-nb> (raw)
In-Reply-To: <CAOMZO5DT=ua+vCX5OQq6sx1nfcFjFY0S1hJ8OmNE1RWCQAmxJg@mail.gmail.com>

On Mon, Jul 28, 2025 at 11:25:04AM -0300, Fabio Estevam wrote:
> On Mon, Jul 28, 2025 at 7:29 AM Francesco Dolcini <francesco@dolcini.it> wrote:
> 
> > You worked on a similar topic, for imx8m, some time ago, commit
> > d175982c207b ("imx8mn/8mp: Allow booting via USB"). What's your opinion
> > here? How should we fix this?
> 
> The solution on the commit you mentioned is related to the
> i.MX8MN/i.MX8MP boot ROM implementation, and it works there.
> 
> What confuses me about João's patch is that it was not tested against
> the mainline U-Boot.
> 
> Can U-Boot mainline boot over serial download on i.MX95?
> 
> Adding Tim, as he has been trying to get USB boot to work with i.MX95.
> 
> I don't have access to any i.MX95 board to try it myself, so I
> appreciate some help here.

The issue here is not specific about USB, this is about code assuming
that the relevant env config options are enabled without checking
anything AND assuming that ENVL_NOWHERE is working as a default in any
case.

See also env_locations in env/env.c and arch_env_get_location()

Francesco


  reply	other threads:[~2025-07-28 16:45 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-22 14:04 [PATCH] arm: imx: imx9: scmi: Fix booting from USB João Paulo Gonçalves
2025-07-24  3:38 ` Peng Fan
2025-07-24 11:28   ` João Paulo Gonçalves
2025-07-28 10:29     ` Francesco Dolcini
2025-07-28 14:25       ` Fabio Estevam
2025-07-28 16:44         ` Francesco Dolcini [this message]
2025-07-30  6:35           ` Frieder Schrempf
2025-07-31 18:47             ` João Paulo Gonçalves
2025-08-05 23:07               ` Fabio Estevam
2025-08-06  6:53               ` Frieder Schrempf
2025-08-22 17:05                 ` Tim Harvey
2025-08-25  7:50                   ` Frieder Schrempf
2025-08-25 16:27                     ` Tim Harvey

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=20250728164441.GA15074@francesco-nb \
    --to=francesco@dolcini.it \
    --cc=alice.guo@nxp.com \
    --cc=festevam@gmail.com \
    --cc=joao.goncalves@toradex.com \
    --cc=jpaulo.silvagoncalves@gmail.com \
    --cc=peng.fan@nxp.com \
    --cc=peng.fan@oss.nxp.com \
    --cc=sbabic@nabladev.com \
    --cc=tharvey@gateworks.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-imx@nxp.com \
    --cc=ye.li@nxp.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.