[coreboot-gerrit] [M] Change in coreboot[master]: cpu/intel/microcode: Have CPU microcode per CPUID into CBFS

[Arthur Heymans (Code Review)]

Attention is currently required from: Himanshu Sahdev, Julius Werner, Lean Sheng Tan, Rizwan Qureshi, Subrata Banik, Tarun Tuli, Wonkyu Kim.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75357?usp=email )

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: cpu/intel/microcode: Have CPU microcode per CPUID into CBFS
......................................................................

Patch Set 18:

(1 comment)

File src/cpu/Kconfig:

https://review.coreboot.org/c/coreboot/+/75357/comment/ce261704_c7b41645 :
PS14, Line 123: bool "Include microcode per CPUID into CBFS"
> > > > This option will show on ALL platforms. Does it even make sense to have it as a user visible option?
> > >
> > > it will allow to override the default value from site-local rather forcing to enable from SoC/MB config using upstream coreboot.
> > >
> > > As we are not uploading the ucode blobs in coreboot upstream build hence it's easy to go by default value which is `n` and don't need to specify the split ucode blob path.
> >
> > @Arthur, ping! can you please take a relook.
>
> marking this resolved for now after keeping open for 1 week

It's not about default values. If I run make menuconfig and select a mainboard I should not be able to see any option that makes no sense.