Karthik Ramasubramanian has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63354 )
Change subject: mb/google/guybrush/var/dewatt: Correct samsung part number value in SPD data ......................................................................
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(-)
Approvals: build bot (Jenkins): Verified Rob Barnes: Looks good to me, approved Robert Zieba: Looks good to me, but someone else must approve
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.