summaryrefslogtreecommitdiff
path: root/src/northbridge/intel
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-08-03 14:18:41 +0200
committerAngel Pons <th3fanbus@gmail.com>2020-08-06 07:25:25 +0000
commit20905cfe26448166f3224bb59b38dba6cc132274 (patch)
tree3b9773d835897238f5bcfce04a771a2d52529a05 /src/northbridge/intel
parent96b32f194bf6ed4de7d495acc0c50106cf3c72d7 (diff)
downloadcoreboot-20905cfe26448166f3224bb59b38dba6cc132274.tar.xz
nb/intel/sandybridge: Refactor `get_pcie_bar`
Turn it into `decode_pcie_bar`, taken from gm45. Change-Id: Id1c2cfbcac1a798d046beced790930511dc97972 Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/44121 Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/northbridge/intel')
-rw-r--r--src/northbridge/intel/sandybridge/northbridge.c33
1 files changed, 17 insertions, 16 deletions
diff --git a/src/northbridge/intel/sandybridge/northbridge.c b/src/northbridge/intel/sandybridge/northbridge.c
index 1d20a4b57a..8f1db9791b 100644
--- a/src/northbridge/intel/sandybridge/northbridge.c
+++ b/src/northbridge/intel/sandybridge/northbridge.c
@@ -2,6 +2,7 @@
#include <console/console.h>
#include <acpi/acpi.h>
+#include <commonlib/helpers.h>
#include <device/pci_ops.h>
#include <stdint.h>
#include <delay.h>
@@ -39,18 +40,16 @@ int bridge_silicon_revision(void)
static const int legacy_hole_base_k = 0xa0000 / 1024;
static const int legacy_hole_size_k = 384;
-static int get_pcie_bar(u32 *base)
+static int decode_pcie_bar(u32 *const base, u32 *const len)
{
- struct device *dev;
- u32 pciexbar_reg;
-
*base = 0;
+ *len = 0;
- dev = pcidev_on_root(0, 0);
+ struct device *dev = pcidev_on_root(0, 0);
if (!dev)
return 0;
- pciexbar_reg = pci_read_config32(dev, PCIEXBAR);
+ const u32 pciexbar_reg = pci_read_config32(dev, PCIEXBAR);
/* MMCFG not supported or not enabled */
if (!(pciexbar_reg & (1 << 0)))
@@ -58,14 +57,17 @@ static int get_pcie_bar(u32 *base)
switch ((pciexbar_reg >> 1) & 3) {
case 0: /* 256MB */
- *base = pciexbar_reg & (0xffffffffULL << 28);
- return 256;
+ *base = pciexbar_reg & (0x0f << 28);
+ *len = 256 * MiB;
+ return 1;
case 1: /* 128M */
- *base = pciexbar_reg & (0xffffffffULL << 27);
- return 128;
+ *base = pciexbar_reg & (0x1f << 27);
+ *len = 128 * MiB;
+ return 1;
case 2: /* 64M */
- *base = pciexbar_reg & (0xffffffffULL << 26);
- return 64;
+ *base = pciexbar_reg & (0x3f << 26);
+ *len = 64 * MiB;
+ return 1;
}
return 0;
@@ -129,8 +131,7 @@ static void add_fixed_resources(struct device *dev, int index)
static void mc_read_resources(struct device *dev)
{
- u32 pcie_config_base;
- int buses;
+ u32 pcie_config_base, pcie_config_len;
uint64_t tom, me_base, touud;
uint32_t tseg_base, uma_size, tolud;
uint16_t ggc;
@@ -139,8 +140,8 @@ static void mc_read_resources(struct device *dev)
pci_dev_read_resources(dev);
- buses = get_pcie_bar(&pcie_config_base);
- if (buses) {
+ if (decode_pcie_bar(&pcie_config_base, &pcie_config_len)) {
+ const int buses = pcie_config_len / MiB;
struct resource *resource = new_resource(dev, PCIEXBAR);
mmconf_resource_init(resource, pcie_config_base, buses);
}