nb/intel/sandybridge: Do not define tables in a header
Header files are supposed to not make allocations from .bss. Builds
fail if said file is included multiple times. To prevent this from
happening, move the definitions to a C file.
Also, rename raminit_patterns to raminit_tables. This is because more
tables that are not patterns will be added here in subsequent changes.
Tested on Asus P8Z77-V LX2, still boots fine.
Change-Id: If8e3a285ecdc4df9e978ae156be915ced6e1750b
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39754
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index e289070..9eb60c7 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -24,6 +24,7 @@
#include "raminit_native.h"
#include "raminit_common.h"
+#include "raminit_tables.h"
#include "sandybridge.h"
/* FIXME: no ECC support */
@@ -2100,8 +2101,6 @@
return lanes_ok != ((1 << NUM_LANES) - 1);
}
-#include "raminit_patterns.h"
-
static void fill_pattern5(ramctr_timing *ctrl, int channel, int patno)
{
unsigned int i, j;