Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40836 )
Change subject: mb/google/dedede: Read DRAM part number from CBI ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/40836/2/src/mainboard/google/dedede... File src/mainboard/google/dedede/romstage.c:
https://review.coreboot.org/c/coreboot/+/40836/2/src/mainboard/google/dedede... PS2, Line 42: strlen(part_num_store) Did you mean to use sizeof(part_num_store) instead of strlen?