Felix Held submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Rob Barnes: Looks good to me, approved
mb/google/guybrush/var/dewatt: update DRAM ID table

1. Samsung LPDDR4X 4266 4G K4UBE3D4AB-MGCL
2. Hynix LPDDR4X 4266 4G H54G56CYRBX247 (already used by other variants)

BUG=b:203014978
BRANCH=guybrush
TEST=emerge-guybrush coreboot chromeos-bootimage

Signed-off-by: Kenneth Chan <kenneth.chan@quanta.corp-partner.google.com>
Change-Id: Ie5ece849a86c75be5af9bc0393090b5f1e33bfed
Reviewed-on: https://review.coreboot.org/c/coreboot/+/60090
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Reviewed-by: Rob Barnes <robbarnes@google.com>
---
M src/mainboard/google/guybrush/variants/dewatt/memory/Makefile.inc
M src/mainboard/google/guybrush/variants/dewatt/memory/dram_id.generated.txt
M src/mainboard/google/guybrush/variants/guybrush/memory/mem_parts_used.txt
3 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/mainboard/google/guybrush/variants/dewatt/memory/Makefile.inc b/src/mainboard/google/guybrush/variants/dewatt/memory/Makefile.inc
index 26f27d5..313feb5 100644
--- a/src/mainboard/google/guybrush/variants/dewatt/memory/Makefile.inc
+++ b/src/mainboard/google/guybrush/variants/dewatt/memory/Makefile.inc
@@ -7,4 +7,4 @@
SPD_SOURCES += spd/lp4x/set-1/spd-4.hex # ID = 0(0b0000) Parts = MT53E1G32D2NP-046 WT:A
SPD_SOURCES += spd/lp4x/set-1/spd-1.hex # ID = 1(0b0001) Parts = MT53E512M32D2NP-046 WT:F, H9HCNNNBKMMLXR-NEE, MT53E512M32D2NP-046 WT:E, K4U6E3S4AA-MGCR, K4U6E3S4AB-MGCL, H54G46CYRBX267, MT53E512M32D1NP-046 WT:B
SPD_SOURCES += spd/lp4x/set-1/spd-9.hex # ID = 2(0b0010) Parts = NT6AP256T32AV-J1
-SPD_SOURCES += spd/lp4x/set-1/spd-3.hex # ID = 3(0b0011) Parts = MT53E1G32D2NP-046 WT:B, H9HCNNNCPMMLXR-NEE, K4UBE3D4AA-MGCR
+SPD_SOURCES += spd/lp4x/set-1/spd-3.hex # ID = 3(0b0011) Parts = MT53E1G32D2NP-046 WT:B, H9HCNNNCPMMLXR-NEE, K4UBE3D4AA-MGCR, H54G56CYRBX247, K4UBE3D4AB-MGCL
diff --git a/src/mainboard/google/guybrush/variants/dewatt/memory/dram_id.generated.txt b/src/mainboard/google/guybrush/variants/dewatt/memory/dram_id.generated.txt
index 6a7b18b..dfd1aac 100644
--- a/src/mainboard/google/guybrush/variants/dewatt/memory/dram_id.generated.txt
+++ b/src/mainboard/google/guybrush/variants/dewatt/memory/dram_id.generated.txt
@@ -16,3 +16,5 @@
K4U6E3S4AB-MGCL 1 (0001)
H54G46CYRBX267 1 (0001)
MT53E512M32D1NP-046 WT:B 1 (0001)
+H54G56CYRBX247 3 (0011)
+K4UBE3D4AB-MGCL 3 (0011)
diff --git a/src/mainboard/google/guybrush/variants/guybrush/memory/mem_parts_used.txt b/src/mainboard/google/guybrush/variants/guybrush/memory/mem_parts_used.txt
index 361e3f5..298e58b 100644
--- a/src/mainboard/google/guybrush/variants/guybrush/memory/mem_parts_used.txt
+++ b/src/mainboard/google/guybrush/variants/guybrush/memory/mem_parts_used.txt
@@ -11,3 +11,4 @@
H54G46CYRBX267
MT53E512M32D1NP-046 WT:B
H54G56CYRBX247
+K4UBE3D4AB-MGCL

2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie5ece849a86c75be5af9bc0393090b5f1e33bfed
Gerrit-Change-Number: 60090
Gerrit-PatchSet: 7
Gerrit-Owner: Kenneth Chan <kenneth.chan@quanta.corp-partner.google.com>
Gerrit-Reviewer: Bhanu Prakash Maiya <bhanumaiya@google.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Paul Menzel <paulepanter@mailbox.org>
Gerrit-Reviewer: Rob Barnes <robbarnes@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged