U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Simon Glass <sjg@chromium.org>
Cc: u-boot@lists.denx.de, Alper Nebi Yasak <alpernebiyasak@gmail.com>,
	Aristo Chen <jj251510319013@gmail.com>,
	Bin Meng <bmeng.cn@gmail.com>, Bryan Brattlof <bb@ti.com>,
	Heinrich Schuchardt <xypron.glpk@gmx.de>,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	Marek Vasut <marek.vasut+renesas@mailbox.org>,
	Michal Simek <michal.simek@amd.com>,
	Neha Malcom Francis <n-francis@ti.com>,
	Peng Fan <peng.fan@nxp.com>,
	Philippe Reynes <philippe.reynes@softathome.com>,
	Quentin Schulz <quentin.schulz@cherry.de>,
	Raymond Mao <raymondmaoca@gmail.com>,
	Stefan Herbrechtsmeier <stefan.herbrechtsmeier@weidmueller.com>,
	Sughosh Ganu <sughosh.ganu@arm.com>,
	Tom Rini <trini@konsulko.com>,
	Wojciech Dubowik <Wojciech.Dubowik@mt.com>,
	Yannic Moog <y.moog@phytec.de>
Subject: Re: [PATCH 0/5] edison: Fix mainline boot and convert the image to binman
Date: Mon, 8 Jun 2026 11:31:34 +0300	[thread overview]
Message-ID: <aiZ95kbAQbS7sfBH@ashevche-desk.local> (raw)
In-Reply-To: <20260607230519.3022520-1-sjg@chromium.org>

On Sun, Jun 07, 2026 at 05:05:09PM -0600, Simon Glass wrote:
> Mainline U-Boot does not boot on the Intel Edison. The mask ROM enters
> the OS image 0x1000 above its load address, so U-Boot needs 4KB of zero
> padding ahead of it. That padding used to come from an alignment rule in
> board/intel/edison/config.mk, but the Kbuild-6.1 bump stopped that input
> being built, so u-boot.bin now starts with code at offset 0 and the ROM
> jumps into the middle of it.
> 
> This series moves the image build into binman: a new intel-osip entry
> generates the OSIP header (with the GPT-protective MBR the mask ROM
> needs) and a fill provides the 4KB gap, replacing the config.mk hack and
> restoring the boot. It also widens the DFU auto-probe window so an
> automated flash has time to start, adds the board to the sjg-lab CI and
> rewrites the documentation to separate a normal DFU update from xFSTK
> recovery.
> 
> Tested on real hardware: a bricked board recovers via xFSTK, takes a
> fresh U-Boot over DFU and boots the result.

I have got only patches 0-3, I will try to get them from lore.kernel.org.
So far I reviewed them and fully support the series, thanks!

-- 
With Best Regards,
Andy Shevchenko



      parent reply	other threads:[~2026-06-08  8:31 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-07 23:05 [PATCH 0/5] edison: Fix mainline boot and convert the image to binman Simon Glass
2026-06-07 23:05 ` [PATCH 1/5] binman: Add an entry type for the Intel OSIP header Simon Glass
2026-06-08  8:22   ` Andy Shevchenko
2026-06-07 23:05 ` [PATCH 2/5] edison: Widen the DFU auto-probe window for flashing U-Boot Simon Glass
2026-06-08  8:24   ` Andy Shevchenko
2026-06-07 23:05 ` [PATCH 3/5] x86: edison: Align U-Boot via binman to fix the boot Simon Glass
2026-06-07 23:47   ` Tom Rini
2026-06-08  8:29     ` Andy Shevchenko
2026-06-08 14:00       ` Tom Rini
2026-06-13 13:05         ` Simon Glass
2026-06-08  7:47   ` Ilias Apalodimas
2026-06-08  8:27   ` Andy Shevchenko
2026-06-07 23:05 ` [PATCH 4/5] CI: Add an sjg-lab job for the Intel Edison Simon Glass
2026-06-07 23:05 ` [PATCH 5/5] doc: edison: Rewrite the update instructions Simon Glass
2026-06-08  9:34   ` Andy Shevchenko
2026-06-08  8:31 ` Andy Shevchenko [this message]

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=aiZ95kbAQbS7sfBH@ashevche-desk.local \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=Wojciech.Dubowik@mt.com \
    --cc=alpernebiyasak@gmail.com \
    --cc=bb@ti.com \
    --cc=bmeng.cn@gmail.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=jj251510319013@gmail.com \
    --cc=marek.vasut+renesas@mailbox.org \
    --cc=michal.simek@amd.com \
    --cc=n-francis@ti.com \
    --cc=peng.fan@nxp.com \
    --cc=philippe.reynes@softathome.com \
    --cc=quentin.schulz@cherry.de \
    --cc=raymondmaoca@gmail.com \
    --cc=sjg@chromium.org \
    --cc=stefan.herbrechtsmeier@weidmueller.com \
    --cc=sughosh.ganu@arm.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    --cc=y.moog@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