[coreboot-gerrit] Change in coreboot[master]: util/apcb: Strip SPD manufacturer information

[Rob Barnes (Code Review)]

Rob Barnes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43832 )

Change subject: util/apcb: Strip SPD manufacturer information
......................................................................

util/apcb: Strip SPD manufacturer information

Manufacturer information from SPDs before injecting into APCB. This
allows more flexibilty around changing DRAM modules in the future.

BUG=b:162098961
TEST=Boot, dump memory info

Signed-off-by: Rob Barnes 
Change-Id: I1bbc81a858f381f62dbd38bb57b3df0e6707d647
---
M src/soc/amd/picasso/Makefile.inc
M util/apcb/apcb_edit.py
2 files changed, 13 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/43832/1

diff --git a/src/soc/amd/picasso/Makefile.inc b/src/soc/amd/picasso/Makefile.inc
index b3af1fd..6db130e 100644
--- a/src/soc/amd/picasso/Makefile.inc
+++ b/src/soc/amd/picasso/Makefile.inc
@@ -395,6 +395,7 @@
    $(APCB_MAGIC_BLOB) \
    $@ \
    --hex \
+	--strip_manufacturer_information \
    --spd_0_0 $< \
    --board_id_gpio0 $(APCB_BOARD_ID_GPIO0) \
    --board_id_gpio1 $(APCB_BOARD_ID_GPIO1) \
@@ -408,6 +409,7 @@
    $(APCB_MAGIC_BLOB) \
    $@ \
    --hex \
+	--strip_manufacturer_information \
    --spd_0_0 $< \
    --spd_1_0 $< \
    --board_id_gpio0 $(APCB_BOARD_ID_GPIO0) \
diff --git a/util/apcb/apcb_edit.py b/util/apcb/apcb_edit.py
index e5dd0cd..9d97cd8 100755
--- a/util/apcb/apcb_edit.py
+++ b/util/apcb/apcb_edit.py
@@ -59,6 +59,10 @@
         action='store_true',
         help='SPD input file is hex encoded, binary otherwise')
     parser.add_argument(
+        '--strip_manufacturer_information',
+        action='store_true',
+        help='Strip all manufacturer information from SPD')
+    parser.add_argument(
         '--board_id_gpio0',
         type=int,
         required=True,
@@ -155,6 +159,13 @@
             assert len(spd) == 512, \
                             "Expected SPD to be 512 bytes, got %d" % len(spd)

+            if args.strip_manufacturer_information:
+                print("Stripping manufacturer infromation from SPD")
+                spd = spd[0:320] + b'\x00'*64 + spd[320+64:]
+
+                assert len(spd) == 512, \
+                                "Error while stripping SPD manufacurer infromation"
+
             print("Enabling channel %d, dimm %d and injecting SPD\n" %
                   (spd_ssp.ChannelNumber, spd_ssp.DimmNumber))
             spd_ssp = spd_ssp._replace(SpdValid=True, DimmPresent=True)