[coreboot-gerrit] Change in coreboot[master]: mainboard/intel/common: Add config for intel common mainboard support

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37705 )

Change subject: mainboard/intel/common: Add config for intel common mainboard support
......................................................................

Patch Set 5:
> Patch Set 5:
>
> Random thought: Couldn't this be factored out into drivers instead?

I initially had that thought. But some of the things here are more APIs than hardware management/configuration. I can see that the drivers enable configuring the device through device tree and export ACPI objects for use by kernel. I also thought about factoring into lib. But again the things here are more vendor/mainboard specific. Just my thoughts.