Re: [PATCH v2 6/6] log: Convert log values to printf() if not enabled

2021-03-14 Thread Simon Glass
Hi Tom, On Sun, 14 Mar 2021 at 12:33, Tom Rini wrote: > > On Wed, Jan 20, 2021 at 08:10:57PM -0700, Simon Glass wrote: > > > At present if logging not enabled, log_info() becomes a nop. But we want > > log output at the 'info' level to be akin to printf(). Update the macro to > > pass the output

Re: [PATCH v2 6/6] log: Convert log values to printf() if not enabled

2021-03-14 Thread Tom Rini
On Wed, Jan 20, 2021 at 08:10:57PM -0700, Simon Glass wrote: > At present if logging not enabled, log_info() becomes a nop. But we want > log output at the 'info' level to be akin to printf(). Update the macro to > pass the output straight to printf() in this case. > > This mimics the behaviour f

Re: [PATCH v2 6/6] log: Convert log values to printf() if not enabled

2021-02-04 Thread Simon Glass
Hi Sean, On Thu, 4 Feb 2021 at 18:28, Sean Anderson wrote: > > On 1/20/21 10:10 PM, Simon Glass wrote: > > At present if logging not enabled, log_info() becomes a nop. But we want > > log output at the 'info' level to be akin to printf(). Update the macro to > > pass the output straight to printf

Re: [PATCH v2 6/6] log: Convert log values to printf() if not enabled

2021-02-04 Thread Sean Anderson
On 1/20/21 10:10 PM, Simon Glass wrote: At present if logging not enabled, log_info() becomes a nop. But we want log output at the 'info' level to be akin to printf(). Update the macro to pass the output straight to printf() in this case. This mimics the behaviour for the log_...() macros like l

[PATCH v2 6/6] log: Convert log values to printf() if not enabled

2021-01-20 Thread Simon Glass
At present if logging not enabled, log_info() becomes a nop. But we want log output at the 'info' level to be akin to printf(). Update the macro to pass the output straight to printf() in this case. This mimics the behaviour for the log_...() macros like log_debug() and log_info(), so we can drop