1 comment:
File src/mainboard/clevo/kbl-u/ramstage.c:
Patch Set #11, 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.
To view, visit change 43852. To unsubscribe, or for help writing mail filters, visit settings.