Hi Yannic,
On Thu, Jul 31, 2025 at 9:34 AM Yannic Moog <y.m...@phytec.de> wrote:
>
> binman can omit packaging an optional blob when it is missing.
> This allows us to not bother with config options.
> The core challenge is the interaction between tf-a and OP-TEE where
> U-Boot/binman does not know whether tf-a was built with SPD=opteed or
> without. This is important because tf-a might jump into the void when no
> optee_os is present, leading to boot failure. Thus by marking it
> optional, user is prompted to recheck (due to the warning message)
> whether they really have the right combination of tf-a and optee.
>
> Note that after this commit a warning will be printed when optee is not
> present for a binman build.
>
> Image 'image' is missing optional external blobs but is still functional:
> tee-os
>
> /binman/section/fit/images/tee/tee-os (tee.bin):
> See the documentation for your board. You may need to build Open Portable
> Trusted Execution Environment (OP-TEE) and build with TEE=/path/to/tee.bin
>
> Signed-off-by: Yannic Moog <y.m...@phytec.de>
> ---
> Due to a bug in binman, we had to guard binman tee.bin with OPTEE config
> as builds would error when tee.bin was not present in path; Even though
> optee_os was marked as optional in the binman tree.
> Since the bug has been resolved in
> d4f61eae2ab7 ('Merge patch series "Fix handling of optional blobs in
> binman"'),
> we can mark it optional again without getting build errors.
>
> I remember Fabio disliking that a warning would be emitted in builds, so
> I marked this as RFC.
>
> @Fabio what is your opinion now?
It looks good to me. Thanks
