summaryrefslogtreecommitdiff
path: root/src/soc/intel/xeon_sp/finalize.c
diff options
context:
space:
mode:
authorJonathan Zhang <jonzhang@meta.com>2023-01-24 11:18:15 -0800
committerFelix Held <felix-coreboot@felixheld.de>2023-03-09 21:40:23 +0000
commit665d870244a26e28e445ac8e6945ddf986d9050a (patch)
tree07cea0977cce4d6b17ac607ab174c9211666618c /src/soc/intel/xeon_sp/finalize.c
parentca520a726a6d47c31c5a8c278e2a272b1f89bac4 (diff)
soc/intel/xeon_sp: rework lock_pam0123() to accomodate hidden SAD device
For Intel SPR-SP, the SAD device is hidden, so pcidev_path_on_bus() returns NULL. Therefore use pci_s_write_config32() instead. Move lock_pam0123() from finalize.c to util.c, to be together with unlock_pam_regions(). Change-Id: Ib08d423d8c4d482612077b66dab3878018da8f2b Signed-off-by: Jonathan Zhang <jonzhang@meta.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/72432 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: David Hendricks <david.hendricks@gmail.com>
Diffstat (limited to 'src/soc/intel/xeon_sp/finalize.c')
-rw-r--r--src/soc/intel/xeon_sp/finalize.c11
1 files changed, 0 insertions, 11 deletions
diff --git a/src/soc/intel/xeon_sp/finalize.c b/src/soc/intel/xeon_sp/finalize.c
index 6b8395c783..7830a4b27f 100644
--- a/src/soc/intel/xeon_sp/finalize.c
+++ b/src/soc/intel/xeon_sp/finalize.c
@@ -16,17 +16,6 @@
#include "chip.h"
-static void lock_pam0123(void)
-{
- const struct device *dev;
-
- if (get_lockdown_config() != CHIPSET_LOCKDOWN_COREBOOT)
- return;
-
- dev = pcidev_path_on_bus(get_stack_busno(1), PCI_DEVFN(SAD_ALL_DEV, SAD_ALL_FUNC));
- pci_or_config32(dev, SAD_ALL_PAM0123_CSR, PAM_LOCK);
-}
-
DECLARE_SPIN_LOCK(msr_ppin_lock);
static void lock_msr_ppin_ctl(void *unused)