Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41345 )
Change subject: mb/google/volteer: fix error in generic SPD ......................................................................
Patch Set 9:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41345/9/src/mainboard/google/voltee... File src/mainboard/google/volteer/variants/volteer/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/41345/9/src/mainboard/google/voltee... PS9, Line 9: MT53E1G32D2NP-046 WT:A
Numbers were collapsed to be sequential, and I notified variant projects planning to use said parts, […]
Do you have all the tables captured in one place regarding what is provided to ODMs? I want to make sure that we can keep using the same IDs, but for that we need to ensure that the memory part used list (https://review.coreboot.org/c/coreboot/+/41617/8/src/mainboard/google/voltee...) matches what was assigned before. Also, do we know by when we can make changes to the ID numbers?
https://review.coreboot.org/c/coreboot/+/41345/9/src/mainboard/google/voltee... PS9, Line 5: K4U6E3S4AA-MGCL : # H9HCNNNBKMMLXR-NEE
That implies that the SPD for our Ripto memory part is incorrect, yet we haven't seen any issues wit […]
That is not what the datasheet says though. As per datasheet, K4U6E3S4AA-MGCL is a single die package: https://drive.google.com/corp/drive/u/0/search?q=K4U6E3S4AA-MGCL
The 3S stands for monolithic package.