[coreboot-gerrit] Change in coreboot[master]: amd/stoneyridge: Set BERT region size when no TSEG used

Marshall Dawson (Code Review) gerrit at coreboot.org
Mon Sep 10 22:17:22 CEST 2018


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 at 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)

-- 
To view, visit https://review.coreboot.org/28554
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I90753fa408cfac4de38aff08979c45349bb62a66
Gerrit-Change-Number: 28554
Gerrit-PatchSet: 1
Gerrit-Owner: Marshall Dawson <marshalldawson3rd at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180910/f4ade3ef/attachment.html>


More information about the coreboot-gerrit mailing list