[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: log the memory part name

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45459 )

Change subject: soc/intel/tigerlake: log the memory part name
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45459/7/src/lib/spd_bin.c 
File src/lib/spd_bin.c:

https://review.coreboot.org/c/coreboot/+/45459/7/src/lib/spd_bin.c@196 
PS7, Line 196: 
You could also do:
switch (dram_type) {
   case SPD_DRAM_DDR3:
        name_ptr = (char *) &spd[DDR3_SPD_PART_OFF];
        name_len = DDR3_SPD_PART_LEN;
        break;
   // ... others
}

if (spd_part_name_override) {
    name_ptr = spd_name_part_override;
    name_len = MIN(strlen(spd_part_name_override), name_len);
}