Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40463 )
Change subject: mb/google/volteer: add generic SPDs
......................................................................
Patch Set 16:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40463/16/src/mainboard/google/volte...
File src/mainboard/google/volteer/spd/SPD_LPDDR4X_200b_2R_32Gb_QDP_4266.spd.hex:
PS16:
Yes, Micron/Hynix (SPD_LPDDR4X_200b_2R_32Gb_QDP_4267) and Samsung (SPD_LPDDR4X_200b_2R_32Gb_QDP_4266 […]
Intel simplified the naming by adding "Samsung" and "MicronHynix" to the filename as differentiators, but we strip those vendor names from the generic SPD filename.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/40463
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib4cd148e28efe458f2b312a7f1c5b6d71e8ef433
Gerrit-Change-Number: 40463
Gerrit-PatchSet: 16
Gerrit-Owner: Nick Vaccaro
nvaccaro@google.com
Gerrit-Reviewer: Caveh Jalali
caveh@chromium.org
Gerrit-Reviewer: Dossym Nurmukhanov
dossym@google.com
Gerrit-Reviewer: Duncan Laurie
dlaurie@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Raj Astekar
raj.astekar@intel.com
Gerrit-Reviewer: Shreesh Chhabbi
shreesh.chhabbi@intel.corp-partner.google.com
Gerrit-Reviewer: Srinidhi N Kaushik
srinidhi.n.kaushik@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Mon, 27 Apr 2020 22:12:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh
furquan@google.com
Comment-In-Reply-To: Nick Vaccaro
nvaccaro@google.com
Gerrit-MessageType: comment