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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
