[coreboot-gerrit] Change in coreboot[master]: mb/google/zork: Update SPD table for Shuboz

[Kane Chen (Code Review)]

Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47902 )

Change subject: mb/google/zork: Update SPD table for Shuboz
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47902/1/src/mainboard/google/zork/v... 
File src/mainboard/google/zork/variants/shuboz/spd/mem_parts_used.txt:

https://review.coreboot.org/c/coreboot/+/47902/1/src/mainboard/google/zork/v... 
PS1, Line 10:  0
> Is there a bug where this was discussed?

Hi Furquan,
We try remove Fixed ID to generate SPD table, the result is as follows:
SPD_SOURCES =
SPD_SOURCES += ddr4-spd-1.hex      # ID = 0(0b0000)  Parts = MT40A512M16TB-062E:J, H5AN8G6NCJR-XNC
SPD_SOURCES += ddr4-spd-7.hex      # ID = 1(0b0001)  Parts = MT40A1G16KD-062E:E, K4AAG165WA-BCWE
We think the result does not meet, so add Fixed ID to generate, or we need to change to this result?
Thanks.