Bill XIE has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31255 )
Change subject: cpu/intel/model_1067x: Check for lock bit on IA32_FEATURE_CONTROL ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/#/c/31255/1/src/cpu/intel/model_1067x/mp_init.c File src/cpu/intel/model_1067x/mp_init.c:
https://review.coreboot.org/#/c/31255/1/src/cpu/intel/model_1067x/mp_init.c@... PS1, Line 63: ia32_ft_ctrl It seems supposed to be ia32_ft_ctrl.lo.
https://review.coreboot.org/#/c/31255/1/src/cpu/intel/model_1067x/mp_init.c@... PS1, Line 69: ia32_ft_ctrl It seems supposed to be ia32_ft_ctrl.lo.