On Mon, Aug 18, 2014 at 12:24:06PM -0600, Simon Glass wrote:
> On 18 August 2014 01:16, Thierry Reding <[email protected]> wrote:
> > From: Thierry Reding <[email protected]>
> >
> > This macro can be overridden in source files (before including common.h)
> > and can be used to specify a prefix for debug and error messages. An
> > example of how to use this is shown below:
> >
> >         #define pr_fmt(fmt) "foo: " fmt
> >
> >         #include <common.h>
> >
> >         ...
> >         debug("bar");
> >
> > The resulting message will read:
> >
> >         foo: bar
> >
> > Signed-off-by: Thierry Reding <[email protected]>
> 
> Acked-by: Simon Glass <[email protected]>
> 
> Seems OK. But I wonder if a string might be simpler?
> 
> #define DEBUG_PREFIX "foo: "
> 
> +#ifndef DEBUG_PREFIX
> +#define DEBUG_PREFIX ""
> +#endif

The advantage of using the macro is that it can perform more complicated
transformations on the format string, like this for example:

        #define pr_fmt(fmt) "foo: %s():" fmt, __func__

Thierry

Attachment: pgp9t0FUc7mVF.pgp
Description: PGP signature

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to