Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45765 )
Change subject: console/init: Drop CONSOLE_LEVEL_CONST ......................................................................
Patch Set 1:
(1 comment)
https://qa.coreboot.org/job/coreboot-gerrit/143008/ : UNSTABLE
So verstage lacks cmos_get_option(), but we are allowed to select that combination in Kconfig. What would be the correct semantics? I don't want to enable option support in something called `verstage` without asking :)
https://review.coreboot.org/c/coreboot/+/45765/1/src/console/init.c File src/console/init.c:
https://review.coreboot.org/c/coreboot/+/45765/1/src/console/init.c@49 PS1, Line 49: init_log_level();
I recall seeing something on `bootblock_southbridge_early_init` but I'm not sure if all platforms ha […]
AFAIR, most of the common option-table code ignores the upper bank (e.g. when restoring defaults, we copy up to 128B).
I've also checked the `cmos.layout` files. `debug_level` and the cheksum are always below 1024 (bits).