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 <[email protected]> > --- > > 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
signature.asc
Description: PGP signature
