summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Reinauer <stepan@coresystems.de>2010-05-27 15:41:15 +0000
committerStefan Reinauer <stepan@openbios.org>2010-05-27 15:41:15 +0000
commitc56e5ad7254ddf7ee9f1f016150de9e28c3ff57e (patch)
tree4b79dcc461ce7d98a85c107b0ca04c90f19f59f8
parent89ef0a9f4114f88dfd4c8053152e3448d971bd92 (diff)
downloadcoreboot-c56e5ad7254ddf7ee9f1f016150de9e28c3ff57e.tar.xz
fix warnings.
Signed-off-by: Stefan Reinauer <stepan@coresystems.de> Acked-by: Stefan Reinauer <stepan@coresystems.de> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5594 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
-rw-r--r--src/devices/oprom/yabel/device.c2
-rw-r--r--src/northbridge/intel/i82830/i82830_smihandler.c8
-rw-r--r--src/southbridge/intel/i82801dx/i82801dx_smihandler.c2
3 files changed, 5 insertions, 7 deletions
diff --git a/src/devices/oprom/yabel/device.c b/src/devices/oprom/yabel/device.c
index 5e5dc2877d..8bfb1fd0c6 100644
--- a/src/devices/oprom/yabel/device.c
+++ b/src/devices/oprom/yabel/device.c
@@ -44,7 +44,6 @@ static void
biosemu_dev_get_addr_info(void)
{
int taa_index = 0;
- int i = 0;
struct resource *r;
u8 bus = bios_device.dev->bus->link;
u16 devfn = bios_device.dev->path.pci.devfn;
@@ -115,6 +114,7 @@ biosemu_dev_get_addr_info(void)
//dump translate_address_array
printf("translate_address_array: \n");
translate_address_t ta;
+ int i;
for (i = 0; i <= taa_last_entry; i++) {
ta = translate_address_array[i];
printf
diff --git a/src/northbridge/intel/i82830/i82830_smihandler.c b/src/northbridge/intel/i82830/i82830_smihandler.c
index f20b099e9c..cf5840296a 100644
--- a/src/northbridge/intel/i82830/i82830_smihandler.c
+++ b/src/northbridge/intel/i82830/i82830_smihandler.c
@@ -217,7 +217,7 @@ static void mbi_call(u8 subf, banner_id_t *banner_id)
printk(BIOS_DEBUG, "%c", mbi_header->name[j]);
printk(BIOS_DEBUG, "' found.\n");
#ifdef DEBUG_SMI_I82830
- dump(banner_id, sizeof(obj_header_t) + 16);
+ dump((u8 *)banner_id, sizeof(obj_header_t) + ALIGN(mbi_header->name_len, 16));
#endif
break;
}
@@ -233,7 +233,7 @@ static void mbi_call(u8 subf, banner_id_t *banner_id)
mbi_header_t *mbi_header = NULL;
printk(BIOS_DEBUG, "|- MBI_GetObject\n");
#ifdef DEBUG_SMI_I82830
- printk(BIOS_DEBUG, "| |- handle = %016lx\n", getobj->handle);
+ printk(BIOS_DEBUG, "| |- handle = %016Lx\n", getobj->handle);
#endif
printk(BIOS_DEBUG, "| |- objnum = %d\n", getobj->objnum);
printk(BIOS_DEBUG, "| |- start = %x\n", getobj->start);
@@ -262,8 +262,8 @@ static void mbi_call(u8 subf, banner_id_t *banner_id)
getobj->banner.retsts = MSH_OK;
#ifdef DEBUG_SMI_I82830
- dump(banner_id, sizeof(*getobj));
- dump(getobj->buffer + OBJ_OFFSET, len);
+ dump((u8 *)banner_id, sizeof(*getobj));
+ dump((u8 *)getobj->buffer + OBJ_OFFSET, len);
#endif
break;
}
diff --git a/src/southbridge/intel/i82801dx/i82801dx_smihandler.c b/src/southbridge/intel/i82801dx/i82801dx_smihandler.c
index 107cf80aea..5b7520f553 100644
--- a/src/southbridge/intel/i82801dx/i82801dx_smihandler.c
+++ b/src/southbridge/intel/i82801dx/i82801dx_smihandler.c
@@ -449,8 +449,6 @@ static void southbridge_smi_gpe0(unsigned int node, smm_state_save_area_t *state
dump_gpe0_status(gpe0_sts);
}
-void __attribute__((weak)) mainboard_smi_gpi(u16 gpi_sts);
-
static void southbridge_smi_gpi(unsigned int node, smm_state_save_area_t *state_save)
{
u16 reg16;