[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 58:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/44272/50/src/mainboard/google/volte... 
PS50, Line 2: H5AN8G6NDJR-XNC                0 (0000)
            : MT40A512M16TB-062E:J           1 (0001)
            : H5ANAG6NCMR-XNC                2 (0010)
> Didn't the ODM already have a build using IDs 0,1 and 2? Wouldn't it create a problem if the IDs are […]

I have addressed/ask that on the bug, but have not heard back yet as to whether they needed specific assignment of DRAM IDs or whether they could go with the ones selected by the tool.