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;