All of 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: 15+ 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-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 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.