On Wed, Jan 24, 2018 at 05:52:37AM -0700, Simon Glass wrote: > Hi Tom, > On 19 January 2018 at 09:20, Tom Rini <tr...@konsulko.com> wrote: > > > > On Thu, Jan 18, 2018 at 01:40:51PM -0800, Simon Glass wrote: > > > Hi, > > > > > > On 28 December 2017 at 12:14, Simon Glass <s...@chromium.org> wrote: > > > > This adds a few more features to the log system: > > > > > > > > - 'log format' command to control the log output format > > > > - 'log rec' command to output a log record manually > > > > - log_ret() function to log error-return values > > > > > > > > With these we have more control over how log records are output as well > > > > as the ability to log errors more easily. > > > > > > > > > > > > Simon Glass (9): > > > > dm: core: Add a function to look up a uclass by name > > > > log: Add functions to convert IDs to/from names > > > > log: Add control over log formatting > > > > log: Update log_console to honour the log format > > > > log: Add a command to control the log output format > > > > log: Add a command to output a log record > > > > log: Add tests for the new log features > > > > log: Add documentation for commands and formatting > > > > log: Add a way to log error-return values > > > > > > > > cmd/log.c | 82 > > > > +++++++++++++++++++++++++++++++++++++++ > > > > common/Kconfig | 13 +++++++ > > > > common/log.c | 68 ++++++++++++++++++++++++++++++++ > > > > common/log_console.c | 27 ++++++++++++- > > > > configs/sandbox_defconfig | 1 + > > > > doc/README.log | 35 +++++++++++++++++ > > > > drivers/core/uclass.c | 14 +++++++ > > > > include/asm-generic/global_data.h | 1 + > > > > include/dm/uclass.h | 8 ++++ > > > > include/log.h | 64 +++++++++++++++++++++++++++++- > > > > test/dm/core.c | 9 +++++ > > > > test/py/tests/test_log.py | 32 +++++++++++++-- > > > > 12 files changed, 348 insertions(+), 6 deletions(-) > > > > > > Are there any comments / reviews on this series, please? I'd like to > > > apply it in this merge window. > > > > What's the size change in the cases where boards can't opt-out? > > There is no size change for boards which don't enable CONFIG_LOG. For > sandbox (which does) the change is: > > sandbox: (for 4/4 boards) all +970.0 data +136.0 rodata +212.0 text +622.0
OK, that's my main concern taken care of, thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
