fix a bunch of cast and type warnings and don't call the apic "nvram", that
authorStefan Reinauer <stepan@coresystems.de>
Fri, 6 Mar 2009 17:24:29 +0000 (17:24 +0000)
committerStefan Reinauer <stepan@openbios.org>
Fri, 6 Mar 2009 17:24:29 +0000 (17:24 +0000)
doesn't make no sense. (trivial)

Signed-off-by: Stefan Reinauer <stepan@coresystems.de>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3977 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/arch/i386/boot/acpi.c
src/arch/i386/boot/multiboot.c
src/arch/i386/boot/tables.c
src/arch/i386/smp/ioapic.c
src/cpu/x86/lapic/lapic_cpu_init.c
src/cpu/x86/tsc/delay_tsc.c
src/devices/device.c

index 40fdf3d36862834a08d30b3a0cc8f5f58c9c80e4..92338046c0d706643b7d2ad95cabab9122d274db 100644 (file)
@@ -202,7 +202,7 @@ void acpi_create_ssdt_generator(acpi_header_t *ssdt, char *oem_table_id)
        ssdt->asl_compiler_revision = 42;
        ssdt->length = sizeof(acpi_header_t);
 
-       acpigen_set_current((unsigned char *) current);
+       acpigen_set_current((char *) current);
        current = acpi_fill_ssdt_generator(current, oem_table_id);
 
        /* recalculate length */
index 31e9e4aa4f43cf73d5aeb989d762d30b2c26d2f6..d331193416735fa9faf9edf1bfce2672cd5489f1 100644 (file)
@@ -72,7 +72,7 @@ static void build_mb_mem_range(void *gp, struct device *dev, struct resource *re
        build_mb_mem_range_nooverlap(res->base, res->size);
 }
 
-#define ROUND(_r,_a) ((_r) + (((_a) - 1)) & ~((_a) - 1))
+#define ROUND(_r,_a) (((_r) + (((_a) - 1))) & ~((_a) - 1))
 
 unsigned long write_multiboot_info(
        unsigned long low_table_start, unsigned long low_table_end,
@@ -81,12 +81,12 @@ unsigned long write_multiboot_info(
        struct multiboot_info *mbi;
        int i;
 
-       mbi = rom_table_end;
+       mbi = (struct multiboot_info *)rom_table_end;
        memset(mbi, 0, sizeof(*mbi));
        rom_table_end += sizeof(*mbi);
 
        mbi->mmap_addr = (u32) rom_table_end;
-       mb_mem = rom_table_end;
+       mb_mem = (struct multiboot_mmap_entry *)rom_table_end;
 
        /* reserved regions */
        reserved_mem[0].addr = low_table_start;
@@ -111,5 +111,5 @@ unsigned long write_multiboot_info(
 
        printk_info("Multiboot Information structure has been written.\n");
 
-       return mb_mem;
+       return (unsigned long)mb_mem;
 }
index e8de68a88a6c614c4456653116288e3211d09a91..71459570f6039cde1805629f3e90adbba19c0d4f 100644 (file)
@@ -182,7 +182,7 @@ struct lb_memory *write_tables(void)
 
 #if CONFIG_MULTIBOOT
        /* The Multiboot information structure */
-       mbi = rom_table_end;
+       mbi = (struct multiboot_info *)rom_table_end;
        rom_table_end = write_multiboot_info(
                                low_table_start, low_table_end,
                                rom_table_start, rom_table_end);
index 4e72f46f035232cb91a10818f14eee5591d4e964..d64b0ad3f3b09cdf5de07f157379b8be232818c7 100644 (file)
@@ -55,11 +55,11 @@ void setup_ioapic(void)
 {
        int i;
        unsigned long value_low, value_high;
-       unsigned long nvram = 0xfec00000;
+       unsigned long ioapicaddr = 0xfec00000;
        volatile unsigned long *l;
        struct ioapicreg *a = ioapicregvalues;
 
-       l = (unsigned long *) nvram;
+       l = (unsigned long *) ioapicaddr;
 #if defined(i786)
        /* For the pentium 4 and above apic deliver their interrupts
         * on the front side bus, enable that.
index 75437fc6e9e243a2a58f044c197e2958a5d3d20b..d7fdd51886555537f2f657843003b94c070d5c1c 100644 (file)
@@ -295,7 +295,7 @@ void stop_this_cpu(void)
 
        lapicid = lapic_read(LAPIC_ID) >> 24;
 
-       printk_debug("CPU %d going down...\n", lapicid);
+       printk_debug("CPU %ld going down...\n", lapicid);
 
        /* send an LAPIC INIT to myself */
        lapic_write_around(LAPIC_ICR2, SET_LAPIC_DEST_FIELD(lapicid));
index 92de0288a7005660550a2b7194b6a3962102094d..946d2cf512c23190c85a8cb348b078f718810477 100644 (file)
@@ -104,6 +104,7 @@ static unsigned long long calibrate_tsc(void)
        unsigned long long start, end, delta;
        unsigned long result, count;
        
+       printk_spew("Calibrating delay loop...\n");
        start = rdtscll();
        // no udivdi3, dammit.
        // so we count to 1<< 20 and then right shift 20
@@ -127,10 +128,10 @@ static unsigned long long calibrate_tsc(void)
        delta >>= 20;
        // save this for microsecond timing.
        result = delta;
-       printk_spew("end %x, start %x\n", end, start);
-       printk_spew("32-bit delta %d\n", (unsigned long) delta);
+       printk_spew("end %llx, start %llx\n", end, start);
+       printk_spew("32-bit delta %ld\n", (unsigned long) delta);
        
-       printk_spew("%s 32-bit result is %d\n",
+       printk_spew("%s 32-bit result is %ld\n",
                        __func__,
                        result);
        return delta;
@@ -143,7 +144,7 @@ void init_timer(void)
 {
        if (!clocks_per_usec) {
                clocks_per_usec = calibrate_tsc();
-               printk_info("clocks_per_usec: %u\n", clocks_per_usec);
+               printk_info("clocks_per_usec: %lu\n", clocks_per_usec);
        }
 }
 
index a66969d9b7beb902cb59ad25721aff4c045bfc16..06d32ea1f8bb61260a99f95613d816352a4d9898 100644 (file)
@@ -357,8 +357,7 @@ void compute_allocate_resource(
                        resource->flags &= ~IORESOURCE_STORED;
                        base += size;
                        
-                       printk_spew(
-                               "%s %02lx *  [0x%08Lx - 0x%08Lx] %s\n",
+                       printk_spew("%s %02lx *  [0x%08Lx - 0x%08Lx] %s\n",
                                dev_path(dev),
                                resource->index, 
                                resource->base,