Attention is currently required from: Shuo Liu.
yuchi.chen@intel.com has posted comments on this change by yuchi.chen@intel.com. ( https://review.coreboot.org/c/coreboot/+/84200?usp=email )
Change subject: soc/intel/common/systemagent: Add Kconfig item HAVE_TSEG_LIMIT_REGISTER ......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/84200/comment/e1612075_63f973e8?usp... : PS1, Line 9: Systemagent assumes GSM region is next to TSEG region, but some SoC may
System agent
Done
https://review.coreboot.org/c/coreboot/+/84200/comment/f6bb815a_e072a246?usp... : PS1, Line 11: On such platforms, TSEG region is limited by the TSEG limit register,
defined
Done
https://review.coreboot.org/c/coreboot/+/84200/comment/c935c23a_49d4076b?usp... : PS1, Line 12: and the default offset for that is (TSEG + 4).
I guess you are meaning the register offset of TSEG limit is TSEG base + 4, right?
Yes