[coreboot-gerrit] Change in coreboot[master]: console: Allow VPD to disable an otherwise enabled coreboot console

[Marc Jones (Code Review)]

Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45208 )

Change subject: console: Allow VPD to disable an otherwise enabled coreboot console
......................................................................

Patch Set 4:

Do we need to put VPD into the console code? There is already an override for get_console_loglevel() for mainboards to set the level. It seems it could be set via cmos, vpd, gpio, bmc, cbfs file, or other mechanisms.

A standard would be good, but putting everything in VPD doesn't seem right, either. These these types of optional user settings don't really need RO/RW. It isn't like a serial # or something that needs protection.