Author: stepan
Date: Thu May 27 17:41:15 2010
New Revision: 5594
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5594

Log:
fix warnings.
Signed-off-by: Stefan Reinauer <[email protected]>
Acked-by: Stefan Reinauer <[email protected]>

Modified:
   trunk/src/devices/oprom/yabel/device.c
   trunk/src/northbridge/intel/i82830/i82830_smihandler.c
   trunk/src/southbridge/intel/i82801dx/i82801dx_smihandler.c

Modified: trunk/src/devices/oprom/yabel/device.c
==============================================================================
--- trunk/src/devices/oprom/yabel/device.c      Wed May 26 19:08:13 2010        
(r5593)
+++ trunk/src/devices/oprom/yabel/device.c      Thu May 27 17:41:15 2010        
(r5594)
@@ -44,7 +44,6 @@
 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 @@
        //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

Modified: trunk/src/northbridge/intel/i82830/i82830_smihandler.c
==============================================================================
--- trunk/src/northbridge/intel/i82830/i82830_smihandler.c      Wed May 26 
19:08:13 2010        (r5593)
+++ trunk/src/northbridge/intel/i82830/i82830_smihandler.c      Thu May 27 
17:41:15 2010        (r5594)
@@ -217,7 +217,7 @@
                                        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 @@
                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 @@
 
                                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;
                        }

Modified: trunk/src/southbridge/intel/i82801dx/i82801dx_smihandler.c
==============================================================================
--- trunk/src/southbridge/intel/i82801dx/i82801dx_smihandler.c  Wed May 26 
19:08:13 2010        (r5593)
+++ trunk/src/southbridge/intel/i82801dx/i82801dx_smihandler.c  Thu May 27 
17:41:15 2010        (r5594)
@@ -449,8 +449,6 @@
        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;

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to