summaryrefslogtreecommitdiff
path: root/src/soc
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2020-11-10 15:55:31 +0100
committerHung-Te Lin <hungte@chromium.org>2020-12-28 13:39:39 +0000
commit7a36ca5a3af464bab21e61256e41c4c8eb220f7d (patch)
treebdd607596430ffce17d5191690a0e11e98745156 /src/soc
parent42a6f7e417f64a475f6e2b54ea59ee0a733a9c79 (diff)
soc/intel/xeon_sp: Lock down IIO DFX Global registers
This is required for CbNT. Change-Id: I565a95cd2e76cb1c648884be6d1954288f6e4804 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/47447 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/soc')
-rw-r--r--src/soc/intel/xeon_sp/cpx/include/soc/pci_devs.h4
-rw-r--r--src/soc/intel/xeon_sp/skx/include/soc/pci_devs.h4
-rw-r--r--src/soc/intel/xeon_sp/uncore.c31
3 files changed, 39 insertions, 0 deletions
diff --git a/src/soc/intel/xeon_sp/cpx/include/soc/pci_devs.h b/src/soc/intel/xeon_sp/cpx/include/soc/pci_devs.h
index 6ddcce4cfe..95290f2f55 100644
--- a/src/soc/intel/xeon_sp/cpx/include/soc/pci_devs.h
+++ b/src/soc/intel/xeon_sp/cpx/include/soc/pci_devs.h
@@ -127,4 +127,8 @@
#define DMIRCBAR 0x50
#define ERRINJCON 0x1d8
+// IIO DFX Global D7F7 registers
+#define IIO_DFX_TSWCTL0 0x30c
+#define IIO_DFX_LCK_CTL 0x504
+
#endif /* _SOC_PCI_DEVS_H_ */
diff --git a/src/soc/intel/xeon_sp/skx/include/soc/pci_devs.h b/src/soc/intel/xeon_sp/skx/include/soc/pci_devs.h
index 5fa2a38387..02061f98f0 100644
--- a/src/soc/intel/xeon_sp/skx/include/soc/pci_devs.h
+++ b/src/soc/intel/xeon_sp/skx/include/soc/pci_devs.h
@@ -172,4 +172,8 @@
#define DMIRCBAR 0x50
#define ERRINJCON 0x1d8
+// IIO DFX Global D7F7 registers
+#define IIO_DFX_TSWCTL0 0x30c
+#define IIO_DFX_LCK_CTL 0x504
+
#endif /* _SOC_PCI_DEVS_H_ */
diff --git a/src/soc/intel/xeon_sp/uncore.c b/src/soc/intel/xeon_sp/uncore.c
index 2663023742..7679f2f1af 100644
--- a/src/soc/intel/xeon_sp/uncore.c
+++ b/src/soc/intel/xeon_sp/uncore.c
@@ -374,3 +374,34 @@ static const struct pci_driver dmi3_driver __pci_driver = {
.vendor = PCI_VENDOR_ID_INTEL,
.device = DMI3_DEVID,
};
+
+static void iio_dfx_global_init(struct device *dev)
+{
+ uint16_t reg16;
+ pci_or_config16(dev, IIO_DFX_LCK_CTL, 0x3ff);
+ reg16 = pci_read_config16(dev, IIO_DFX_TSWCTL0);
+ reg16 &= ~(1 << 4); // allow ib mmio cfg
+ reg16 &= ~(1 << 5); // ignore acs p2p ma lpbk
+ reg16 |= (1 << 3); // me disable
+ pci_write_config16(dev, IIO_DFX_TSWCTL0, reg16);
+}
+
+static const unsigned short iio_dfx_global_ids[] = {
+ 0x202d,
+ 0x203d,
+ 0
+};
+
+static struct device_operations iio_dfx_global_ops = {
+ .read_resources = pci_dev_read_resources,
+ .set_resources = pci_dev_set_resources,
+ .enable_resources = pci_dev_enable_resources,
+ .init = iio_dfx_global_init,
+ .ops_pci = &soc_pci_ops,
+};
+
+static const struct pci_driver iio_dfx_global_driver __pci_driver = {
+ .ops = &iio_dfx_global_ops,
+ .vendor = PCI_VENDOR_ID_INTEL,
+ .devices = iio_dfx_global_ids,
+};