[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/gm45: Use a common romstage

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31190 )

Change subject: nb/intel/gm45: Use a common romstage
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/#/c/31190/4/src/mainboard/roda/rk9/romstage.c 
File src/mainboard/roda/rk9/romstage.c:

https://review.coreboot.org/#/c/31190/4/src/mainboard/roda/rk9/romstage.c@86 
PS4, Line 86: 0x50
> It would be great to place those values in devicetree, but that's out of scope of this patch.

I think ideally you have a function pointer for the SPD. that way you can also provide them via cbfsfiles if soldered. Given that no board needs it, it is overkill atm.