All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Carlo Caione" <ccaione@baylibre.com>
To: "Simon Glass" <sjg@chromium.org>, <ccaione@baylibre.com>
Cc: <u-boot@lists.u-boot-project.org>
Subject: Re: [v2,0/5] firmware-owned devicetree for EBBR / SystemReady IR
Date: Mon, 24 Aug 2026 16:48:36 +0200	[thread overview]
Message-ID: <DKX99AW58JTS.8UMKA33XJC4J@baylibre.com> (raw)
In-Reply-To: <CAFLszTgX08U3GpcvwkvzBinY7X21EC_P2nEZPx3ikVH=vBt_yA@mail.gmail.com>

On Sat Aug 15, 2026 at 8:45 PM CEST, Simon Glass wrote:
> Hi Carlo,
>
> On 2026-07-28T13:20:42, Carlo Caione <ccaione@baylibre.com> wrote:
>
>> Patch 1 makes an FDT passed to efi_bootmgr_run() outrank a Boot#### FDT,
>> as an independent behaviour fix. Patch 2 adds the loader, binding,
>> documentation and shared EFI staging policy. Patches 3 and 4 integrate
>> the EFI bootmeth and boot manager. Patch 5 adds sandbox coverage.
> [..]
>
>> - 'fw_fdt_part' can pin an A/B partition and 'fw_fdt_config' can select
>>   an explicit configuration.
>
> Just to check - is exposing these as environment variables really the
> right long-term interface? U-Boot has been moving configuration into
> the control devicetree, and both of these look like properties that
> would sit naturally on the u-boot,firmware-fdt-block node, with the
> env var overriding for A/B slot selection only. What do you think?

Hi Simon,
Replying only to this point because all the other comments have my ACK.

Agreed regarding fw_fdt_part: it is indeed intended as the runtime A/B
slot override, while the stable storage location remains described by
the provider phandle and partition UUID/name in the control devicetree.

Now, fw_fdt_config is slightly different though. We need configuration
selection to depend on the selected boot target. For example in our
usecase, the same firmware FIT may provide a base-only configuration
for a generic distribution and an overlay configuration for a platform
image. The control devicetree is identical in both cases, so a fixed
property there cannot express that choice.

Static selection is already covered by compatible best-match and
the FIT default. I would retain fw_fdt_config only as an explicit
runtime override above those mechanisms and clarify that role in the
documentation if that is ok with you.

Cheers,

--
Carlo Caione

  reply	other threads:[~2026-08-24 14:48 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-28 13:20 [PATCH v2 0/5] firmware-owned devicetree for EBBR / SystemReady IR Carlo Caione
2026-07-28 13:20 ` [PATCH v2 1/5] efi_loader: bootmgr: preserve a passed devicetree Carlo Caione
2026-08-04  8:51   ` Ilias Apalodimas
2026-08-09  0:34   ` Simon Glass
2026-07-28 13:20 ` [PATCH v2 2/5] boot: add a firmware-owned devicetree source Carlo Caione
2026-08-04  8:58   ` Ilias Apalodimas
2026-08-15 18:44   ` Simon Glass
2026-07-28 13:20 ` [PATCH v2 3/5] bootmeth: efi: use the firmware-owned devicetree Carlo Caione
2026-08-15 18:44   ` Simon Glass
2026-07-28 13:20 ` [PATCH v2 4/5] efi_loader: bootmgr: install " Carlo Caione
2026-08-15 18:44   ` Simon Glass
2026-07-28 13:20 ` [PATCH v2 5/5] test: boot: add firmware-FDT source tests Carlo Caione
2026-08-15 18:45   ` Simon Glass
2026-07-29  7:10 ` [PATCH v2 0/5] firmware-owned devicetree for EBBR / SystemReady IR Peter Robinson
2026-08-15 18:45 ` [v2,0/5] " Simon Glass
2026-08-24 14:48   ` Carlo Caione [this message]
2026-08-25 12:44     ` 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=DKX99AW58JTS.8UMKA33XJC4J@baylibre.com \
    --to=ccaione@baylibre.com \
    --cc=sjg@chromium.org \
    --cc=u-boot@lists.u-boot-project.org \
    /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.