summaryrefslogtreecommitdiff
path: root/src/soc/intel
diff options
context:
space:
mode:
authorSubrata Banik <subrata.banik@intel.com>2021-09-30 13:15:50 +0530
committerSubrata Banik <subrata.banik@intel.com>2021-10-02 08:44:42 +0000
commit38abbdab71e6bf275c9c49748f1830576ddb2f22 (patch)
treea6c5a16af287b4cc3ce96e0464f7eb7e4a438245 /src/soc/intel
parentf576581954baa8df95e93bf136d1d2b0e8a7b646 (diff)
soc/intel/common/../cse: Avoid caching of CSE BAR
This patch ensures all attempts to read CSE BAR is performing PCI config space read and returning the BAR value rather than using cached value. This refactoring is useful to read BAR of all CSE devices rather than just HECI 1 alone. Additionally, change the return type of get_cse_bar() from `uintptr_t` to `void *` to avoid typecasting while calling read32/write32 functions. BUG=b:200644229 TEST=Able to build and boot ADLRVP where CSE is able to perform PCI enumeration and send the EOP message at post. Signed-off-by: Subrata Banik <subrata.banik@intel.com> Change-Id: Id4ecc9006d6323b7c9d7a6af1afa5cfe63d933e5 Reviewed-on: https://review.coreboot.org/c/coreboot/+/58062 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Diffstat (limited to 'src/soc/intel')
-rw-r--r--src/soc/intel/common/block/cse/cse.c59
1 files changed, 17 insertions, 42 deletions
diff --git a/src/soc/intel/common/block/cse/cse.c b/src/soc/intel/common/block/cse/cse.c
index ffe10a5ea5..d3b7d9ba13 100644
--- a/src/soc/intel/common/block/cse/cse.c
+++ b/src/soc/intel/common/block/cse/cse.c
@@ -64,9 +64,19 @@
#define MEI_HDR_CSE_ADDR_START 0
#define MEI_HDR_CSE_ADDR (((1 << 8) - 1) << MEI_HDR_CSE_ADDR_START)
-static struct cse_device {
- uintptr_t sec_bar;
-} cse;
+/* Get HECI BAR 0 from PCI configuration space */
+static uintptr_t get_cse_bar(void)
+{
+ uintptr_t bar;
+
+ bar = pci_read_config32(PCH_DEV_CSE, PCI_BASE_ADDRESS_0);
+ assert(bar != 0);
+ /*
+ * Bits 31-12 are the base address as per EDS for SPI,
+ * Don't care about 0-11 bit
+ */
+ return bar & ~PCI_BASE_ADDRESS_MEM_ATTR_MASK;
+}
/*
* Initialize the device with provided temporary BAR. If BAR is 0 use a
@@ -83,7 +93,7 @@ void heci_init(uintptr_t tempbar)
u16 pcireg;
/* Assume it is already initialized, nothing else to do */
- if (cse.sec_bar)
+ if (get_cse_bar())
return;
/* Use default pre-ram bar */
@@ -102,38 +112,16 @@ void heci_init(uintptr_t tempbar)
/* Enable Bus Master and MMIO Space */
pci_or_config16(dev, PCI_COMMAND, PCI_COMMAND_MASTER | PCI_COMMAND_MEMORY);
-
- cse.sec_bar = tempbar;
-}
-
-/* Get HECI BAR 0 from PCI configuration space */
-static uint32_t get_cse_bar(void)
-{
- uintptr_t bar;
-
- bar = pci_read_config32(PCH_DEV_CSE, PCI_BASE_ADDRESS_0);
- assert(bar != 0);
- /*
- * Bits 31-12 are the base address as per EDS for SPI,
- * Don't care about 0-11 bit
- */
- return bar & ~PCI_BASE_ADDRESS_MEM_ATTR_MASK;
}
static uint32_t read_bar(uint32_t offset)
{
- /* Load and cache BAR */
- if (!cse.sec_bar)
- cse.sec_bar = get_cse_bar();
- return read32((void *)(cse.sec_bar + offset));
+ return read32p(get_cse_bar() + offset);
}
static void write_bar(uint32_t offset, uint32_t val)
{
- /* Load and cache BAR */
- if (!cse.sec_bar)
- cse.sec_bar = get_cse_bar();
- return write32((void *)(cse.sec_bar + offset), val);
+ return write32p(get_cse_bar() + offset, val);
}
static uint32_t read_cse_csr(void)
@@ -968,21 +956,8 @@ bool set_cse_device_state(enum cse_device_state requested_state)
#if ENV_RAMSTAGE
-static void update_sec_bar(struct device *dev)
-{
- cse.sec_bar = find_resource(dev, PCI_BASE_ADDRESS_0)->base;
-}
-
-static void cse_set_resources(struct device *dev)
-{
- if (dev->path.pci.devfn == PCH_DEVFN_CSE)
- update_sec_bar(dev);
-
- pci_dev_set_resources(dev);
-}
-
static struct device_operations cse_ops = {
- .set_resources = cse_set_resources,
+ .set_resources = pci_dev_set_resources,
.read_resources = pci_dev_read_resources,
.enable_resources = pci_dev_enable_resources,
.init = pci_dev_init,