[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Read DRAM part number from CBI

[Marco Chen (Code Review)]

Marco Chen 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)
> Just think why do we need to return a fixed size which normally is bigger then size of part number i […]

And refer to [1], it is processed by strncpy and MIN(sizeof(dimm->module_part_number), module_part_number_size). If we do sizeof here then no optimization copy there but a duplicated info?

[1] https://chromium.googlesource.com/chromiumos/third_party/coreboot/+/refs/hea...