[coreboot-gerrit] Change in coreboot[master]: lib/spd_bin: Extend DDR4 spd information

[EricR Lai (Code Review)]

EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35206 )

Change subject: lib/spd_bin: Extend DDR4 spd information
......................................................................

Patch Set 13:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35206/13/src/lib/spd_bin.c 
File src/lib/spd_bin.c:

https://review.coreboot.org/c/coreboot/+/35206/13/src/lib/spd_bin.c@46 
PS13, Line 46: case SPD_DRAM_LPDDR3_INTEL:
> I know I'm being nitpicky, but would you mind adding a comment indicating that the fallthrough of th […]

I think here is pretty clear by type name.

https://review.coreboot.org/c/coreboot/+/35206/13/src/lib/spd_bin.c@61 
PS13, Line 61: case SPD_DRAM_DDR3:
             : 	case SPD_DRAM_LPDDR3_INTEL:
> /* intentional fallthrough */ or something like that?

I can add some here, /* DDR3 and LPDDR3 has the same bank definition */