diff options
| author | Timothy Pearson <tpearson@raptorengineeringinc.com> | 2015-03-19 00:03:59 -0500 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2015-03-19 08:27:09 +0100 |
| commit | e24f7d37cef5acb71c070f934a74286efb6ee32e (patch) | |
| tree | 778a90ca39c8f970659cddd584075391783321b1 /src | |
| parent | 4da487eae8858dc132d60b7ffb0015e9c96b0df6 (diff) | |
| download | coreboot-e24f7d37cef5acb71c070f934a74286efb6ee32e.tar.xz | |
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>
Diffstat (limited to 'src')
| -rw-r--r-- | src/cpu/amd/model_10xxx/ram_calc.c | 6 |
1 files 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 98b80e96b9..c37c0ce41d 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 */ } |