[PATCH v3 0/6] Fix handling of optional blobs in binman

[Yannic Moog]

Hi Tim,

Am Donnerstag, dem 14.08.2025 um 12:54 -0700 schrieb Tim Harvey:
> On Thu, Jun 26, 2025 at 10:57 AM Tom Rini <trini at konsulko.com> wrote:
> > 
> > On Fri, 13 Jun 2025 14:02:39 +0200, Yannic Moog wrote:
> > 
> > > This series solves a contradiction regarding ext blobs packaged in
> > > binman. When they are marked as optional, by default they are faked, two
> > > messages are emitted. One says the image is not functional the other
> > > says the image is still functional. Both concern the same binman
> > > entry/blob.
> > > 
> > > Binman is set up to have fake external blobs in case they are missing.
> > > This is regardless on whether they are optional or not.
> > > The implementation does not allow different types of entries to override
> > > the faking decision; at least there wouldn't be much sense in doing so.
> > > 
> > > [...]
> > 
> > Applied to u-boot/next, thanks!
> > 
> > [1/6] binman: drop "faked" return value from check_fake_fname
> >       commit: a657d87f08789c4c50423897bd5ee052c6223ee6
> > [2/6] binman: mark optional missing blobs as absent
> >       commit: 3ae668b9d2101d111d257ee7d8912c19cfb0e102
> > [3/6] binman: rework dropping absent entries from packaged image
> >       commit: 21bc3433a43d3add3430543c9eee2a95d4dee2f6
> > [4/6] binman: add faked optional entry case in CheckForProblems
> >       commit: f5817e05efd39dd455c2e6a0f0622d709fe7ea2d
> > [5/6] binman: ftest: pass allow_fake_blob to _DoReadFileDtb
> >       commit: e749d64b4622806072c56a839cf9a36a5081b575
> > [6/6] binman: test: assert optional blobs don't cause non-functionality
> >       commit: 0dbb0a8d1f2aefb2a7fdd8e627edd955e4fe8d3a
> > --
> > Tom
> > 
> > 
> 
> Hi Yannic,
> 
> I haven't had time to dig into it in detail yet but it looks like
> imx8m boards are failing again in master due to tee changes again and
> perhaps its this series that is causing it.

Thanks for reporting,

> 
> Here's what I see with today's mater:
> make imx8mm_evk_defconfig
> make
> ...
>   BINMAN  .binman_stamp
> Image 'image' is missing optional external blobs but is still functional: tee-
> os
> 
> binman: module 'importlib.resources' has no attribute 'files'
> make: *** [Makefile:1339: .binman_stamp] Error 1
> 
> I thought I had read something about a warning that wasn't an error
> but above is clearly a build failure.
> 
> Am I missing something here?

My initial gut feeling says it could be related to this series: 
https://lore.kernel.org/u-boot/20250701-pkg_resoures-removal-v3-1-270188b335ae@phytec.de/

Maybe the combination of the 2 series causes this. Anyway, I'll investigate and
report back.

Yannic
> 
> Best Regards,
> 
> Tim