[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