[coreboot-gerrit] Change in coreboot[master]: mb/google/volteer: add generic DDR4 SPDs for Eldrid

[Nick Vaccaro (Code Review)]

Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44272 )

Change subject: mb/google/volteer: add generic DDR4 SPDs for Eldrid
......................................................................

Patch Set 67:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44272/66/src/mainboard/google/volte... 
File src/mainboard/google/volteer/variants/eldrid/memory/mem_list_variant.txt:

https://review.coreboot.org/c/coreboot/+/44272/66/src/mainboard/google/volte... 
PS66, Line 4: ,3
            : K4AAG165WA-BCWE,4
            : MT40A1G16KD-062E:E,5
> Posted comment on bug too.

They requested specific DRAM IDs when they requested the additional three parts in https://b.corp.google.com/issues/161772961#comment35.  I asked if they really needed them or not, but did not get a response back quickly enough, and after re-reading their comments, it seemed clear they were asking for those parts to be assigned specific IDs, so I made the change (I'm OOO for the next 10 days, so was trying to finish this up before I have to leave, so I moved forward without an answer).

They have responded that what I had before making the IDs match their request was sufficient (0-2 specified, the rest chosen by the tool), so I will revert back to that.