Move CACHE_AS_RAM_ADDRESS_DEBUG out of romstage.c into Kconfig,
rename it slightly, make it visible only on relevant northbridges, drop it entirely from via boards (as they seem to have picked it up from AMD code without using it themselves), and make it default to false for all boards. Some romstages used to set this to "true" (ie. "print debug output"), but I didn't follow up on it in Kconfig - if you need it to debug CAR, enable it yourself. Signed-off-by: Patrick Georgi <patrick.georgi@coresystems.de> Acked-by: Peter Stuge <peter@stuge.se> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5898 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
parent
f11b81d18d
commit
e82618d037
10
src/Kconfig
10
src/Kconfig
|
@ -499,6 +499,16 @@ config DEBUG_RAM_SETUP
|
|||
|
||||
If unsure, say N.
|
||||
|
||||
config HAVE_DEBUG_CAR
|
||||
def_bool n
|
||||
|
||||
config DEBUG_CAR
|
||||
bool "Output verbose Cache-as-RAM debug messages"
|
||||
default n
|
||||
depends on HAVE_DEBUG_CAR
|
||||
help
|
||||
This option enables additional CAR related debug messages.
|
||||
|
||||
config DEBUG_PIRQ
|
||||
bool "Check PIRQ table consistency"
|
||||
default n
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -2,7 +2,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -1,5 +1,4 @@
|
|||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -24,8 +24,6 @@
|
|||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
unsigned int get_sbdn(unsigned bus);
|
||||
|
||||
/* Used by raminit. */
|
||||
|
|
|
@ -24,8 +24,6 @@
|
|||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
unsigned int get_sbdn(unsigned bus);
|
||||
|
||||
/* Used by raminit. */
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -25,7 +25,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -22,7 +22,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -1,5 +1,4 @@
|
|||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -1,5 +1,4 @@
|
|||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -1,5 +1,4 @@
|
|||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
//#define SYSTEM_TYPE 2 /* MOBILE */
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -24,7 +24,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
#define SET_NB_CFG_54 1 /* Used by RAM init. */
|
||||
#define QRANK_DIMM_SUPPORT 1
|
||||
#define K8_REV_F_SUPPORT_F0_F1_WORKAROUND 0
|
||||
|
|
|
@ -20,7 +20,6 @@
|
|||
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
|
||||
*/
|
||||
|
||||
// #define CACHE_AS_RAM_ADDRESS_DEBUG 1
|
||||
// #define RAM_TIMING_DEBUG 1
|
||||
// #define DQS_TRAIN_DEBUG 1
|
||||
// #define RES_DEBUG 1
|
||||
|
|
|
@ -24,7 +24,6 @@
|
|||
*/
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
*/
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
#define __PRE_RAM__
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#define SET_NB_CFG_54 1
|
||||
|
||||
|
|
|
@ -23,7 +23,6 @@
|
|||
*/
|
||||
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
#define PAYLOAD_IS_SEABIOS 0
|
||||
|
||||
#include <stdint.h>
|
||||
|
|
|
@ -21,6 +21,7 @@ config NORTHBRIDGE_AMD_AMDFAM10
|
|||
bool
|
||||
select HAVE_DEBUG_RAM_SETUP
|
||||
select HAVE_DEBUG_SMBUS
|
||||
select HAVE_DEBUG_CAR
|
||||
select HYPERTRANSPORT_PLUGIN_SUPPORT
|
||||
select NORTHBRIDGE_AMD_AMDFAM10_ROOT_COMPLEX
|
||||
select MMCONF_SUPPORT
|
||||
|
|
|
@ -26,7 +26,7 @@
|
|||
|
||||
static inline void print_debug_addr(const char *str, void *val)
|
||||
{
|
||||
#if defined(CACHE_AS_RAM_ADDRESS_DEBUG) && CACHE_AS_RAM_ADDRESS_DEBUG == 1
|
||||
#if CONFIG_DEBUG_CAR
|
||||
printk(BIOS_DEBUG, "------Address debug: %s%p------\n", str, val);
|
||||
#endif
|
||||
}
|
||||
|
|
|
@ -21,6 +21,7 @@ config NORTHBRIDGE_AMD_AMDK8
|
|||
bool
|
||||
select HAVE_DEBUG_RAM_SETUP
|
||||
select HAVE_DEBUG_SMBUS
|
||||
select HAVE_DEBUG_CAR
|
||||
select HYPERTRANSPORT_PLUGIN_SUPPORT
|
||||
|
||||
config AGP_APERTURE_SIZE
|
||||
|
|
|
@ -3,13 +3,9 @@
|
|||
*
|
||||
*/
|
||||
|
||||
#ifndef CACHE_AS_RAM_ADDRESS_DEBUG
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
#endif
|
||||
|
||||
static inline void print_debug_addr(const char *str, void *val)
|
||||
{
|
||||
#if CACHE_AS_RAM_ADDRESS_DEBUG == 1
|
||||
#if CONFIG_DEBUG_CAR
|
||||
printk(BIOS_DEBUG, "------Address debug: %s%x------\n", str, val);
|
||||
#endif
|
||||
}
|
||||
|
|
|
@ -21,7 +21,6 @@
|
|||
#define ASSEMBLY 1
|
||||
#define __PRE_RAM__
|
||||
#define RAMINIT_SYSINFO 1
|
||||
#define CACHE_AS_RAM_ADDRESS_DEBUG 0
|
||||
|
||||
#include <stdint.h>
|
||||
#include <device/pci_def.h>
|
||||
|
|
Loading…
Reference in New Issue