Attention is currently required from: Kapil Porwal, Subrata Banik, Tarun Tuli, Wonkyu Kim.
Hello Kapil Porwal, Subrata Banik, Tarun Tuli, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/75714?usp=email
to look at the new patch set (#3).
Change subject: [TEST ONLY] add VPRO features control by fw config ......................................................................
[TEST ONLY] add VPRO features control by fw config
Define keylocker, tme fw_config bits(24-25) field KEYLOCKER 24 option KEYLOCKER_EN 0 option KEYLOCKER_DIS 1 end field TME 25 option TME_EN 0 option TME_DIS 1 end
Vpro features Disable: 3XXXXXX (XXXXXX is existing fw_config value) Vpro features Enable : XXXXXX (XXXXXX is existing fw_config value)
ex)proto1 SKU1 fw_config: 0x1561 0x1561 => Vpro features(keylocker/tme) are enabled 0x3001561 => Vpro features(keylocker/tme) are disabled
localhost ~ # cbmem -c | grep fw_config ... [DEBUG] keylocker is disabled by fw_config ... [DEBUG] TME is disabled by fw config
How to read FW config From EC console, type "cbi" to read FW config ex) 23-05-17 23:33:33.229 ec:~$ cbi ... 23-05-17 23:41:33.248 FW_CONFIG: 5473 (0x1561) ...
How to write FW config From EC consone, type "cbi set 6 <FW config value> 4 ex) 23-05-17 23:43:42.320 ec:~$ cbi set 6 0x3001561 4
Signed-off-by: Wonkyu Kim wonkyu.kim@intel.com Change-Id: I746ba80279dc69d06ac9e06bfeae068e31d23694 --- M src/cpu/intel/common/common_init.c M src/mainboard/google/rex/variants/rex0/Makefile.inc A src/mainboard/google/rex/variants/rex0/cpulib.c M src/mainboard/google/rex/variants/rex0/overridetree.cb M src/soc/intel/common/block/cpu/cpulib.c M src/soc/intel/common/block/include/intelblocks/cpulib.h 6 files changed, 74 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/75714/3