[coreboot-gerrit] New patch to review for coreboot: 767e0dd AMD: Drop redundant test for CONFIG_RAMTOP

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Wed Apr 30 08:17:34 CEST 2014


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5620

-gerrit

commit 767e0dd523d252807142e967c76e12c11b101aef
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Wed Apr 30 01:37:58 2014 +0300

    AMD: Drop redundant test for CONFIG_RAMTOP
    
    Same test is already done in x86/mtrr.h.
    
    Change-Id: Ib0785d047567374294b9ee7afc4f4244f9ced926
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/northbridge/amd/amdk8/raminit.c   | 4 ----
 src/northbridge/amd/amdk8/raminit_f.c | 4 ----
 2 files changed, 8 deletions(-)

diff --git a/src/northbridge/amd/amdk8/raminit.c b/src/northbridge/amd/amdk8/raminit.c
index e4fe3df..25eb007 100644
--- a/src/northbridge/amd/amdk8/raminit.c
+++ b/src/northbridge/amd/amdk8/raminit.c
@@ -18,10 +18,6 @@
 #include <arch/early_variables.h>
 struct sys_info sysinfo_car CAR_GLOBAL;
 
-#if (CONFIG_RAMTOP & (CONFIG_RAMTOP -1)) != 0
-# error "CONFIG_RAMTOP must be a power of 2"
-#endif
-
 void setup_resource_map(const unsigned int *register_values, int max)
 {
 	int i;
diff --git a/src/northbridge/amd/amdk8/raminit_f.c b/src/northbridge/amd/amdk8/raminit_f.c
index 9c99250..36e0527 100644
--- a/src/northbridge/amd/amdk8/raminit_f.c
+++ b/src/northbridge/amd/amdk8/raminit_f.c
@@ -43,10 +43,6 @@
 #include <arch/early_variables.h>
 struct sys_info sysinfo_car CAR_GLOBAL;
 
-#if (CONFIG_RAMTOP & (CONFIG_RAMTOP -1)) != 0
-# error "CONFIG_RAMTOP must be a power of 2"
-#endif
-
 #include "f_pci.c"
 
 



More information about the coreboot-gerrit mailing list