[coreboot-gerrit] Change in coreboot[master]: mb/clevo/kbl-u: Install Intel VGA INT15 handler

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43852 )

Change subject: mb/clevo/kbl-u: Install Intel VGA INT15 handler
......................................................................

Patch Set 11:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43852/11/src/mainboard/clevo/kbl-u/... 
File src/mainboard/clevo/kbl-u/ramstage.c:

https://review.coreboot.org/c/coreboot/+/43852/11/src/mainboard/clevo/kbl-u/... 
PS11, Line 21: mainboard_enable
> Most boards use this name here, I would +2 a change that renames all of them at once

Coreboot coding style guide says "use descriptive names". `mainboard_enable` is maybe some sort of descriptive but simply wrong. "enable mainboard" -> "enable_mainboard". Just because XYZ does this, it's not inherently correct.