diff options
author | Martin Roth <martin.roth@se-eng.com> | 2013-02-18 21:23:16 -0700 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2013-02-19 23:46:41 +0100 |
commit | 0fd0a054d46673e7375eb4808a3b66952fbd9ad3 (patch) | |
tree | 3e1c5d4689d0272406af9c7773d7b0d870fc1b58 /src/mainboard/tyan/s8226/mptable.c | |
parent | 510171e23bc6beaf502b0549e6ee446e47860715 (diff) | |
download | coreboot-0fd0a054d46673e7375eb4808a3b66952fbd9ad3.tar.xz |
Tyan S8226: Fix incompatible pointer warning
Fix warning:
mptable.c:52, GNU Compiler 4 (gcc), Priority: Normal
passing argument 3 of 'mptable_write_buses' from incompatible pointer type [enabled by default]
mptable_write_buses is expecting a pointer to an int, so I changed the
U8 isa_bus to an int to match. A U8 doesn't make sense if the value could
be greater than 255 - certainly unlikely, but possible since the value
of isa_bus gets set to the maximum PCI bus number + 1.
Change-Id: I7ea416f48285922d6cf341382109993fd3f6405c
Signed-off-by: Martin Roth <martin.roth@se-eng.com>
Reviewed-on: http://review.coreboot.org/2450
Tested-by: build bot (Jenkins)
Reviewed-by: Siyuan Wang <wangsiyuanbuaa@gmail.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Dave Frodin <dave.frodin@se-eng.com>
Reviewed-by: Marc Jones <marcj303@gmail.com>
Diffstat (limited to 'src/mainboard/tyan/s8226/mptable.c')
-rw-r--r-- | src/mainboard/tyan/s8226/mptable.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mainboard/tyan/s8226/mptable.c b/src/mainboard/tyan/s8226/mptable.c index 61a7bd4550..c5c1ab64ae 100644 --- a/src/mainboard/tyan/s8226/mptable.c +++ b/src/mainboard/tyan/s8226/mptable.c @@ -33,7 +33,7 @@ extern u8 bus_sp5100[2]; extern u32 bus_type[256]; extern u32 sbdn_sr5650; extern u32 sbdn_sp5100; -extern u8 bus_isa; +extern int bus_isa; static void *smp_write_config_table(void *v) |