Tristan Corrick has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43094 )
Change subject: nb/intel/haswell: Add `mb_post_raminit_setup` function ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Perhaps a name such as `mb_romstage_final` would be more appropriate? If the T440p PEG code runs after raminit, it will have to come after `haswell_unhide_peg` too.
https://review.coreboot.org/c/coreboot/+/43094/1/src/mainboard/lenovo/t440p/... File src/mainboard/lenovo/t440p/romstage.c:
https://review.coreboot.org/c/coreboot/+/43094/1/src/mainboard/lenovo/t440p/... PS1, Line 29: mainboard_config_superio It would be better to drop this function in a separate commit.