Paul Menzel has posted comments on this change. ( https://review.coreboot.org/21129 )
Change subject: nb/intel/*/northbridge.c: add a final() to trigger finalize_smm() at boot time ......................................................................
Patch Set 1:
(2 comments)
Could that method be moved to a generic location?
https://review.coreboot.org/#/c/21129/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/21129/1//COMMIT_MSG@10 PS1, Line 10: do have *has* or *does have*
https://review.coreboot.org/#/c/21129/1//COMMIT_MSG@20 PS1, Line 20: Only the change on sandybridge is well tested.
The change is only well tested on Sandy Bridge.
Maybe add the board/device you used?