GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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>
This commit is contained in:
Martin Roth 2013-02-18 21:23:16 -07:00 committed by Stefan Reinauer
parent 510171e23b
commit 0fd0a054d4
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/mainboard/tyan/s8226/get_bus_conf.c
View File

@ -32,7 +32,7 @@
/* Global variables for MB layouts and these will be shared by irqtable mptable /* Global variables for MB layouts and these will be shared by irqtable mptable
* and acpi_tables busnum is default. * and acpi_tables busnum is default.
*/ */
u8 bus_isa; int bus_isa;
u8 bus_sp5100[2]; u8 bus_sp5100[2];
u8 bus_sr5650[14]; u8 bus_sr5650[14];
@ -101,7 +101,7 @@ void get_bus_conf(void)
if (dev) { if (dev) {
bus_sp5100[1] = pci_read_config8(dev, PCI_SECONDARY_BUS); bus_sp5100[1] = pci_read_config8(dev, PCI_SECONDARY_BUS);
bus_isa = pci_read_config8(dev, PCI_SUBORDINATE_BUS); bus_isa = (int)pci_read_config8(dev, PCI_SUBORDINATE_BUS);
bus_isa++; bus_isa++;
for (j = bus_sp5100[1]; j < bus_isa; j++) for (j = bus_sp5100[1]; j < bus_isa; j++)
bus_type[j] = 1; bus_type[j] = 1;
@ -113,7 +113,7 @@ void get_bus_conf(void)
if (dev) { if (dev) {
bus_sr5650[i] = pci_read_config8(dev, PCI_SECONDARY_BUS); bus_sr5650[i] = pci_read_config8(dev, PCI_SECONDARY_BUS);
if(255 != bus_sr5650[i]) { if(255 != bus_sr5650[i]) {
bus_isa = pci_read_config8(dev, PCI_SUBORDINATE_BUS); bus_isa = (int)pci_read_config8(dev, PCI_SUBORDINATE_BUS);
bus_isa++; bus_isa++;
bus_type[bus_sr5650[i]] = 1; /* PCI bus. */ bus_type[bus_sr5650[i]] = 1; /* PCI bus. */
} }

2
src/mainboard/tyan/s8226/mptable.c
View File

@ -33,7 +33,7 @@ extern u8 bus_sp5100[2];
extern u32 bus_type[256]; extern u32 bus_type[256];
extern u32 sbdn_sr5650; extern u32 sbdn_sr5650;
extern u32 sbdn_sp5100; extern u32 sbdn_sp5100;
extern u8 bus_isa; extern int bus_isa;
static void *smp_write_config_table(void *v) static void *smp_write_config_table(void *v)