Marshall Dawson has uploaded this change for review. ( https://review.coreboot.org/28554
Change subject: amd/stoneyridge: Set BERT region size when no TSEG used ......................................................................
amd/stoneyridge: Set BERT region size when no TSEG used
Expand the BERT reserved region size setting to account for the possibility of no TSEG configuration. This change is only for completeness, as stoneyridge must always use TSEG.
Change-Id: I90753fa408cfac4de38aff08979c45349bb62a66 Signed-off-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/soc/amd/stoneyridge/ramtop.c 1 file changed, 7 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/28554/1
diff --git a/src/soc/amd/stoneyridge/ramtop.c b/src/soc/amd/stoneyridge/ramtop.c index e9a0e53..72db8ec 100644 --- a/src/soc/amd/stoneyridge/ramtop.c +++ b/src/soc/amd/stoneyridge/ramtop.c @@ -38,10 +38,14 @@ }
#if IS_ENABLED(CONFIG_ACPI_BERT) -/* SMM_TSEG_SIZE must stay on a boundary appropriate for its granularity */ -#define BERT_REGION_MAX_SIZE CONFIG_SMM_TSEG_SIZE + #if CONFIG_SMM_TSEG_SIZE == 0x0 + #define BERT_REGION_MAX_SIZE 0x100000 + #else + /* SMM_TSEG_SIZE must stay on a boundary appropriate for its granularity */ + #define BERT_REGION_MAX_SIZE CONFIG_SMM_TSEG_SIZE + #endif #else -#define BERT_REGION_MAX_SIZE 0 + #define BERT_REGION_MAX_SIZE 0 #endif
void bert_reserved_region(void **start, size_t *size)