Author: stepan Date: 2007-10-24 00:17:45 +0200 (Wed, 24 Oct 2007) New Revision: 2890
Modified: trunk/LinuxBIOSv2/src/arch/i386/boot/linuxbios_table.c trunk/LinuxBIOSv2/src/arch/i386/boot/tables.c trunk/LinuxBIOSv2/src/boot/elfboot.c trunk/LinuxBIOSv2/src/cpu/x86/mtrr/mtrr.c trunk/LinuxBIOSv2/src/cpu/x86/tsc/delay_tsc.c trunk/LinuxBIOSv2/src/devices/device_util.c trunk/LinuxBIOSv2/src/devices/hypertransport.c Log: fix a whole bunch of warnings. (trivial)
Signed-off-by: Stefan Reinauer stepan@coresystems.de Acked-by: Stefan Reinauer stepan@coresystems.de
Modified: trunk/LinuxBIOSv2/src/arch/i386/boot/linuxbios_table.c =================================================================== --- trunk/LinuxBIOSv2/src/arch/i386/boot/linuxbios_table.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/arch/i386/boot/linuxbios_table.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -120,7 +120,7 @@ { static const struct { uint32_t tag; - const uint8_t *string; + const char *string; } strings[] = { { LB_TAG_VERSION, linuxbios_version, }, { LB_TAG_EXTRA_VERSION, linuxbios_extra_version, },
Modified: trunk/LinuxBIOSv2/src/arch/i386/boot/tables.c =================================================================== --- trunk/LinuxBIOSv2/src/arch/i386/boot/tables.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/arch/i386/boot/tables.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -8,6 +8,7 @@ #include <arch/pirq_routing.h> #include <arch/smp/mpspec.h> #include <arch/acpi.h> +#include <string.h> #include "linuxbios_table.h"
// Global Descriptor Table, defined in c_start.S
Modified: trunk/LinuxBIOSv2/src/boot/elfboot.c =================================================================== --- trunk/LinuxBIOSv2/src/boot/elfboot.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/boot/elfboot.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -144,7 +144,7 @@ { struct verify_callback *cb_chain; unsigned char *note, *end; - char *program, *version; + unsigned char *program, *version;
cb_chain = 0; note = header + offset;
Modified: trunk/LinuxBIOSv2/src/cpu/x86/mtrr/mtrr.c =================================================================== --- trunk/LinuxBIOSv2/src/cpu/x86/mtrr/mtrr.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/cpu/x86/mtrr/mtrr.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -349,7 +349,6 @@ * mtrrs. If this doesn't work out we can get smart again * and clear out the mtrrs. */ - struct var_mtrr_state var_state;
printk_debug("\n"); /* Initialized the fixed_mtrrs to uncached */
Modified: trunk/LinuxBIOSv2/src/cpu/x86/tsc/delay_tsc.c =================================================================== --- trunk/LinuxBIOSv2/src/cpu/x86/tsc/delay_tsc.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/cpu/x86/tsc/delay_tsc.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -102,8 +102,7 @@ static unsigned long long calibrate_tsc(void) { unsigned long long start, end, delta; - unsigned long allones = (unsigned long) -1, result; - unsigned long count; + unsigned long result, count; start = rdtscll(); // no udivdi3, dammit.
Modified: trunk/LinuxBIOSv2/src/devices/device_util.c =================================================================== --- trunk/LinuxBIOSv2/src/devices/device_util.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/devices/device_util.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -454,7 +454,7 @@ void report_resource_stored(device_t dev, struct resource *resource, const char *comment) { if (resource->flags & IORESOURCE_STORED) { - unsigned char buf[10]; + char buf[10]; unsigned long long base, end; base = resource->base; end = resource_end(resource);
Modified: trunk/LinuxBIOSv2/src/devices/hypertransport.c =================================================================== --- trunk/LinuxBIOSv2/src/devices/hypertransport.c 2007-10-23 19:23:52 UTC (rev 2889) +++ trunk/LinuxBIOSv2/src/devices/hypertransport.c 2007-10-23 22:17:45 UTC (rev 2890) @@ -37,6 +37,9 @@ #include <part/hard_reset.h> #include <part/fallback_boot.h>
+/* The hypertransport link is already optimized in pre-ram code + * so don't do it again + */ #define OPT_HT_LINK 0
#if OPT_HT_LINK == 1 @@ -123,15 +126,17 @@
static int ht_setup_link(struct ht_link *prev, device_t dev, unsigned pos) { +#if OPT_HT_LINK == 1 static const uint8_t link_width_to_pow2[]= { 3, 4, 0, 5, 1, 2, 0, 0 }; static const uint8_t pow2_to_link_width[] = { 0x7, 4, 5, 0, 1, 3 }; - struct ht_link cur[1]; unsigned present_width_cap, upstream_width_cap; unsigned present_freq_cap, upstream_freq_cap; unsigned ln_present_width_in, ln_upstream_width_in; unsigned ln_present_width_out, ln_upstream_width_out; unsigned freq, old_freq; unsigned present_width, upstream_width, old_width; +#endif + struct ht_link cur[1]; int reset_needed; int linkb_to_host;
@@ -485,7 +490,7 @@ next_unitid = HT_CHAIN_END_UNITID_BASE; end_used = 1; } else { - goto out; + goto end_of_chain; } }
@@ -535,7 +540,6 @@ (dev->enabled? "enabled": "disabled"), next_unitid);
} while (last_unitid != next_unitid); - out: end_of_chain: #if OPT_HT_LINK == 1 if(bus->reset_needed) {