Get rid of a few more warnings.

Signed-off-by: Myles Watson <mylesgw@gmail.com>
Acked-by: Myles Watson <mylesgw@gmail.com>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5512 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

diff --git a/src/mainboard/arima/hdama/romstage.c b/src/mainboard/arima/hdama/romstage.c
index 7be6cc6a9673589db91d16318573ea9adc2efe02..dfae779412777207d427327ecc703df1b3892f3b 100644 (file)
--- a/src/mainboard/arima/hdama/romstage.c
+++ b/src/mainboard/arima/hdama/romstage.c
@@ -1,10 +1,10 @@
 #include <stdint.h>
+#include <string.h>
 #include <device/pci_def.h>
 #include <arch/io.h>
 #include <device/pnp_def.h>
 #include <arch/romcc_io.h>
 #include <cpu/x86/lapic.h>
-#include <stdlib.h>
 #include "option_table.h"
 #include "pc80/mc146818rtc_early.c"
 #include "pc80/serial.c"

diff --git a/src/mainboard/pcengines/alix1c/romstage.c b/src/mainboard/pcengines/alix1c/romstage.c
index a9b6db5a6c5a7212a60317fb49b60dc69e1e3f90..ded187d6331d015a1f93d03f3f3e27147d012a6d 100644 (file)
--- a/src/mainboard/pcengines/alix1c/romstage.c
+++ b/src/mainboard/pcengines/alix1c/romstage.c
@@ -36,7 +36,7 @@
 #define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1)
 
 /* The ALIX1.C has no SMBus; the setup is hard-wired. */
-void cs5536_enable_smbus(void)
+static void cs5536_enable_smbus(void)
 {
 }
 

diff --git a/src/northbridge/amd/amdfam10/Makefile.inc b/src/northbridge/amd/amdfam10/Makefile.inc
index 54dc2e97b4000fe47f0ce9cafbc8ac9f395b59fd..c73f82a52c0879ce552cd012dcb4e06baaafb15c 100644 (file)
--- a/src/northbridge/amd/amdfam10/Makefile.inc
+++ b/src/northbridge/amd/amdfam10/Makefile.inc
@@ -10,4 +10,3 @@ obj-$(CONFIG_GENERATE_ACPI_TABLES) += sspr4.o
 obj-$(CONFIG_GENERATE_ACPI_TABLES) += sspr5.o
 
 obj-y += get_pci1234.o
-

diff --git a/src/northbridge/amd/amdfam10/debug.c b/src/northbridge/amd/amdfam10/debug.c
index cab2ec86d9273f7f88e68dc01551626358bc6469..7aa7751820b097566c35209c45813160b56128c9 100644 (file)
--- a/src/northbridge/amd/amdfam10/debug.c
+++ b/src/northbridge/amd/amdfam10/debug.c
@@ -26,7 +26,7 @@
 
 static inline void print_debug_addr(const char *str, void *val)
 {
-#if CACHE_AS_RAM_ADDRESS_DEBUG == 1
+#if defined(CACHE_AS_RAM_ADDRESS_DEBUG) && CACHE_AS_RAM_ADDRESS_DEBUG == 1
                printk(BIOS_DEBUG, "------Address debug: %s%p------\n", str, val);
 #endif
 }

diff --git a/src/northbridge/amd/amdfam10/get_pci1234.c b/src/northbridge/amd/amdfam10/get_pci1234.c
index fa2e56065db597160bfa0131c9a4b35699d2601f..3c6143be2fef4735fa28b34de7f3759b8f08111d 100644 (file)
--- a/src/northbridge/amd/amdfam10/get_pci1234.c
+++ b/src/northbridge/amd/amdfam10/get_pci1234.c
@@ -55,6 +55,7 @@
  *
  */
 
