Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Nick Vaccaro: Looks good to me, approved Karthik Ramasubramanian: Looks good to me, approved
lp4x: Add new memory parts and generate SPDs

This change adds the following memory parts to LP4x global list and
generates SPDs using gen_spd.go for TGL and JSL:
1. MT53E512M32D2NP-046 WT:E
2. K4U6E3S4AA-MGCR
3. H9HCNNNCPMMLXR-NEE
4. K4UBE3D4AA-MGCR

BUG=b:157862308, b:157732528

Change-Id: Ib7538247d39dfe5faab277d646f87f09103d6969
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41989
Reviewed-by: Nick Vaccaro <nvaccaro@google.com>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M src/soc/intel/jasperlake/spd/lp4x/spd_manifest.generated.txt
M src/soc/intel/tigerlake/spd/lp4x/spd_manifest.generated.txt
M util/spd_tools/intel/lp4x/global_lp4x_mem_parts.json.txt
3 files changed, 57 insertions(+), 1 deletion(-)

diff --git a/src/soc/intel/jasperlake/spd/lp4x/spd_manifest.generated.txt b/src/soc/intel/jasperlake/spd/lp4x/spd_manifest.generated.txt
index e916733..564322e 100644
--- a/src/soc/intel/jasperlake/spd/lp4x/spd_manifest.generated.txt
+++ b/src/soc/intel/jasperlake/spd/lp4x/spd_manifest.generated.txt
@@ -7,3 +7,7 @@
MT53E1G64D4SQ-046 WT:A,spd-6.hex
MT53E512M32D2NP-046 WT:F,spd-1.hex
NT6AP256T32AV-J2,spd-7.hex
+K4U6E3S4AA-MGCR,spd-1.hex
+MT53E512M32D2NP-046 WT:E,spd-1.hex
+H9HCNNNCPMMLXR-NEE,spd-3.hex
+K4UBE3D4AA-MGCR,spd-3.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x/spd_manifest.generated.txt b/src/soc/intel/tigerlake/spd/lp4x/spd_manifest.generated.txt
index d11e93d..49fe445 100644
--- a/src/soc/intel/tigerlake/spd/lp4x/spd_manifest.generated.txt
+++ b/src/soc/intel/tigerlake/spd/lp4x/spd_manifest.generated.txt
@@ -7,3 +7,7 @@
MT53E1G64D4SQ-046 WT:A,spd-4.hex
MT53E512M32D2NP-046 WT:F,spd-1.hex
NT6AP256T32AV-J2,spd-5.hex
+K4U6E3S4AA-MGCR,spd-1.hex
+MT53E512M32D2NP-046 WT:E,spd-1.hex
+H9HCNNNCPMMLXR-NEE,spd-3.hex
+K4UBE3D4AA-MGCR,spd-3.hex
diff --git a/util/spd_tools/intel/lp4x/global_lp4x_mem_parts.json.txt b/util/spd_tools/intel/lp4x/global_lp4x_mem_parts.json.txt
index 5635158..b9dd4b3 100644
--- a/util/spd_tools/intel/lp4x/global_lp4x_mem_parts.json.txt
+++ b/util/spd_tools/intel/lp4x/global_lp4x_mem_parts.json.txt
@@ -109,6 +109,54 @@
"tckMaxPs": 1250,
"casLatencies": "14 20 24 28 32"
}
- }
+ },
+ {
+ "name": "K4U6E3S4AA-MGCR",
+ "attribs": {
+ "densityPerChannelGb": 8,
+ "banks": 8,
+ "channelsPerDie": 2,
+ "diesPerPackage": 1,
+ "bitWidthPerChannel": 16,
+ "ranksPerChannel": 1,
+ "speedMbps": 4267
+ }
+ },
+ {
+ "name": "MT53E512M32D2NP-046 WT:E",
+ "attribs": {
+ "densityPerChannelGb": 8,
+ "banks": 8,
+ "channelsPerDie": 2,
+ "diesPerPackage": 1,
+ "bitWidthPerChannel": 16,
+ "ranksPerChannel": 1,
+ "speedMbps": 4267
+ }
+ },
+ {
+ "name": "H9HCNNNCPMMLXR-NEE",
+ "attribs": {
+ "densityPerChannelGb": 8,
+ "banks": 8,
+ "channelsPerDie": 2,
+ "diesPerPackage": 2,
+ "bitWidthPerChannel": 16,
+ "ranksPerChannel": 2,
+ "speedMbps": 4267
+ }
+ },
+ {
+ "name": "K4UBE3D4AA-MGCR",
+ "attribs": {
+ "densityPerChannelGb": 8,
+ "banks": 8,
+ "channelsPerDie": 2,
+ "diesPerPackage": 2,
+ "bitWidthPerChannel": 16,
+ "ranksPerChannel": 2,
+ "speedMbps": 4267
+ }
+ }
]
}

To view, visit change 41989. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib7538247d39dfe5faab277d646f87f09103d6969
Gerrit-Change-Number: 41989
Gerrit-PatchSet: 10
Gerrit-Owner: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub@google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: merged