[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