build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27872 )
Change subject: cpu/intel/smm: Don't make assumptions on TSEG_SIZE ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/27872/1/src/cpu/intel/smm/gen1/smmrelocate.c File src/cpu/intel/smm/gen1/smmrelocate.c:
https://review.coreboot.org/#/c/27872/1/src/cpu/intel/smm/gen1/smmrelocate.c... PS1, Line 161: ASSERT(CONFIG_IED_REGION_SIZE > params->smram_size); Comparisons should place the constant on the right side of the test
https://review.coreboot.org/#/c/27872/1/src/cpu/intel/smm/gen1/smmrelocate.c... PS1, Line 169: ASSERT(CONFIG_SMM_RESERVED_SIZE > params->smram_size); Comparisons should place the constant on the right side of the test