[coreboot-gerrit] Change in coreboot[master]: mb/google/drallion: memory spd data debug

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35157 )

Change subject: mb/google/drallion: memory spd data debug
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35157/4/src/mainboard/google/dralli... 
File src/mainboard/google/drallion/romstage.c:

https://review.coreboot.org/c/coreboot/+/35157/4/src/mainboard/google/dralli... 
PS4, Line 54: print_spd_info
If you add call to print_spd_info() here: https://review.coreboot.org/cgit/coreboot.git/tree/src/soc/intel/cannonlake/..., then you wouldn't need all the logic to read SPD file again here in mainboard. Also, is there really a need to guard this with debug? I think its okay to print SPD info in SoC since it can be helpful.