summaryrefslogtreecommitdiff
path: root/src/soc/intel/common/block/systemagent
diff options
context:
space:
mode:
authorEran Mitrani <mitrani@google.com>2022-05-25 16:29:19 -0700
committerFelix Held <felix-coreboot@felixheld.de>2022-06-15 23:58:31 +0000
commit400c30005e11e155a58f4acfc9bc5d4f34ece5a4 (patch)
tree93aaac46e8634c4db7e01376a09d4bcde731626d /src/soc/intel/common/block/systemagent
parent14908bf05ad3cb5a44c1a0a8197f04e5cd8658d9 (diff)
soc/intel/common: support for configurable memory regions claimed by SA
see https://review.coreboot.org/c/coreboot/+/65072/8 BUG=b:149830546 BRANCH=firmware-brya-14505.B TEST='emerge-brya coreboot chromeos-bootimage' builds correctly. Tested on an Anahera device which successfully boots to ChromeOS with kernel version 5.10.109-15688-g857e654d1705. Change-Id: I80df95f9146934d6a2d23e525c22be3a9a7e2b9f Signed-off-by: Eran Mitrani <mitrani@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/64677 Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Nick Vaccaro <nvaccaro@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/intel/common/block/systemagent')
-rw-r--r--src/soc/intel/common/block/systemagent/systemagent.c14
-rw-r--r--src/soc/intel/common/block/systemagent/systemagent_def.h12
2 files changed, 15 insertions, 11 deletions
diff --git a/src/soc/intel/common/block/systemagent/systemagent.c b/src/soc/intel/common/block/systemagent/systemagent.c
index 55e892c941..19a413f9d1 100644
--- a/src/soc/intel/common/block/systemagent/systemagent.c
+++ b/src/soc/intel/common/block/systemagent/systemagent.c
@@ -30,6 +30,12 @@ __weak void soc_add_fixed_mmio_resources(struct device *dev,
/* no-op */
}
+__weak void soc_add_configurable_mmio_resources(struct device *dev,
+ int *resource_cnt)
+{
+ /* no-op */
+}
+
__weak int soc_get_uncore_prmmr_base_and_mask(uint64_t *base,
uint64_t *mask)
{
@@ -98,6 +104,10 @@ void sa_add_fixed_mmio_resources(struct device *dev, int *resource_cnt,
size = sa_fixed_resources[i].size;
base = sa_fixed_resources[i].base;
+ printk(BIOS_DEBUG, "SA MMIO resource: %s -> base = 0x%llx, size = 0x%llx\n",
+ sa_fixed_resources[i].description, sa_fixed_resources[i].base,
+ sa_fixed_resources[i].size);
+
mmio_resource(dev, index++, base / KiB, size / KiB);
}
@@ -273,6 +283,10 @@ static void systemagent_read_resources(struct device *dev)
/* Add all fixed MMIO resources. */
soc_add_fixed_mmio_resources(dev, &index);
+
+ /* Add all configurable MMIO resources. */
+ soc_add_configurable_mmio_resources(dev, &index);
+
/* Calculate and add DRAM resources. */
sa_add_dram_resources(dev, &index);
if (CONFIG(SA_ENABLE_IMR))
diff --git a/src/soc/intel/common/block/systemagent/systemagent_def.h b/src/soc/intel/common/block/systemagent/systemagent_def.h
index 0add13e93e..09a99ea2e9 100644
--- a/src/soc/intel/common/block/systemagent/systemagent_def.h
+++ b/src/soc/intel/common/block/systemagent/systemagent_def.h
@@ -5,12 +5,7 @@
/* Device 0:0.0 PCI configuration space */
-/* GMCH Graphics Control Register */
-#define GGC 0x50
-#define G_GMS_OFFSET 0x8
-#define G_GMS_MASK 0xff00
-#define G_GGMS_OFFSET 0x6
-#define G_GGMS_MASK 0xc0
+
/* DPR register in case CONFIG_SA_ENABLE_DPR is selected by SoC */
#define DPR 0x5c
#define DPR_EPM (1 << 2)
@@ -22,11 +17,6 @@
#define CAPID_PDCD (1 << 12)
#define CAPID_DDRSZ(x) (((x) >> 19) & 0x3)
-#define PCIEXBAR_LENGTH_64MB 2
-#define PCIEXBAR_LENGTH_128MB 1
-#define PCIEXBAR_LENGTH_256MB 0
-#define PCIEXBAR_PCIEXBAREN (1 << 0)
-
#define PAM0 0x80
#define PAM1 0x81
#define PAM2 0x82