Remove inline from FAM10 CPU initialization functions.
This doesn't save any space for me but it is the right thing to allow GCC to optimize. Signed-off-by: Marc Jones <marc.jones@amd.com> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3266 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
@ -25,21 +25,21 @@
|
||||
// if we are tight of CAR stack, disable it
|
||||
#define FAM10_SET_FIDVID_STORE_AP_APICID_AT_FIRST 1
|
||||
|
||||
static inline void print_debug_fv(const char *str, u32 val)
|
||||
static void print_debug_fv(const char *str, u32 val)
|
||||
{
|
||||
#if FAM10_SET_FIDVID_DEBUG == 1
|
||||
printk_debug("%s%x\n", str, val);
|
||||
#endif
|
||||
}
|
||||
|
||||
static inline void print_debug_fv_8(const char *str, u8 val)
|
||||
static void print_debug_fv_8(const char *str, u8 val)
|
||||
{
|
||||
#if FAM10_SET_FIDVID_DEBUG == 1
|
||||
printk_debug("%s%02x\n", str, val);
|
||||
#endif
|
||||
}
|
||||
|
||||
static inline void print_debug_fv_64(const char *str, u32 val, u32 val2)
|
||||
static void print_debug_fv_64(const char *str, u32 val, u32 val2)
|
||||
{
|
||||
#if FAM10_SET_FIDVID_DEBUG == 1
|
||||
printk_debug("%s%x%x\n", str, val, val2);
|
||||
|
@ -30,24 +30,24 @@
|
||||
#define FAM10_SET_FIDVID_CORE0_ONLY 0
|
||||
#endif
|
||||
|
||||
static inline void print_initcpu8 (const char *strval, u8 val)
|
||||
static void print_initcpu8 (const char *strval, u8 val)
|
||||
{
|
||||
printk_debug("%s%02x\n", strval, val);
|
||||
}
|
||||
|
||||
static inline void print_initcpu8_nocr (const char *strval, u8 val)
|
||||
static void print_initcpu8_nocr (const char *strval, u8 val)
|
||||
{
|
||||
printk_debug("%s%02x", strval, val);
|
||||
}
|
||||
|
||||
|
||||
static inline void print_initcpu16 (const char *strval, u16 val)
|
||||
static void print_initcpu16 (const char *strval, u16 val)
|
||||
{
|
||||
printk_debug("%s%04x\n", strval, val);
|
||||
}
|
||||
|
||||
|
||||
static inline void print_initcpu(const char *strval, u32 val)
|
||||
static void print_initcpu(const char *strval, u32 val)
|
||||
{
|
||||
printk_debug("%s%08x\n", strval, val);
|
||||
}
|
||||
@ -59,7 +59,7 @@ static void init_fidvid_stage2(u32 apicid, u32 nodeid);
|
||||
void cpuSetAMDMSR(void);
|
||||
|
||||
#if PCI_IO_CFG_EXT == 1
|
||||
static inline void set_EnableCf8ExtCfg(void)
|
||||
static void set_EnableCf8ExtCfg(void)
|
||||
{
|
||||
// set the NB_CFG[46]=1;
|
||||
msr_t msr;
|
||||
@ -69,7 +69,7 @@ static inline void set_EnableCf8ExtCfg(void)
|
||||
wrmsr(NB_CFG_MSR, msr);
|
||||
}
|
||||
#else
|
||||
static inline void set_EnableCf8ExtCfg(void) { }
|
||||
static void set_EnableCf8ExtCfg(void) { }
|
||||
#endif
|
||||
|
||||
|
||||
@ -78,7 +78,7 @@ static inline void set_EnableCf8ExtCfg(void) { }
|
||||
/* because we will use gs to store hi, so need to make sure lo can start
|
||||
from 0, So PCI_MMIO_BASE & 0x00ffffff should be equal to 0*/
|
||||
|
||||
static inline void set_pci_mmio_conf_reg(void)
|
||||
static void set_pci_mmio_conf_reg(void)
|
||||
{
|
||||
#if MMCONF_SUPPORT
|
||||
msr_t msr;
|
||||
@ -184,7 +184,7 @@ static void for_each_ap(u32 bsp_apicid, u32 core_range,
|
||||
}
|
||||
|
||||
/* FIXME: Duplicate of what is in lapic.h? */
|
||||
static inline int lapic_remote_read(int apicid, int reg, u32 *pvalue)
|
||||
static int lapic_remote_read(int apicid, int reg, u32 *pvalue)
|
||||
{
|
||||
int timeout;
|
||||
u32 status;
|
||||
|
@ -24,7 +24,7 @@
|
||||
|
||||
#include "cpu/amd/quadcore/quadcore_id.c"
|
||||
|
||||
static inline u32 get_core_num_in_bsp(u32 nodeid)
|
||||
static u32 get_core_num_in_bsp(u32 nodeid)
|
||||
{
|
||||
u32 dword;
|
||||
dword = pci_read_config32(NODE_PCI(nodeid, 3), 0xe8);
|
||||
@ -34,7 +34,7 @@ static inline u32 get_core_num_in_bsp(u32 nodeid)
|
||||
}
|
||||
|
||||
#if SET_NB_CFG_54 == 1
|
||||
static inline u8 set_apicid_cpuid_lo(void)
|
||||
static u8 set_apicid_cpuid_lo(void)
|
||||
{
|
||||
// set the NB_CFG[54]=1; why the OS will be happy with that ???
|
||||
msr_t msr;
|
||||
@ -46,12 +46,12 @@ static inline u8 set_apicid_cpuid_lo(void)
|
||||
}
|
||||
#else
|
||||
|
||||
static inline void set_apicid_cpuid_lo(void) { }
|
||||
static void set_apicid_cpuid_lo(void) { }
|
||||
|
||||
#endif
|
||||
|
||||
|
||||
static inline void real_start_other_core(u32 nodeid, u32 cores)
|
||||
static void real_start_other_core(u32 nodeid, u32 cores)
|
||||
{
|
||||
u32 dword;
|
||||
|
||||
@ -78,7 +78,7 @@ static inline void real_start_other_core(u32 nodeid, u32 cores)
|
||||
}
|
||||
|
||||
//it is running on core0 of node0
|
||||
static inline void start_other_cores(void)
|
||||
static void start_other_cores(void)
|
||||
{
|
||||
u32 nodes;
|
||||
u32 nodeid;
|
||||
|
@ -32,7 +32,7 @@ u32 read_nb_cfg_54(void)
|
||||
return ( ( msr.hi >> (54-32)) & 1);
|
||||
}
|
||||
|
||||
static inline u32 get_initial_apicid(void)
|
||||
static u32 get_initial_apicid(void)
|
||||
{
|
||||
return ((cpuid_ebx(1) >> 24) & 0xff);
|
||||
}
|
||||
@ -67,12 +67,12 @@ struct node_core_id get_node_core_id(u32 nb_cfg_54)
|
||||
return id;
|
||||
}
|
||||
|
||||
static inline u32 get_core_num(void)
|
||||
static u32 get_core_num(void)
|
||||
{
|
||||
return (cpuid_ecx(0x80000008) & 0xff);
|
||||
}
|
||||
|
||||
static inline struct node_core_id get_node_core_id_x(void) {
|
||||
static struct node_core_id get_node_core_id_x(void) {
|
||||
|
||||
return get_node_core_id( read_nb_cfg_54() );
|
||||
}
|
||||
|
Reference in New Issue
Block a user