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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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 66:

(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
> Because it was requested.  Please see bug.

These parts are being requested for next build. You can just let the tool assign the IDs for the new parts and let the ODM know what IDs to use. The only reason we have used fixed IDs for parts 0,1,2 is to maintain backward compatibility with last build. Can you please drop the fixed IDs and provide the updated table to ODM to use for the next build.