+#include "northbridge.h"
 
 void get_pci1234(void)
 {

diff --git a/src/northbridge/amd/amdfam10/northbridge.h b/src/northbridge/amd/amdfam10/northbridge.h
index 053ac3bd02c4bdbd6f4139cc60d8be3830b4bbb8..c30c99f6c9ec54c7497a0c9a1b08ff1267f6d9f1 100644 (file)
--- a/src/northbridge/amd/amdfam10/northbridge.h
+++ b/src/northbridge/amd/amdfam10/northbridge.h
@@ -21,5 +21,6 @@
 #define NORTHBRIDGE_AMD_AMDFAM10_H
 
 u32 amdfam10_scan_root_bus(device_t root, u32 max);
+void get_pci1234(void);
 
 #endif /* NORTHBRIDGE_AMD_AMDFAM10_H */

diff --git a/src/northbridge/amd/amdk8/raminit_f.c b/src/northbridge/amd/amdk8/raminit_f.c
index b89aa38d6a63e102643297a70542b0b0a711fad2..6652783e6648b9478d49a931ae0eece00962b282 100644 (file)
--- a/src/northbridge/amd/amdk8/raminit_f.c
+++ b/src/northbridge/amd/amdk8/raminit_f.c
@@ -2511,9 +2511,8 @@ static void set_misc_timing(const struct mem_controller *ctrl, struct mem_info *
        unsigned SlowAccessMode = 0;
 #endif
 
-       long dimm_mask = meminfo->dimm_mask & 0x0f;
-
 #if CONFIG_DIMM_SUPPORT==0x0104   /* DDR2 and REG */
+       long dimm_mask = meminfo->dimm_mask & 0x0f;
        /* for REG DIMM */
        dword = 0x00111222;
        dwordx = 0x002f0000;
@@ -2578,6 +2577,7 @@ static void set_misc_timing(const struct mem_controller *ctrl, struct mem_info *
 #endif
 
 #if CONFIG_DIMM_SUPPORT==0x0004  /* DDR2 and unbuffered */
+       long dimm_mask = meminfo->dimm_mask & 0x0f;
        /* for UNBUF DIMM */
        dword = 0x00111222;
        dwordx = 0x002f2f00;

diff --git a/src/northbridge/amd/amdk8/raminit_f_dqs.c b/src/northbridge/amd/amdk8/raminit_f_dqs.c
index c56e51deb05d2dcec477c42ec24aca4a580ee10b..81e38ecc6d9946cf01c7baa8bd19f9dbeb23e48d 100644 (file)
--- a/src/northbridge/amd/amdk8/raminit_f_dqs.c
+++ b/src/northbridge/amd/amdk8/raminit_f_dqs.c
@@ -528,7 +528,7 @@ static unsigned TrainRcvrEn(const struct mem_controller *ctrl, unsigned Pass, st
        unsigned is_Width128 = sysinfo->meminfo[ctrl->node_id].is_Width128;
 
 #if K8_REV_F_SUPPORT_F0_F1_WORKAROUND == 1
-       unsigned cpu_f0_f1;
+       unsigned cpu_f0_f1 = 0;
 #endif
 
        if(Pass == DQS_FIRST_PASS) {

diff --git a/src/northbridge/amd/amdmct/mct/mct_d.c b/src/northbridge/amd/amdmct/mct/mct_d.c
index 3c4fa89e896b36f474456a579a83ade0d9448abc..c56576aa6c9772f5062972f66ef5855ca36527d4 100644 (file)
--- a/src/northbridge/amd/amdmct/mct/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct/mct_d.c
@@ -42,8 +42,6 @@ static void DQSTiming_D(struct MCTStatStruc *pMCTstat,
                                struct DCTStatStruc *pDCTstatA);
 static void LoadDQSSigTmgRegs_D(struct MCTStatStruc *pMCTstat,
                                        struct DCTStatStruc *pDCTstatA);
-static void ResetNBECCstat_D(struct MCTStatStruc *pMCTstat,
-                               struct DCTStatStruc *pDCTstatA);
 static void HTMemMapInit_D(struct MCTStatStruc *pMCTstat,
                                struct DCTStatStruc *pDCTstatA);
 static void MCTMemClr_D(struct MCTStatStruc *pMCTstat,
@@ -477,6 +475,8 @@ static void LoadDQSSigTmgRegs_D(struct MCTStatStruc *pMCTstat,
 }
 
 #ifdef UNUSED_CODE
+static void ResetNBECCstat_D(struct MCTStatStruc *pMCTstat,
+                               struct DCTStatStruc *pDCTstatA);
 static void ResetNBECCstat_D(struct MCTStatStruc *pMCTstat,
                                struct DCTStatStruc *pDCTstatA)
 {

diff --git a/src/northbridge/intel/i3100/raminit.c b/src/northbridge/intel/i3100/raminit.c
index 242dcf9105ac89fb93c88912f9e5ad970dbab3c3..e2c30f0d1bf8809a8340ee9fb0ba3410a7d87d81 100644 (file)
--- a/src/northbridge/intel/i3100/raminit.c
+++ b/src/northbridge/intel/i3100/raminit.c
@@ -944,7 +944,7 @@ static void sdram_enable(int controllers, const struct mem_controller *ctrl)
        u32 drc;
        u32 data32;
        u32 mode_reg;
-       u32 *iptr;
+       u32 const *iptr;
        u16 data16;
        static const struct {
                u32 clkgr[4];

diff --git a/src/northbridge/via/vx800/vga.c b/src/northbridge/via/vx800/vga.c
index 7bdc3418f1d86bff1f963b16b0424299fa2dd422..cd337708231a5db052804b9ca1756fc2a2d8944f 100644 (file)
--- a/src/northbridge/via/vx800/vga.c
+++ b/src/northbridge/via/vx800/vga.c
@@ -126,6 +126,7 @@ static int via_vx800_int15_handler(struct eregs *regs)
        return res;
 }
 
+#ifdef UNUSED_CODE
 void write_protect_vgabios(void)
 {
        device_t dev;
@@ -141,6 +142,7 @@ void write_protect_vgabios(void)
           //if(dev)
           //   pci_write_config8(dev, 0x61, 0xff); */
 }
+#endif
 
 extern u8 acpi_sleep_type;
 static void vga_init(device_t dev)

diff --git a/src/southbridge/intel/esb6300/esb6300_smbus.h b/src/southbridge/intel/esb6300/esb6300_smbus.h
index e7a0d5c711520e6936c7624ed8a6bf4016a91f66..4f4ec5c9998dff51a0e428cd7993657d43e75c0f 100644 (file)
--- a/src/southbridge/intel/esb6300/esb6300_smbus.h
+++ b/src/southbridge/intel/esb6300/esb6300_smbus.h
@@ -14,6 +14,8 @@
 
 #define SMBUS_TIMEOUT (100*1000*10)
 
+#include <delay.h>
+
 static int smbus_wait_until_ready(unsigned smbus_io_base)
 {
        unsigned loops = SMBUS_TIMEOUT;

diff --git a/src/southbridge/nvidia/mcp55/mcp55_early_setup_car.c b/src/southbridge/nvidia/mcp55/mcp55_early_setup_car.c
index 60337d16351186b9d7dc22be4affddc2df6dbee7..fb84f5a36dc9f288092b600fad09233abc7e433b 100644 (file)
--- a/src/southbridge/nvidia/mcp55/mcp55_early_setup_car.c
+++ b/src/southbridge/nvidia/mcp55/mcp55_early_setup_car.c
@@ -19,9 +19,10 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
-static int set_ht_link_buffer_counts_chain(uint8_t ht_c_num, unsigned vendorid, unsigned val);
 
 #ifdef UNUSED_CODE
+int set_ht_link_buffer_counts_chain(uint8_t ht_c_num, unsigned vendorid, unsigned val);
+
 static int set_ht_link_mcp55(uint8_t ht_c_num)
 {
        unsigned vendorid = 0x10de;