[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Add option to control microcode update inclusion

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37514 )

Change subject: soc/intel/skylake: Add option to control microcode update inclusion
......................................................................

Patch Set 3: Code-Review-1

(1 comment)

The current approach implies that users can't change that behavior without editing Kconfig files. I'd rather have an option in menuconfig to only include microcode updates for a specific CPUID, that can be used with any mainboard. Those who squeeze Linux kernels into flash chips next to coreboot like extra space, so even desktop boards with socketed CPUs could use this idea.

https://review.coreboot.org/c/coreboot/+/37514/1/src/soc/intel/skylake/Kconf... 
File src/soc/intel/skylake/Kconfig:

https://review.coreboot.org/c/coreboot/+/37514/1/src/soc/intel/skylake/Kconf... 
PS1, Line 292: SkyLake
> Done

It's actually "Skylake"

https://ark.intel.com/content/www/us/en/ark/products/codename/37572/skylake....