[PATCH v1 1/2] hexdump: Introduce debug APIs
andriy.shevchenko at linux.intel.com
Wed Mar 30 11:30:20 CEST 2022
On Sat, Nov 13, 2021 at 11:14:28AM -0700, Simon Glass wrote:
> On Tue, 9 Nov 2021 at 05:03, Andy Shevchenko
> <andriy.shevchenko at linux.intel.com> wrote:
> > debug_hex_dump() and debug_hex_dump_bytes() conditionally print
> > the dump based on DEBUG definition.
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > ---
> > include/hexdump.h | 4 ++++
> > lib/hexdump.c | 60 ++++++++++++++++++++++++++++++++++++++++-------
> > 2 files changed, 56 insertions(+), 8 deletions(-)
> +Heinrich Schuchardt
Heinrich, it's several months passed. What should we do?
> Is it possible to resolve this in the header file? It is a bit odd to
> have to add DEBUG to hexdump.c in order to get debug output.
> Note also the logging system so you can do
> log(LOG_CATEGORY, dbg ? LOGL_DEBUG : LOGL_INFO, "fmt ...", ...)
With Best Regards,
More information about the U-Boot