Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31851 )
Change subject: mb/google/hatch: Provide DRAM part number from EEPROM ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/#/c/31851/6/src/mainboard/google/hatch/romstage.... File src/mainboard/google/hatch/romstage.c:
https://review.coreboot.org/#/c/31851/6/src/mainboard/google/hatch/romstage.... PS6, Line 58: *len = strlen(part_num_store); Does this get padded out in the other parts of the stack? Or do we need to +1 ?