[coreboot-gerrit] Change in coreboot[master]: console: Unify overriding console log level with get_option()

[Johnny Lin (Code Review)]

Johnny Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45134 )

Change subject: console: Unify overriding console log level with get_option()
......................................................................

Patch Set 11:

(3 comments)

https://review.coreboot.org/c/coreboot/+/45134/10/src/include/option.h 
File src/include/option.h:

https://review.coreboot.org/c/coreboot/+/45134/10/src/include/option.h@26 
PS10, Line 26: 		return cmos_get_option(dest, name);
> Similarly, we could have here: […]

I added a generic mainboard_get_option() because mainboard may not choose to use VPD.

https://review.coreboot.org/c/coreboot/+/45134/10/src/include/option.h@30 
PS10, Line 30: 		return get_console_override_option(dest, name);
> I don't understand why this needs another function. We could also have the […]

Done

https://review.coreboot.org/c/coreboot/+/45134/10/src/include/option.h@32 
PS10, Line 32: }
> If we decide for a normalized […]

Done