Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34737 )
Change subject: intel/haswell: Use smm_subregion() ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/34737/1/src/northbridge/intel/haswe... File src/northbridge/intel/haswell/memmap.c:
https://review.coreboot.org/c/coreboot/+/34737/1/src/northbridge/intel/haswe... PS1, Line 42: ALIGN_DOWN
Isn't this already done above in smm_region_start()?
Done