Karthik Ramasubramanian submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Rob Barnes: Looks good to me, approved Robert Zieba: Looks good to me, but someone else must approve
mb/google/guybrush/var/dewatt: Correct samsung part number value in SPD data

The value at offset 329 should be:
0x4B -> "K" not 0x48 -> "H" in ASCII code.

BUG=b:224884904
TEST=Build, confirm the part number is matched the corresponding parts

Signed-off-by: Chris.Wang <chris.wang@amd.corp-partner.google.com>
Change-Id: I35dc5f036a29cdf4763389b6425df99ff63bbfa0
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63354
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Robert Zieba <robertzieba@google.com>
Reviewed-by: Rob Barnes <robbarnes@google.com>
---
M src/mainboard/google/guybrush/variants/dewatt/memory/K4UBE3D4AB-MGCL.hex
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/google/guybrush/variants/dewatt/memory/K4UBE3D4AB-MGCL.hex b/src/mainboard/google/guybrush/variants/dewatt/memory/K4UBE3D4AB-MGCL.hex
index 00b26e6..3019147 100644
--- a/src/mainboard/google/guybrush/variants/dewatt/memory/K4UBE3D4AB-MGCL.hex
+++ b/src/mainboard/google/guybrush/variants/dewatt/memory/K4UBE3D4AB-MGCL.hex
@@ -18,7 +18,7 @@
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
-00 00 00 00 00 00 00 00 00 48 34 55 42 45 33 44
+00 00 00 00 00 00 00 00 00 4B 34 55 42 45 33 44
34 41 42 2D 4D 47 43 4C 00 00 00 00 00 00 80 CE
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00

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

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I35dc5f036a29cdf4763389b6425df99ff63bbfa0
Gerrit-Change-Number: 63354
Gerrit-PatchSet: 5
Gerrit-Owner: Chris Wang <chris.wang@amd.corp-partner.google.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub@google.com>
Gerrit-Reviewer: Rob Barnes <robbarnes@google.com>
Gerrit-Reviewer: Robert Zieba <robertzieba@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-MessageType: merged