summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Reinauer <stepan@coresystems.de>2007-10-23 22:17:45 +0000
committerStefan Reinauer <stepan@openbios.org>2007-10-23 22:17:45 +0000
commit0dff6e3fa95ea13f6ee6cb2d4277e83076d81bad (patch)
treeaaf9a4fcab39c9dba0c4dc7457a79fc0d327cf4c
parent643eee07541f10fcc3b34a56dc8bf2cf8d93c649 (diff)
downloadcoreboot-0dff6e3fa95ea13f6ee6cb2d4277e83076d81bad.tar.xz
fix a whole bunch of warnings. (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@2890 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
-rw-r--r--src/arch/i386/boot/linuxbios_table.c2
-rw-r--r--src/arch/i386/boot/tables.c1
-rw-r--r--src/boot/elfboot.c2
-rw-r--r--src/cpu/x86/mtrr/mtrr.c1
-rw-r--r--src/cpu/x86/tsc/delay_tsc.c3
-rw-r--r--src/devices/device_util.c2
-rw-r--r--src/devices/hypertransport.c10
7 files changed, 12 insertions, 9 deletions
diff --git a/src/arch/i386/boot/linuxbios_table.c b/src/arch/i386/boot/linuxbios_table.c
index 9074d11685..e32d2467a9 100644
--- a/src/arch/i386/boot/linuxbios_table.c
+++ b/src/arch/i386/boot/linuxbios_table.c
@@ -120,7 +120,7 @@ void lb_strings(struct lb_header *header)
{
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, },
diff --git a/src/arch/i386/boot/tables.c b/src/arch/i386/boot/tables.c
index 79ddd07ee0..29fcc13da4 100644
--- a/src/arch/i386/boot/tables.c
+++ b/src/arch/i386/boot/tables.c
@@ -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
diff --git a/src/boot/elfboot.c b/src/boot/elfboot.c
index 9050d7ba01..6f307edd73 100644
--- a/src/boot/elfboot.c
+++ b/src/boot/elfboot.c
@@ -144,7 +144,7 @@ static struct verify_callback *process_elf_notes(
{
struct verify_callback *cb_chain;
unsigned char *note, *end;
- char *program, *version;
+ unsigned char *program, *version;
cb_chain = 0;
note = header + offset;
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c
index ab28a23a02..1a9e9d4b54 100644
--- a/src/cpu/x86/mtrr/mtrr.c
+++ b/src/cpu/x86/mtrr/mtrr.c
@@ -349,7 +349,6 @@ void x86_setup_fixed_mtrrs(void)
* 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 */
diff --git a/src/cpu/x86/tsc/delay_tsc.c b/src/cpu/x86/tsc/delay_tsc.c
index eaf85d2c76..a791f7930e 100644
--- a/src/cpu/x86/tsc/delay_tsc.c
+++ b/src/cpu/x86/tsc/delay_tsc.c
@@ -102,8 +102,7 @@ bad_ctc:
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.
diff --git a/src/devices/device_util.c b/src/devices/device_util.c
index efb1f0f257..c6ef541282 100644
--- a/src/devices/device_util.c
+++ b/src/devices/device_util.c
@@ -454,7 +454,7 @@ const char *resource_type(struct resource *resource)
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);
diff --git a/src/devices/hypertransport.c b/src/devices/hypertransport.c
index 9846794edd..01096d054e 100644
--- a/src/devices/hypertransport.c
+++ b/src/devices/hypertransport.c
@@ -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 @@ struct ht_link {
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 @@ unsigned int hypertransport_scan_chain(struct bus *bus,
next_unitid = HT_CHAIN_END_UNITID_BASE;
end_used = 1;
} else {
- goto out;
+ goto end_of_chain;
}
}
@@ -535,7 +540,6 @@ unsigned int hypertransport_scan_chain(struct bus *bus,
(dev->enabled? "enabled": "disabled"), next_unitid);
} while (last_unitid != next_unitid);
- out:
end_of_chain:
#if OPT_HT_LINK == 1
if(bus->reset_needed) {