Angel Pons 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 2:
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.
I used the same name as in x4x, and to preserve the original behavior (which may be wrong) I placed the code so that it runs at the same point as before (only difference being the postcode call, which doesn't really matter). In any case, once I turn romstage_common inside-out in CB:43108 it will be easier to move the function call to a better place.