[PATCH 1/6] bloblist: add blob type for DT overlay

[Raymond Mao]

Hi Tom,

On Wed, 18 Jun 2025 at 13:37, Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Jun 18, 2025 at 07:59:23AM -0700, Raymond Mao wrote:
>
> > Add blob type for DT overlay according to the update of Firmware
> > Handoff spec[1].
> >
> > [1] https://github.com/FirmwareHandoff/firmware_handoff/pull/74
> >
> > Signed-off-by: Raymond Mao <raymond.mao at linaro.org>
>
> Once merged, this should be a Link: line for tooling to better pick it
> up. And I don't want to merge this here until it is merged / agreed to
> in FirmwareHandoff.
>
> > diff --git a/include/bloblist.h b/include/bloblist.h
> > index f32faf78560..b40e04fe2d3 100644
> > --- a/include/bloblist.h
> > +++ b/include/bloblist.h
> > @@ -110,7 +110,8 @@ enum bloblist_tag_t {
> >       BLOBLISTT_ACPI_TABLES = 4,
> >       BLOBLISTT_TPM_EVLOG = 5,
> >       BLOBLISTT_TPM_CRB_BASE = 6,
> > -     BLOBLISTT_ACPI_PP = 7,
> > +     BLOBLISTT_FDT_OVERLAY = 7,
> > +     BLOBLISTT_ACPI_PP = 8,
>
> ... sigh, so BLOBLISTT_ACPI_PP isn't something in FirmwareHandoff yet.
> Was someone working on that it should it be moved down to the project
> internal numbering area?
Since we decided to make bloblist compliant to Firmware Handoff, it is
required to get an ID placeholder in the spec first (at least a PR)
before updating the code to avoid conflicts, but BLOBLISTT_ACPI_PP
does not exist in the spec. In this case, I will consider it as a test
only ID and can be shifted. Or I can move it into the custom range if
you think that is better.

Regards,
Raymond


>
> --
> Tom