nb/intel/haswell: Use new fixed BAR accessors
There are some cases in `northbridge_topology_init` where condensing the
operation using one macro changes the binary, and have been left as-is.
Tested with BUILD_TIMELESS=1, Asrock B85M Pro4 remains identical.
Change-Id: I59c7d1f8d816b95e86d39dcbf7bc7ce8c34f0770
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51865
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
diff --git a/src/northbridge/intel/haswell/acpi.c b/src/northbridge/intel/haswell/acpi.c
index 96448f3..15f2c53 100644
--- a/src/northbridge/intel/haswell/acpi.c
+++ b/src/northbridge/intel/haswell/acpi.c
@@ -20,16 +20,16 @@
static unsigned long acpi_fill_dmar(unsigned long current)
{
struct device *const igfx_dev = pcidev_on_root(2, 0);
- const u32 gfxvtbar = MCHBAR32(GFXVTBAR) & ~0xfff;
- const u32 vtvc0bar = MCHBAR32(VTVC0BAR) & ~0xfff;
- const bool gfxvten = MCHBAR32(GFXVTBAR) & 0x1;
- const bool vtvc0en = MCHBAR32(VTVC0BAR) & 0x1;
+ const u32 gfxvtbar = mchbar_read32(GFXVTBAR) & ~0xfff;
+ const u32 vtvc0bar = mchbar_read32(VTVC0BAR) & ~0xfff;
+ const bool gfxvten = mchbar_read32(GFXVTBAR) & 0x1;
+ const bool vtvc0en = mchbar_read32(VTVC0BAR) & 0x1;
/* iGFX has to be enabled; GFXVTBAR set, enabled, in 32-bit space */
const bool emit_igd =
igfx_dev && igfx_dev->enabled &&
gfxvtbar && gfxvten &&
- !MCHBAR32(GFXVTBAR + 4);
+ !mchbar_read32(GFXVTBAR + 4);
/* First, add DRHD entries */
if (emit_igd) {
@@ -42,7 +42,7 @@
}
/* VTVC0BAR has to be set, enabled, and in 32-bit space */
- if (vtvc0bar && vtvc0en && !MCHBAR32(VTVC0BAR + 4)) {
+ if (vtvc0bar && vtvc0en && !mchbar_read32(VTVC0BAR + 4)) {
const unsigned long tmp = current;
current += acpi_create_dmar_drhd(current, DRHD_INCLUDE_PCI_ALL, 0, vtvc0bar);
diff --git a/src/northbridge/intel/haswell/early_init.c b/src/northbridge/intel/haswell/early_init.c
index f81ff9f..15a0b1b 100644
--- a/src/northbridge/intel/haswell/early_init.c
+++ b/src/northbridge/intel/haswell/early_init.c
@@ -138,17 +138,17 @@
u32 reg32;
/* Erratum workarounds */
- reg32 = MCHBAR32(SAPMCTL);
+ reg32 = mchbar_read32(SAPMCTL);
reg32 |= (1 << 9) | (1 << 10);
- MCHBAR32(SAPMCTL) = reg32;
+ mchbar_write32(SAPMCTL, reg32);
/* Enable SA Clock Gating */
- reg32 = MCHBAR32(SAPMCTL);
- MCHBAR32(SAPMCTL) = reg32 | 1;
+ reg32 = mchbar_read32(SAPMCTL);
+ mchbar_write32(SAPMCTL, reg32 | 1);
- reg32 = MCHBAR32(INTRDIRCTL);
+ reg32 = mchbar_read32(INTRDIRCTL);
reg32 |= (1 << 4) | (1 << 5);
- MCHBAR32(INTRDIRCTL) = reg32;
+ mchbar_write32(INTRDIRCTL, reg32);
}
static void haswell_setup_iommu(void)
@@ -159,10 +159,10 @@
return;
/* Setup BARs: zeroize top 32 bits; set enable bit */
- MCHBAR32(GFXVTBAR + 4) = GFXVT_BASE_ADDRESS >> 32;
- MCHBAR32(GFXVTBAR) = GFXVT_BASE_ADDRESS | 1;
- MCHBAR32(VTVC0BAR + 4) = VTVC0_BASE_ADDRESS >> 32;
- MCHBAR32(VTVC0BAR) = VTVC0_BASE_ADDRESS | 1;
+ mchbar_write32(GFXVTBAR + 4, GFXVT_BASE_ADDRESS >> 32);
+ mchbar_write32(GFXVTBAR + 0, GFXVT_BASE_ADDRESS | 1);
+ mchbar_write32(VTVC0BAR + 4, VTVC0_BASE_ADDRESS >> 32);
+ mchbar_write32(VTVC0BAR + 0, VTVC0_BASE_ADDRESS | 1);
/* Set L3HIT2PEND_DIS, lock GFXVTBAR policy config registers */
u32 reg32;
diff --git a/src/northbridge/intel/haswell/northbridge.c b/src/northbridge/intel/haswell/northbridge.c
index 3494422..0cf29b8 100644
--- a/src/northbridge/intel/haswell/northbridge.c
+++ b/src/northbridge/intel/haswell/northbridge.c
@@ -66,7 +66,7 @@
*/
static int get_bar_in_mchbar(struct device *dev, unsigned int index, u32 *base, u32 *len)
{
- u32 bar = MCHBAR32(index);
+ u32 bar = mchbar_read32(index);
/* If not enabled don't report it */
if (!(bar & 0x1))
@@ -399,16 +399,16 @@
static void init_egress(void)
{
/* VC0: Enable, ID0, TC0 */
- EPBAR32(EPVC0RCTL) = (1 << 31) | (0 << 24) | (1 << 0);
+ epbar_write32(EPVC0RCTL, 1 << 31 | 0 << 24 | 1 << 0);
/* No Low Priority Extended VCs, one Extended VC */
- EPBAR32(EPPVCCAP1) = (0 << 4) | (1 << 0);
+ epbar_write32(EPPVCCAP1, 0 << 4 | 1 << 0);
/* VC1: Enable, ID1, TC1 */
- EPBAR32(EPVC1RCTL) = (1 << 31) | (1 << 24) | (1 << 1);
+ epbar_write32(EPVC1RCTL, 1 << 31 | 1 << 24 | 1 << 1);
/* Poll the VC1 Negotiation Pending bit */
- while ((EPBAR16(EPVC1RSTS) & (1 << 1)) != 0)
+ while ((epbar_read16(EPVC1RSTS) & (1 << 1)) != 0)
;
}
@@ -416,39 +416,24 @@
{
const bool is_haswell_h = !CONFIG(INTEL_LYNXPOINT_LP);
- u16 reg16;
- u32 reg32;
-
/* Steps prior to DMI ASPM */
if (is_haswell_h) {
/* Configure DMI De-Emphasis */
- reg16 = DMIBAR16(DMILCTL2);
- reg16 |= (1 << 6); /* 0b: -6.0 dB, 1b: -3.5 dB */
- DMIBAR16(DMILCTL2) = reg16;
+ dmibar_setbits16(DMILCTL2, 1 << 6); /* 0b: -6.0 dB, 1b: -3.5 dB */
- reg32 = DMIBAR32(DMIL0SLAT);
- reg32 |= (1 << 31);
- DMIBAR32(DMIL0SLAT) = reg32;
+ dmibar_setbits32(DMIL0SLAT, 1 << 31);
+ dmibar_setbits32(DMILLTC, 1 << 29);
- reg32 = DMIBAR32(DMILLTC);
- reg32 |= (1 << 29);
- DMIBAR32(DMILLTC) = reg32;
-
- reg32 = DMIBAR32(DMI_AFE_PM_TMR);
- reg32 &= ~0x1f;
- reg32 |= 0x13;
- DMIBAR32(DMI_AFE_PM_TMR) = reg32;
+ dmibar_clrsetbits32(DMI_AFE_PM_TMR, 0x1f, 0x13);
}
/* Clear error status bits */
- DMIBAR32(DMIUESTS) = 0xffffffff;
- DMIBAR32(DMICESTS) = 0xffffffff;
+ dmibar_write32(DMIUESTS, 0xffffffff);
+ dmibar_write32(DMICESTS, 0xffffffff);
if (is_haswell_h) {
/* Enable ASPM L0s and L1 on SA link, should happen before PCH link */
- reg16 = DMIBAR16(DMILCTL);
- reg16 |= (1 << 1) | (1 << 0);
- DMIBAR16(DMILCTL) = reg16;
+ dmibar_setbits16(DMILCTL, 1 << 1 | 1 << 0);
}
}
@@ -460,17 +445,14 @@
u32 reg32;
/* Set the CID1 Egress Port 0 Root Topology */
- reg32 = EPBAR32(EPESD);
- reg32 &= ~(0xff << 16);
- reg32 |= 1 << 16;
- EPBAR32(EPESD) = reg32;
+ epbar_clrsetbits32(EPESD, 0xff << 16, 1 << 16);
- reg32 = EPBAR32(EPLE1D);
+ reg32 = epbar_read32(EPLE1D);
reg32 &= ~(0xff << 16);
reg32 |= 1 | (1 << 16);
- EPBAR32(EPLE1D) = reg32;
- EPBAR32(EPLE1A) = CONFIG_FIXED_DMIBAR_MMIO_BASE;
- EPBAR32(EPLE1A + 4) = 0;
+ epbar_write32(EPLE1D, reg32);
+ epbar_write32(EPLE1A, CONFIG_FIXED_DMIBAR_MMIO_BASE);
+ epbar_write32(EPLE1A + 4, 0);
for (unsigned int i = 0; i <= 2; i++) {
const struct device *const dev = pcidev_on_root(1, i);
@@ -478,13 +460,13 @@
if (!dev || !dev->enabled)
continue;
- EPBAR32(eple_a[i]) = (u32)PCI_DEV(0, 1, i);
- EPBAR32(eple_a[i] + 4) = 0;
+ epbar_write32(eple_a[i], (u32)PCI_DEV(0, 1, i));
+ epbar_write32(eple_a[i] + 4, 0);
- reg32 = EPBAR32(eple_d[i]);
+ reg32 = epbar_read32(eple_d[i]);
reg32 &= ~(0xff << 16);
reg32 |= 1 | (1 << 16);
- EPBAR32(eple_d[i]) = reg32;
+ epbar_write32(eple_d[i], reg32);
pci_update_config32(dev, PEG_ESD, ~(0xff << 16), (1 << 16));
pci_write_config32(dev, PEG_LE1A, CONFIG_FIXED_EPBAR_MMIO_BASE);
@@ -496,43 +478,35 @@
}
/* Set the CID1 DMI Port Root Topology */
- reg32 = DMIBAR32(DMIESD);
- reg32 &= ~(0xff << 16);
- reg32 |= 1 << 16;
- DMIBAR32(DMIESD) = reg32;
+ dmibar_clrsetbits32(DMIESD, 0xff << 16, 1 << 16);
- reg32 = DMIBAR32(DMILE1D);
+ reg32 = dmibar_read32(DMILE1D);
reg32 &= ~(0xffff << 16);
reg32 |= 1 | (2 << 16);
- DMIBAR32(DMILE1D) = reg32;
- DMIBAR32(DMILE1A) = CONFIG_FIXED_RCBA_MMIO_BASE;
- DMIBAR32(DMILE1A + 4) = 0;
+ dmibar_write32(DMILE1D, reg32);
+ dmibar_write32(DMILE1A, CONFIG_FIXED_RCBA_MMIO_BASE);
+ dmibar_write32(DMILE1A + 4, 0);
- DMIBAR32(DMILE2A) = CONFIG_FIXED_EPBAR_MMIO_BASE;
- DMIBAR32(DMILE2A + 4) = 0;
- reg32 = DMIBAR32(DMILE2D);
+ dmibar_write32(DMILE2A, CONFIG_FIXED_EPBAR_MMIO_BASE);
+ dmibar_write32(DMILE2A + 4, 0);
+ reg32 = dmibar_read32(DMILE2D);
reg32 &= ~(0xff << 16);
reg32 |= 1 | (1 << 16);
- DMIBAR32(DMILE2D) = reg32;
+ dmibar_write32(DMILE2D, reg32);
/* Program RO and Write-Once Registers */
- DMIBAR32(DMIPVCCAP1) = DMIBAR32(DMIPVCCAP1);
- DMIBAR32(DMILCAP) = DMIBAR32(DMILCAP);
+ dmibar_setbits32(DMIPVCCAP1, 0);
+ dmibar_setbits32(DMILCAP, 0);
}
static void northbridge_init(struct device *dev)
{
- u8 bios_reset_cpl, pair;
-
init_egress();
northbridge_dmi_init();
northbridge_topology_init();
/* Enable Power Aware Interrupt Routing. */
- pair = MCHBAR8(INTRDIRCTL);
- pair &= ~0x7; /* Clear 2:0 */
- pair |= 0x4; /* Fixed Priority */
- MCHBAR8(INTRDIRCTL) = pair;
+ mchbar_clrsetbits8(INTRDIRCTL, 0x7, 0x4); /* Clear 2:0, set Fixed Priority */
disable_devices();
@@ -540,9 +514,7 @@
* Set bits 0 + 1 of BIOS_RESET_CPL to indicate to the CPU
* that BIOS has initialized memory and power management.
*/
- bios_reset_cpl = MCHBAR8(BIOS_RESET_CPL);
- bios_reset_cpl |= 3;
- MCHBAR8(BIOS_RESET_CPL) = bios_reset_cpl;
+ mchbar_setbits8(BIOS_RESET_CPL, 3);
printk(BIOS_DEBUG, "Set BIOS_RESET_CPL\n");
/* Configure turbo power limits 1ms after reset complete bit. */
@@ -565,21 +537,21 @@
pci_or_config32(dev, TOLUD, 1 << 0);
/* Memory Controller Lockdown */
- MCHBAR32(MC_LOCK) |= 0x8f;
+ mchbar_setbits32(MC_LOCK, 0x8f);
- MCHBAR32_OR(MMIO_PAVP_MSG, 1 << 0); /* PAVP */
- MCHBAR32_OR(PCU_DDR_PTM_CTL, 1 << 5); /* DDR PTM */
- MCHBAR32_OR(DMIVCLIM, 1 << 31);
- MCHBAR32_OR(CRDTLCK, 1 << 0);
- MCHBAR32_OR(MCARBLCK, 1 << 0);
- MCHBAR32_OR(REQLIM, 1 << 31);
- MCHBAR32_OR(UMAGFXCTL, 1 << 0); /* UMA GFX */
- MCHBAR32_OR(VTDTRKLCK, 1 << 0); /* VTDTRK */
+ mchbar_setbits32(MMIO_PAVP_MSG, 1 << 0); /* PAVP */
+ mchbar_setbits32(PCU_DDR_PTM_CTL, 1 << 5); /* DDR PTM */
+ mchbar_setbits32(DMIVCLIM, 1 << 31);
+ mchbar_setbits32(CRDTLCK, 1 << 0);
+ mchbar_setbits32(MCARBLCK, 1 << 0);
+ mchbar_setbits32(REQLIM, 1 << 31);
+ mchbar_setbits32(UMAGFXCTL, 1 << 0); /* UMA GFX */
+ mchbar_setbits32(VTDTRKLCK, 1 << 0); /* VTDTRK */
/* Read+write the following */
- MCHBAR32(VDMBDFBARKVM) = MCHBAR32(VDMBDFBARKVM);
- MCHBAR32(VDMBDFBARPAVP) = MCHBAR32(VDMBDFBARPAVP);
- MCHBAR32(HDAUDRID) = MCHBAR32(HDAUDRID);
+ mchbar_setbits32(VDMBDFBARKVM, 0);
+ mchbar_setbits32(VDMBDFBARPAVP, 0);
+ mchbar_setbits32(HDAUDRID, 0);
}
static struct device_operations mc_ops = {
diff --git a/src/northbridge/intel/haswell/raminit.c b/src/northbridge/intel/haswell/raminit.c
index ff2456b..5f74a89 100644
--- a/src/northbridge/intel/haswell/raminit.c
+++ b/src/northbridge/intel/haswell/raminit.c
@@ -70,10 +70,10 @@
{
int i;
- const u32 addr_decoder_common = MCHBAR32(MAD_CHNL);
+ const u32 addr_decoder_common = mchbar_read32(MAD_CHNL);
printk(BIOS_DEBUG, "memcfg DDR3 clock %d MHz\n",
- (MCHBAR32(MC_BIOS_DATA) * 13333 * 2 + 50) / 100);
+ (mchbar_read32(MC_BIOS_DATA) * 13333 * 2 + 50) / 100);
printk(BIOS_DEBUG, "memcfg channel assignment: A: %d, B % d, C % d\n",
(addr_decoder_common >> 0) & 3,
@@ -81,7 +81,7 @@
(addr_decoder_common >> 4) & 3);
for (i = 0; i < NUM_CHANNELS; i++) {
- const u32 ch_conf = MCHBAR32(MAD_DIMM(i));
+ const u32 ch_conf = mchbar_read32(MAD_DIMM(i));
printk(BIOS_DEBUG, "memcfg channel[%d] config (%8.8x):\n", i, ch_conf);
printk(BIOS_DEBUG, " ECC %s\n", ecc_decoder[(ch_conf >> 24) & 3]);
@@ -164,7 +164,7 @@
}
/* Print the MRC version after executing the UEFI PEI stage */
- u32 version = MCHBAR32(MRC_REVISION);
+ u32 version = mchbar_read32(MRC_REVISION);
printk(BIOS_DEBUG, "MRC Version %d.%d.%d Build %d\n",
(version >> 24) & 0xff, (version >> 16) & 0xff,
(version >> 8) & 0xff, (version >> 0) & 0xff);
@@ -234,10 +234,10 @@
memset(mem_info, 0, sizeof(struct memory_info));
- const u32 ddr_frequency = (MCHBAR32(MC_BIOS_DATA) * 13333 * 2 + 50) / 100;
+ const u32 ddr_frequency = (mchbar_read32(MC_BIOS_DATA) * 13333 * 2 + 50) / 100;
for (ch = 0; ch < NUM_CHANNELS; ch++) {
- const u32 ch_conf = MCHBAR32(MAD_DIMM(ch));
+ const u32 ch_conf = mchbar_read32(MAD_DIMM(ch));
/* DIMMs A/B */
for (d_num = 0; d_num < NUM_SLOTS; d_num++) {
const u32 dimm_size = ((ch_conf >> (d_num * 8)) & 0xff) * 256;