Kevin Chiu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48642 )
Change subject: zork: update gumboz variant ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/48642/2/src/mainboard/google/zork/v... File src/mainboard/google/zork/variants/gumboz/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/48642/2/src/mainboard/google/zork/v... PS2, Line 5: ramstage-y += variant.c
ramstage-y += variant. […]
Done
https://review.coreboot.org/c/coreboot/+/48642/2/src/mainboard/google/zork/v... File src/mainboard/google/zork/variants/gumboz/spd/mem_parts_used.txt:
https://review.coreboot.org/c/coreboot/+/48642/2/src/mainboard/google/zork/v... PS2, Line 17: K4AAG165WA-BCWE, 10
gumboz will be a different firmware binary, so I don't think it's required that the table aligns wit […]
sorry that gumboz already has it's SMT proto board with the same DRAM table as dirinboz. here is the DRAM definition from EE on this tracker: b/173662179, thanks.