summaryrefslogtreecommitdiff
path: root/src/northbridge
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-07-22 17:49:28 +0200
committerPatrick Georgi <pgeorgi@google.com>2020-08-03 05:31:47 +0000
commit10993c4ad40c92b2b2796856f9de2a5f602a2da9 (patch)
tree6c1f108c00a6ebab16d574aa44581bbde6697acd /src/northbridge
parent08143576466918413bfae2303abc4d0d16ae5b5c (diff)
nb/intel/ironlake: Add QPI Physical Layer device definition
Like the QPI Link device, there can be more of these devices on multi-socket platforms. So, name it Physical Layer 0. Tested with BUILD_TIMELESS=1, Packard Bell MS2290 remains identical. Change-Id: Ia5f6e42a742bc69237de38f1833e56c8da7c4f7e Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/43737 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'src/northbridge')
-rw-r--r--src/northbridge/intel/ironlake/ironlake.h5
-rw-r--r--src/northbridge/intel/ironlake/raminit.c26
2 files changed, 18 insertions, 13 deletions
diff --git a/src/northbridge/intel/ironlake/ironlake.h b/src/northbridge/intel/ironlake/ironlake.h
index aa2399f136..d09ccccfd7 100644
--- a/src/northbridge/intel/ironlake/ironlake.h
+++ b/src/northbridge/intel/ironlake/ironlake.h
@@ -70,6 +70,11 @@
#define QPI_QPILS 0x50 /* QPI Link Status */
#define QPI_DEF_RMT_VN_CREDITS 0x58 /* Default Available Remote Credits */
+/*
+ * QPI Physical Layer 0
+ */
+#define QPI_PHY_0 PCI_DEV(QUICKPATH_BUS, 2, 1)
+
/* Device 0:2.0 PCI configuration space (Graphics Device) */
diff --git a/src/northbridge/intel/ironlake/raminit.c b/src/northbridge/intel/ironlake/raminit.c
index f955a1ea46..65d682740f 100644
--- a/src/northbridge/intel/ironlake/raminit.c
+++ b/src/northbridge/intel/ironlake/raminit.c
@@ -3882,8 +3882,8 @@ void raminit(const int s3resume, const u8 *spd_addrmap)
MCHBAR32_OR(0x1890, 0x2000000);
MCHBAR32_OR(0x18b4, 0x8000);
- gav(pci_read_config32(PCI_DEV(0xff, 2, 1), 0x50)); // !!!!
- pci_write_config8(PCI_DEV(0xff, 2, 1), 0x54, 0x12);
+ gav(pci_read_config32(QPI_PHY_0, 0x50)); // !!!!
+ pci_write_config8(QPI_PHY_0, 0x54, 0x12);
gav(MCHBAR16(0x2c10));
MCHBAR16(0x2c10) = 0x412;
@@ -3893,8 +3893,8 @@ void raminit(const int s3resume, const u8 *spd_addrmap)
gav(MCHBAR8(0x2ca8)); // !!!!
MCHBAR32_AND_OR(0x1804, 0xfffffffc, 0x8400080);
- pci_read_config32(PCI_DEV(0xff, 2, 1), 0x6c); // !!!!
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0x6c, 0x40a0a0);
+ pci_read_config32(QPI_PHY_0, 0x6c); // !!!!
+ pci_write_config32(QPI_PHY_0, 0x6c, 0x40a0a0);
gav(MCHBAR32(0x1c04)); // !!!!
gav(MCHBAR32(0x1804)); // !!!!
@@ -3904,16 +3904,16 @@ void raminit(const int s3resume, const u8 *spd_addrmap)
MCHBAR32(0x18d8) = 0x120000;
MCHBAR32(0x18dc) = 0x30a484a;
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xe0, 0x0);
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xf4, 0x9444a);
+ pci_write_config32(QPI_PHY_0, 0xe0, 0x0);
+ pci_write_config32(QPI_PHY_0, 0xf4, 0x9444a);
MCHBAR32(0x18d8) = 0x40000;
MCHBAR32(0x18dc) = 0xb000000;
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xe0, 0x60000);
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xf4, 0x0);
+ pci_write_config32(QPI_PHY_0, 0xe0, 0x60000);
+ pci_write_config32(QPI_PHY_0, 0xf4, 0x0);
MCHBAR32(0x18d8) = 0x180000;
MCHBAR32(0x18dc) = 0xc0000142;
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xe0, 0x20000);
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xf4, 0x142);
+ pci_write_config32(QPI_PHY_0, 0xe0, 0x20000);
+ pci_write_config32(QPI_PHY_0, 0xf4, 0x142);
MCHBAR32(0x18d8) = 0x1e0000;
gav(MCHBAR32(0x18dc)); // !!!!
@@ -3925,7 +3925,7 @@ void raminit(const int s3resume, const u8 *spd_addrmap)
}
MCHBAR32(0x188c) = 0x20bc09;
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0xd0, 0x40b0c09);
+ pci_write_config32(QPI_PHY_0, 0xd0, 0x40b0c09);
MCHBAR32(0x1a10) = 0x4200010e;
MCHBAR32_OR(0x18b8, 0x200);
gav(MCHBAR32(0x1918)); // !!!!
@@ -3935,8 +3935,8 @@ void raminit(const int s3resume, const u8 *spd_addrmap)
MCHBAR32(0x18b8) = 0xe00;
gav(MCHBAR32(0x182c)); // !!!!
MCHBAR32(0x182c) = 0x10202;
- gav(pci_read_config32(PCI_DEV(0xff, 2, 1), 0x94)); // !!!!
- pci_write_config32(PCI_DEV(0xff, 2, 1), 0x94, 0x10202);
+ gav(pci_read_config32(QPI_PHY_0, 0x94)); // !!!!
+ pci_write_config32(QPI_PHY_0, 0x94, 0x10202);
MCHBAR32_AND(0x1a1c, 0x8fffffff);
MCHBAR32_OR(0x1a70, 0x100000);