[PATCH v2 5/7] binman: Fix blank line usage for invalid images warning text

Simon Glass sjg at chromium.org
Wed Aug 2 20:07:53 CEST 2023


Hi Jonas,

On Tue, 18 Jul 2023 at 14:34, Jonas Karlman <jonas at kwiboo.se> wrote:
>
> There is no blank line between last missing blob help message and the
> header line for optional blob help messages.
>
>   Image 'simple-bin' is missing external blobs and is non-functional: atf-bl31
>
>   /binman/simple-bin/fit/images/@atf-SEQ/atf-bl31:
>      See the documentation for your board. You may need to build ARM Trusted
>      Firmware and build with BL31=/path/to/bl31.bin
>   Image 'simple-bin' is missing external blobs but is still functional: tee-os
>
>   /binman/simple-bin/fit/images/@tee-SEQ/tee-os:
>      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
>
>   Some images are invalid
>
> With this a blank line is inserted to make the text more readable.
>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
> v2:
> - Collect r-b tag

IMO you don't need to do this...just collect them and say 'no changes'.

'patman status' is your friend.

Regards,
Simon

Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list