[coreboot-gerrit] Patch set updated for coreboot: f3e9227 cpu/amd/model_10xxx: Fix UMA memory sizes
Timothy Pearson (tpearson@raptorengineeringinc.com)
gerrit at coreboot.org
Thu Mar 19 08:16:51 CET 2015
Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8748
-gerrit
commit f3e922799a70819b1d564c33dafc432f3924cdb0
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date: Thu Mar 19 00:03:59 2015 -0500
cpu/amd/model_10xxx: Fix UMA memory sizes
Fix up commit 4916880 (cpu/amd/model_10xxx: Move GFXUMA size
calculation to separate function) unintentionally changing
behavior when converting the switch statement to an if-else
statement.
Change-Id: I8d126aaec1b324face6407a2b451e603e61db0e5
Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
src/cpu/amd/model_10xxx/ram_calc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/cpu/amd/model_10xxx/ram_calc.c b/src/cpu/amd/model_10xxx/ram_calc.c
index 98b80e9..c37c0ce 100644
--- a/src/cpu/amd/model_10xxx/ram_calc.c
+++ b/src/cpu/amd/model_10xxx/ram_calc.c
@@ -29,13 +29,13 @@ uint64_t get_uma_memory_size(uint64_t topmem)
uint64_t uma_size = 0;
if (IS_ENABLED(CONFIG_GFXUMA)) {
/* refer to UMA Size Consideration in 780 BDG. */
- if (topmem > 0x40000000) /* 1GB and above system memory */
+ if (topmem >= 0x40000000) /* 1GB and above system memory */
uma_size = 0x10000000; /* 256M recommended UMA */
- else if (topmem > 0x20000000) /* 512M - 1023M system memory */
+ else if (topmem >= 0x20000000) /* 512M - 1023M system memory */
uma_size = 0x8000000; /* 128M recommended UMA */
- else if (topmem > 0x10000000) /* 256M - 511M system memory */
+ else if (topmem >= 0x10000000) /* 256M - 511M system memory */
uma_size = 0x4000000; /* 64M recommended UMA */
}
More information about the coreboot-gerrit
mailing list