[U-Boot] [PATCH 0/9] log: Support control over the log output format

[Tom Rini]

On Wed, Jan 24, 2018 at 05:52:37AM -0700, Simon Glass wrote:
> Hi Tom,
> On 19 January 2018 at 09:20, Tom Rini <trini at 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 <sjg at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180124/ca95edfb/attachment.sig>