From: Tom Rini <trini@konsulko.com>
To: Daniel Golle <daniel@makrotopia.org>
Cc: Simon Glass <sjg@chromium.org>,
Dario Binacchi <dario.binacchi@amarulasolutions.com>,
Michael Trimarchi <michael@amarulasolutions.com>,
Frieder Schrempf <frieder.schrempf@kontron.de>,
Quentin Schulz <quentin.schulz@cherry.de>,
Marek Vasut <marek.vasut+renesas@mailbox.org>,
"Lucien.Jheng" <lucienzx159@gmail.com>,
Casey Connolly <casey.connolly@linaro.org>,
Michal Simek <michal.simek@amd.com>,
Johan Jonker <jbx6244@gmail.com>,
Aristo Chen <aristo.chen@canonical.com>,
Nora Schiffer <nora.schiffer@ew.tq-group.com>,
Rasmus Villemoes <rv@rasmusvillemoes.dk>,
Weijie Gao <weijie.gao@mediatek.com>, Peng Fan <peng.fan@nxp.com>,
Vincent Jardin <vjardin@free.fr>,
Neil Armstrong <neil.armstrong@linaro.org>,
Randolph Sapp <rs@ti.com>,
Miquel Raynal <miquel.raynal@bootlin.com>,
Richard Genoud <richard.genoud@bootlin.com>,
u-boot@lists.u-boot-project.org, openwrt-devel@lists.openwrt.org
Subject: Re: [PATCH 0/9] boot: add imagemap on-demand image loading from storage
Date: Mon, 24 Aug 2026 16:59:20 -0600 [thread overview]
Message-ID: <20260824225920.GO523975@bill-the-cat> (raw)
In-Reply-To: <cover.1787272661.git.daniel@makrotopia.org>
[-- Attachment #1: Type: text/plain, Size: 1076 bytes --]
On Sun, Aug 23, 2026 at 08:12:43PM +0100, Daniel Golle wrote:
> This series adds "imagemap", a small layer for reading FIT images
> directly from storage on demand instead of copying the whole image
> into RAM first. It keeps a translation table of the byte ranges
> already loaded, so a header probe, signature verification and the
> final load can reuse a single read, and it hooks transparently into
> fit_image_load() so all existing verification and decompression paths
> run unchanged for both RAM- and storage-backed images.
As I was looking at the patches, I saw at least one checkpatch.pl
alignment place to fix, so please cover those in v2.
Conceptually, I'm OK with this. I need to throw the series at a world
build before/after and see what the fallout is on not-enabled platforms.
I'm assuming that once this gets where I can throw Coverity at it, it
will complain about a few things too.
Before posting v2, please throw this though CI too, I'm pretty sure I
saw a documentation style problem that would blow up there. Thanks!
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2026-08-24 22:59 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-23 19:12 [PATCH 0/9] boot: add imagemap on-demand image loading from storage Daniel Golle
2026-08-23 19:13 ` [PATCH 1/9] mtd: bind an mtd_blk device for non-NAND MTD masters Daniel Golle
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:13 ` [PATCH 2/9] mtd: nand: sandbox: forbid sub-page writes Daniel Golle
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:13 ` [PATCH 3/9] cmd: ubi: create a ubi_blk device when attaching UBI Daniel Golle
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:13 ` [PATCH 4/9] mtd: nand: spi: drop the eager ubi_blk binding Daniel Golle
2026-08-23 19:13 ` [PATCH 5/9] boot: add imagemap on-demand loading from storage Daniel Golle
2026-08-24 22:55 ` Tom Rini
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:14 ` [PATCH 6/9] boot: fit: support on-demand loading in fit_image_load() Daniel Golle
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:14 ` [PATCH 7/9] configs: sandbox: enable imagemap and its storage backends Daniel Golle
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:16 ` [PATCH 8/9] test: boot: add imagemap unit tests Daniel Golle
2026-09-28 15:12 ` Simon Glass
2026-08-23 19:16 ` [PATCH 9/9] doc: imagemap: document on-demand loading framework Daniel Golle
2026-09-28 15:17 ` Simon Glass
2026-08-24 22:59 ` Tom Rini [this message]
2026-09-28 15:13 ` [0/9] boot: add imagemap on-demand image loading from storage Simon Glass
2026-09-29 0:32 ` Daniel Golle
2026-10-01 17:23 ` Simon Glass
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=20260824225920.GO523975@bill-the-cat \
--to=trini@konsulko.com \
--cc=aristo.chen@canonical.com \
--cc=casey.connolly@linaro.org \
--cc=daniel@makrotopia.org \
--cc=dario.binacchi@amarulasolutions.com \
--cc=frieder.schrempf@kontron.de \
--cc=jbx6244@gmail.com \
--cc=lucienzx159@gmail.com \
--cc=marek.vasut+renesas@mailbox.org \
--cc=michael@amarulasolutions.com \
--cc=michal.simek@amd.com \
--cc=miquel.raynal@bootlin.com \
--cc=neil.armstrong@linaro.org \
--cc=nora.schiffer@ew.tq-group.com \
--cc=openwrt-devel@lists.openwrt.org \
--cc=peng.fan@nxp.com \
--cc=quentin.schulz@cherry.de \
--cc=richard.genoud@bootlin.com \
--cc=rs@ti.com \
--cc=rv@rasmusvillemoes.dk \
--cc=sjg@chromium.org \
--cc=u-boot@lists.u-boot-project.org \
--cc=vjardin@free.fr \
--cc=weijie.gao@mediatek.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.