GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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@raptorengineeringinc.com>
Reviewed-on: http://review.coreboot.org/8748
Tested-by: build bot (Jenkins)
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Timothy Pearson 2015-03-19 00:03:59 -05:00 committed by Kyösti Mälkki
parent 4da487eae8
commit e24f7d37ce
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/cpu/amd/model_10xxx/ram_calc.c
View File

@ -29,13 +29,13 @@ uint64_t get_uma_memory_size(uint64_t topmem)
uint64_t uma_size = 0; uint64_t uma_size = 0;
if (IS_ENABLED(CONFIG_GFXUMA)) { if (IS_ENABLED(CONFIG_GFXUMA)) {
/* refer to UMA Size Consideration in 780 BDG. */ /* 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 */ 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 */ 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 */ uma_size = 0x4000000; /* 64M recommended UMA */
} }