[PATCH v2 05/17] boot: Move showing of bootflows out of the command

Tom Rini trini at konsulko.com
Thu Oct 2 22:15:37 CEST 2025


On Wed, Oct 01, 2025 at 03:26:30PM -0600, Simon Glass wrote:

> It is helpful in tests to be able to show the bootflow that is being
> examined. Move show_bootflow() into boot/ and rename it.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
> Changes in v2:
> - Add a log_err() for an invalid state
> 
>  boot/bootflow.c    | 57 ++++++++++++++++++++++++++++++++++++++
>  cmd/bootflow.c     | 68 ++--------------------------------------------
>  include/bootflow.h |  9 ++++++
>  3 files changed, 69 insertions(+), 65 deletions(-)
[snip]
> +	case BOOTFLOWST_COUNT:
> +		log_err("Unexpected boot value of bootflow error %d",
> +			 bflow->state);

A small thing, checkpatch.pl catches that this isn't aligned with the '('
here as it should be.

A larger thing, and please correct me if I'm wrong, but on reading the
whole set of changes, this move + rename just means we're putting more
info in the test output, and nothing else?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20251002/1aab4173/attachment.sig>


More information about the U-Boot mailing list