Re: [PATCH v5 3/6] log: output for CONFIG_LOG=n
On 4/22/20 9:01 PM, Tom Rini wrote: > On Wed, Apr 22, 2020 at 09:00:02PM +0200, Marek Vasut wrote: >> On 2/26/20 9:48 PM, Heinrich Schuchardt wrote: >>> If CONFIG_LOG=n, we should still output errors, warnings, notices, infos, >>> and for DEBUG=1 also debug messages. >>> >>> Signed-off-by: Heinrich Schuchardt >>> Reviewed-by: Simon Glass >> >> NAK >> >> This patch grows SoCFPGA SPL size by more than 6 bytes, and considering >> the recent discussion, I believe it was made very clear that any growth >> in SPL size is not allowed. I am surprised that this was not caught earlier. >> >> So, please revert this patch. >> >> I see the following with socfpga_cyclone5_defconfig >> >>textdata bss dec hex filename >> - 522351916 16 54167d397 spl/u-boot-spl >> + 523191916 16 54251d3eb spl/u-boot-spl > > Bugfix of previously important prints being missing. So, can you please clarify what the rule for acceptable SPL growth is?
Re: [PATCH v5 3/6] log: output for CONFIG_LOG=n
On Wed, Apr 22, 2020 at 09:00:02PM +0200, Marek Vasut wrote: > On 2/26/20 9:48 PM, Heinrich Schuchardt wrote: > > If CONFIG_LOG=n, we should still output errors, warnings, notices, infos, > > and for DEBUG=1 also debug messages. > > > > Signed-off-by: Heinrich Schuchardt > > Reviewed-by: Simon Glass > > NAK > > This patch grows SoCFPGA SPL size by more than 6 bytes, and considering > the recent discussion, I believe it was made very clear that any growth > in SPL size is not allowed. I am surprised that this was not caught earlier. > > So, please revert this patch. > > I see the following with socfpga_cyclone5_defconfig > >textdata bss dec hex filename > - 522351916 16 54167d397 spl/u-boot-spl > + 523191916 16 54251d3eb spl/u-boot-spl Bugfix of previously important prints being missing. Fix your patch and quit trolling please. -- Tom signature.asc Description: PGP signature
Re: [PATCH v5 3/6] log: output for CONFIG_LOG=n
On 2/26/20 9:48 PM, Heinrich Schuchardt wrote: > If CONFIG_LOG=n, we should still output errors, warnings, notices, infos, > and for DEBUG=1 also debug messages. > > Signed-off-by: Heinrich Schuchardt > Reviewed-by: Simon Glass NAK This patch grows SoCFPGA SPL size by more than 6 bytes, and considering the recent discussion, I believe it was made very clear that any growth in SPL size is not allowed. I am surprised that this was not caught earlier. So, please revert this patch. I see the following with socfpga_cyclone5_defconfig textdata bss dec hex filename - 522351916 16 54167d397 spl/u-boot-spl + 523191916 16 54251d3eb spl/u-boot-spl > --- > v5: > no change > v4: > no change > --- > include/log.h | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/include/log.h b/include/log.h > index 62fb8afbd0..0453876001 100644 > --- a/include/log.h > +++ b/include/log.h > @@ -115,11 +115,11 @@ static inline int _log_nop(enum log_category_t cat, > enum log_level_t level, > #define log_io(_fmt...) log(LOG_CATEGORY, LOGL_DEBUG_IO, ##_fmt) > #else > #define _LOG_MAX_LEVEL LOGL_INFO > -#define log_err(_fmt...) log_nop(LOG_CATEGORY, LOGL_ERR, ##_fmt) > -#define log_warning(_fmt...) log_nop(LOG_CATEGORY, LOGL_WARNING, ##_fmt) > -#define log_notice(_fmt...) log_nop(LOG_CATEGORY, LOGL_NOTICE, ##_fmt) > -#define log_info(_fmt...)log_nop(LOG_CATEGORY, LOGL_INFO, ##_fmt) > -#define log_debug(_fmt...) log_nop(LOG_CATEGORY, LOGL_DEBUG, ##_fmt) > +#define log_err(_fmt, ...) printf(_fmt, ##__VA_ARGS__) > +#define log_warning(_fmt, ...) printf(_fmt, ##__VA_ARGS__) > +#define log_notice(_fmt, ...)printf(_fmt, ##__VA_ARGS__) > +#define log_info(_fmt, ...) printf(_fmt, ##__VA_ARGS__) > +#define log_debug(_fmt, ...) debug(_fmt, ##__VA_ARGS__) > #define log_content(_fmt...) log_nop(LOG_CATEGORY, \ > LOGL_DEBUG_CONTENT, ##_fmt) > #define log_io(_fmt...) log_nop(LOG_CATEGORY, LOGL_DEBUG_IO, > ##_fmt) > -- > 2.25.0 >
[PATCH v5 3/6] log: output for CONFIG_LOG=n
If CONFIG_LOG=n, we should still output errors, warnings, notices, infos, and for DEBUG=1 also debug messages. Signed-off-by: Heinrich Schuchardt Reviewed-by: Simon Glass --- v5: no change v4: no change --- include/log.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/include/log.h b/include/log.h index 62fb8afbd0..0453876001 100644 --- a/include/log.h +++ b/include/log.h @@ -115,11 +115,11 @@ static inline int _log_nop(enum log_category_t cat, enum log_level_t level, #define log_io(_fmt...)log(LOG_CATEGORY, LOGL_DEBUG_IO, ##_fmt) #else #define _LOG_MAX_LEVEL LOGL_INFO -#define log_err(_fmt...) log_nop(LOG_CATEGORY, LOGL_ERR, ##_fmt) -#define log_warning(_fmt...) log_nop(LOG_CATEGORY, LOGL_WARNING, ##_fmt) -#define log_notice(_fmt...)log_nop(LOG_CATEGORY, LOGL_NOTICE, ##_fmt) -#define log_info(_fmt...) log_nop(LOG_CATEGORY, LOGL_INFO, ##_fmt) -#define log_debug(_fmt...) log_nop(LOG_CATEGORY, LOGL_DEBUG, ##_fmt) +#define log_err(_fmt, ...) printf(_fmt, ##__VA_ARGS__) +#define log_warning(_fmt, ...) printf(_fmt, ##__VA_ARGS__) +#define log_notice(_fmt, ...) printf(_fmt, ##__VA_ARGS__) +#define log_info(_fmt, ...)printf(_fmt, ##__VA_ARGS__) +#define log_debug(_fmt, ...) debug(_fmt, ##__VA_ARGS__) #define log_content(_fmt...) log_nop(LOG_CATEGORY, \ LOGL_DEBUG_CONTENT, ##_fmt) #define log_io(_fmt...)log_nop(LOG_CATEGORY, LOGL_DEBUG_IO, ##_fmt) -- 2.25.0