summaryrefslogtreecommitdiff
path: root/src/soc
diff options
context:
space:
mode:
authorBenjamin Doron <benjamin.doron00@gmail.com>2020-10-16 18:07:13 +0000
committerPatrick Georgi <pgeorgi@google.com>2020-10-19 07:00:32 +0000
commitdc667980f8ab78296e77b53d4afd1ce8d21dd2a6 (patch)
treebde2b9d4d51271f3d290a1cb205339d84abe307b /src/soc
parent33f234e3561c8cc5b9426fdf88067f2c9ced1052 (diff)
downloadcoreboot-dc667980f8ab78296e77b53d4afd1ce8d21dd2a6.tar.xz
soc/intel/skylake: Do not let FSP set the subsystem IDs
The subsystem ID registers are read/write-once. Writes by coreboot will not take effect if FSP sets them. Note that FSP sets one device ID for the SA devices and another for PCH devices. coreboot will copy individual vendor and device IDs if subsystem is not provided. Change-Id: I9157fb69f2a49dfc08f049da4b39fbf86614ace3 Signed-off-by: Benjamin Doron <benjamin.doron00@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/45006 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/soc')
-rw-r--r--src/soc/intel/skylake/chip.c15
1 files changed, 5 insertions, 10 deletions
diff --git a/src/soc/intel/skylake/chip.c b/src/soc/intel/skylake/chip.c
index 549f403384..89eaef5b56 100644
--- a/src/soc/intel/skylake/chip.c
+++ b/src/soc/intel/skylake/chip.c
@@ -320,16 +320,11 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd)
*/
params->SpiFlashCfgLockDown = 0;
}
- /* only replacing preexisting subsys ID defaults when non-zero */
- if (CONFIG_SUBSYSTEM_VENDOR_ID != 0) {
- params->DefaultSvid = CONFIG_SUBSYSTEM_VENDOR_ID;
- params->PchSubSystemVendorId = CONFIG_SUBSYSTEM_VENDOR_ID;
- }
-
- if (CONFIG_SUBSYSTEM_DEVICE_ID != 0) {
- params->DefaultSid = CONFIG_SUBSYSTEM_DEVICE_ID;
- params->PchSubSystemId = CONFIG_SUBSYSTEM_DEVICE_ID;
- }
+ /* FSP should let coreboot set subsystem IDs, which are read/write-once */
+ params->DefaultSvid = 0;
+ params->PchSubSystemVendorId = 0;
+ params->DefaultSid = 0;
+ params->PchSubSystemId = 0;
params->PchPmWolEnableOverride = config->WakeConfigWolEnableOverride;
params->PchPmPcieWakeFromDeepSx = config->WakeConfigPcieWakeFromDeepSx;