[U-Boot] [RFC PATCH v3 0/2] env: Make environment loading log more clear

Sam Protsenko semen.protsenko at linaro.org
Thu Jul 19 22:28:41 UTC 2018


This is another attempt to make boot log better, without hackery this
time. Basically here we just remove unwanted error messages, relying on
the message from most deep API to be printed (like mmc subsystem). This
approach *might* have its shortcomings, though:
 1. With no "Failed" message, at some point we *can* end up with no
    error messages printed at all
 2. Removing some collateral error messages *may* lead to loss of useful
    debug info in other use-cases (env_load() is not only user of those
    APIs).

That being said, at the moment this looks like most clean solution to
cluttered log problem, as any other solution either will be hackish or
will require some big architectural changes. If one of mentioned
shortcomings occur, we can fix it ad hoc.

With this patch set applied we will see something like this:

    Loading Environment from FAT... MMC: no card present
    Loading Environment from MMC... OK

instead of:

    Loading Environment from FAT... MMC: no card present
    ** Bad device mmc 0 **
    Failed (-5)
    Loading Environment from MMC... OK


Sam Protsenko (2):
  env: Don't print "Failed" error message
  disk: part: Remove redundant error message

 disk/part.c | 1 -
 env/env.c   | 4 +---
 2 files changed, 1 insertion(+), 4 deletions(-)

-- 
2.18.0



More information about the U-Boot mailing list