Comments
Patch
===================================================================
@@ -20,6 +20,8 @@
#ifndef CPU_AMD_MODEL_10XXX_MSR_H
#define CPU_AMD_MODEL_10XXX_MSR_H
+#include <cpu/x86/msr.h>
+
#define HWCR_MSR 0xC0010015
#define NB_CFG_MSR 0xC001001f
#define LS_CFG_MSR 0xC0011020
===================================================================
@@ -19,6 +19,7 @@
*/
#include <arch/romcc_io.h>
+#include <device/pnp_def.h>
/* All known/supported SMSC Super I/Os have the same logical device IDs
* for the serial ports (COM1, COM2).
===================================================================
@@ -17,6 +17,7 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
+#include <console/console.h>
#ifndef SET_NB_CFG_54
#define SET_NB_CFG_54 1
===================================================================
@@ -1,6 +1,7 @@
/* 2005.6 by yhlu
* 2006.3 yhlu add copy data from CAR to ram
*/
+#include <string.h>
#include <arch/stages.h>
#include "cpu/amd/car/disable_cache_as_ram.c"
===================================================================
@@ -17,6 +17,8 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
+#include <northbridge/amd/amdmct/amddefs.h>
+#include <cpu/amd/mtrr.h>
/*
* Default MSR and errata settings.
===================================================================
@@ -18,6 +18,12 @@
*/
#include "defaults.h"
+#include <stdlib.h>
+#include <cpu/x86/lapic.h>
+#include <cpu/x86/mtrr.h>
+#include <northbridge/amd/amdfam10/amdfam10.h>
+#include <northbridge/amd/amdht/AsPsDefs.h>
+#include <northbridge/amd/amdht/porting.h>
//it takes the CONFIG_ENABLE_APIC_EXT_ID and CONFIG_APIC_ID_OFFSET and CONFIG_LIFT_BSP_APIC_ID
#ifndef SET_FIDVID
===================================================================
@@ -19,6 +19,7 @@
#include "mct_d.h"
+#include <cpu/amd/mtrr.h>
static void SetMTRRrangeWB_D(u32 Base, u32 *pLimit, u32 *pMtrrAddr);
static void SetMTRRrange_D(u32 Base, u32 *pLimit, u32 *pMtrrAddr, u16 MtrrType);
===================================================================
@@ -17,6 +17,9 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
+#include <cpu/x86/msr.h>
+#include <console/console.h>
+#include <northbridge/amd/amdfam10/amdfam10.h>
/*----------------------------------------------------------------------------
* TYPEDEFS, DEFINITIONS AND MACROS
===================================================================
@@ -8,7 +8,9 @@
typedef int wchar_t;
typedef unsigned int wint_t;
+#ifndef NULL
#define NULL ((void *)0)
+#endif
#define offsetof(TYPE, MEMBER) ((size_t) &((TYPE *)0)->MEMBER)
Hi, as part of the ongoing quest to clean up the code to make it easier to use and adapt, I added a couple of #include statements to AMD code, so it gains some self-sufficiency. These includes are necessary because those files use their definitions. It only works so far because the romstage carefully includes files in the right order so everything is included before it's needed by another file. This change, and similar ones that will come after it, allow us to simplify the romstage.c of boards that use these chipsets (ie. Fam10 for this patch), making it easier to support new boards. Signed-off-by: Patrick Georgi <patrick.georgi@coresystems.de>