Furquan Shaikh 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:
There is already a SPD_LPDDR4X_200b_2R_32Gb_QDP_4267 spd. […]
So, there are two different memory parts available from vendors which support different attributes? I am still not sure I follow the differences here. Are the differences in bytes you highlighted based on the speed being 4266 v/s 4267? And does that align with what memory vendor supports?
--
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 19:20:51 +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