[U-Boot] [PATCH v2 0/8] Sync and consolidate Linux-derived printk, BUILD_BUG, BUG, WARN, etc.
Masahiro Yamada
yamada.masahiro at socionext.com
Thu Oct 5 03:20:36 UTC 2017
Hi Tom,
2017-10-05 12:06 GMT+09:00 Tom Rini <trini at konsulko.com>:
> On Wed, Oct 04, 2017 at 02:15:19PM +0900, Masahiro Yamada wrote:
>> 2017-09-16 14:10 GMT+09:00 Masahiro Yamada <yamada.masahiro at socionext.com>:
>> >
>> > I tested this series with buildman.
>> >
>> >
>> >
>> > Masahiro Yamada (8):
>> > stdio.h: move printf() stuff from <common.h> to <stdio.h>
>> > printk: collect printk stuff into <linux/printk.h> with loglevel
>> > support
>> > treewide: replace with error() with pr_err()
>> > common.h: remove error()
>> > vsprintf.h: include <linux/types.h>
>> > bug.h: sync BUILD_BUG stuff with Linux 4.13
>> > bug.h: move runtime BUG/WARN macros into <linux/bug.h>
>> > dm: define dev_*() log functions in DM header
>>
>> I am still worried if this series is dismissed.
>>
>> I am being blocked from importing NAND code from Linux
>> due to missing/incompatible Linux-derived macros.
>
> I am looking at this, but a default LOGLEVEL of 5 is just too high and
> I'm seeing what's reasonable now.
>
No. I set the default to 6, not 5.
The reason why I chose 6 was to suppress a bunch of pr_info()
from drivers/mtd/nand/, which is a copy of Linux.
If you change it to a bigger number,
some annoying logs would be displayed after NAND: tag.
If you change it to a smaller number,
pr_notice() would be suppressed.
The Linux doc says:
5 (KERN_NOTICE) normal but significant condition
So, I think this is worth printing by default.
If you do not like it, you can change it,
but please make sure to adjust git-log.
--
Best Regards
Masahiro Yamada
More information about the U-Boot
mailing list