Wonkyu Kim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42788 )
Change subject: soc/intel/tigerlake: Add SkipCpuReplacementCheck to chip options ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42788/2/src/soc/intel/tigerlake/chi... File src/soc/intel/tigerlake/chip.h:
https://review.coreboot.org/c/coreboot/+/42788/2/src/soc/intel/tigerlake/chi... PS2, Line 369: /* : * Enable(0)/Disable(1) CPU Replacement check. : * Default 0. Setting this to 1 skips checking CPU replacement. : */ : uint8_t SkipCpuReplacementCheck; I think most of time it should be skipped in platform. Is there any specific case we need to disable? I think it may requires RVP case which has CPU socket. In that case, can we use DisableSkipCpuReplacementCheck so that we don't need to change in each devicetree change?