From 68f542cdf82efe257ee4251047a264558dd8645f Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Wed, 17 Mar 2010 02:48:24 +0000 Subject: remove more warnings, and fix some boards (watchdog.h) Signed-off-by: Stefan Reinauer Acked-by: Stefan Reinauer git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5239 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/southbridge/intel/i82801ex/i82801ex_lpc.c | 2 +- src/southbridge/intel/i82801ex/i82801ex_pci.c | 2 +- src/southbridge/intel/i82801ex/i82801ex_sata.c | 5 +---- src/southbridge/intel/i82801ex/i82801ex_smbus.c | 12 ++++++++---- src/southbridge/intel/i82801ex/i82801ex_smbus.h | 2 +- src/southbridge/intel/pxhd/pxhd_bridge.c | 10 +++++----- 6 files changed, 17 insertions(+), 16 deletions(-) (limited to 'src/southbridge/intel') diff --git a/src/southbridge/intel/i82801ex/i82801ex_lpc.c b/src/southbridge/intel/i82801ex/i82801ex_lpc.c index 6cf8124c50..39e5343fd9 100644 --- a/src/southbridge/intel/i82801ex/i82801ex_lpc.c +++ b/src/southbridge/intel/i82801ex/i82801ex_lpc.c @@ -232,7 +232,7 @@ const unsigned long hpet_address = 0xfed0000; dword &= ~(3 << 15); /* clear it */ dword |= (code<<15); - printk_debug("enabling HPET @0x%x\n", hpet_address | (code <<12) ); + printk_debug("enabling HPET @0x%lx\n", hpet_address | (code <<12) ); } static void lpc_init(struct device *dev) diff --git a/src/southbridge/intel/i82801ex/i82801ex_pci.c b/src/southbridge/intel/i82801ex/i82801ex_pci.c index 650628b5e5..2394844ba4 100644 --- a/src/southbridge/intel/i82801ex/i82801ex_pci.c +++ b/src/southbridge/intel/i82801ex/i82801ex_pci.c @@ -7,7 +7,6 @@ static void pci_init(struct device *dev) { - uint32_t dword; uint16_t word; /* Clear system errors */ @@ -17,6 +16,7 @@ static void pci_init(struct device *dev) #if 0 /* System error enable */ + uint32_t dword; dword = pci_read_config32(dev, 0x04); dword |= (1<<8); /* SERR# Enable */ dword |= (1<<6); /* Parity Error Response */ diff --git a/src/southbridge/intel/i82801ex/i82801ex_sata.c b/src/southbridge/intel/i82801ex/i82801ex_sata.c index 98431edc78..73f5773fd5 100644 --- a/src/southbridge/intel/i82801ex/i82801ex_sata.c +++ b/src/southbridge/intel/i82801ex/i82801ex_sata.c @@ -7,10 +7,7 @@ static void sata_init(struct device *dev) { - - uint16_t word; - - printk_debug("SATA init\n"); + printk_debug("SATA init\n"); /* SATA configuration */ pci_write_config8(dev, 0x04, 0x07); pci_write_config8(dev, 0x09, 0x8f); diff --git a/src/southbridge/intel/i82801ex/i82801ex_smbus.c b/src/southbridge/intel/i82801ex/i82801ex_smbus.c index adfbcb7cc9..377df11cd0 100644 --- a/src/southbridge/intel/i82801ex/i82801ex_smbus.c +++ b/src/southbridge/intel/i82801ex/i82801ex_smbus.c @@ -8,24 +8,28 @@ #include "i82801ex.h" #include "i82801ex_smbus.h" -static int lsmbus_read_byte(struct bus *bus, device_t dev, uint8_t address) +static int lsmbus_read_byte(device_t dev, u8 address) { - unsigned device; + u16 device; struct resource *res; + struct bus *pbus; device = dev->path.i2c.device; - res = find_resource(bus->dev, 0x20); - + pbus = get_pbus_smbus(dev); + res = find_resource(pbus->dev, 0x20); + return do_smbus_read_byte(res->base, device, address); } static struct smbus_bus_operations lops_smbus_bus = { .read_byte = lsmbus_read_byte, }; + static struct pci_operations lops_pci = { /* The subsystem id follows the ide controller */ .set_subsystem = 0, }; + static struct device_operations smbus_ops = { .read_resources = pci_dev_read_resources, .set_resources = pci_dev_set_resources, diff --git a/src/southbridge/intel/i82801ex/i82801ex_smbus.h b/src/southbridge/intel/i82801ex/i82801ex_smbus.h index 861230e130..27acca494f 100644 --- a/src/southbridge/intel/i82801ex/i82801ex_smbus.h +++ b/src/southbridge/intel/i82801ex/i82801ex_smbus.h @@ -46,7 +46,7 @@ static int smbus_wait_until_done(unsigned smbus_io_base) return loops?0:-1; } -static int smbus_wait_until_blk_done(unsigned smbus_io_base) +static inline int smbus_wait_until_blk_done(unsigned smbus_io_base) { unsigned loops = SMBUS_TIMEOUT; unsigned char byte; diff --git a/src/southbridge/intel/pxhd/pxhd_bridge.c b/src/southbridge/intel/pxhd/pxhd_bridge.c index 1a21a9c03e..0766b22e7b 100644 --- a/src/southbridge/intel/pxhd/pxhd_bridge.c +++ b/src/southbridge/intel/pxhd/pxhd_bridge.c @@ -64,15 +64,15 @@ static unsigned int pxhd_scan_bridge(device_t dev, unsigned int max) } static void pcix_init(device_t dev) { - uint32_t dword; - uint16_t word; - uint8_t byte; - int nmi_option; - /* Bridge control ISA enable */ pci_write_config8(dev, 0x3e, 0x07); +#warning "Please review lots of dead code here." #if 0 + int nmi_option; + uint32_t dword; + uint16_t word; + uint8_t byte; /* Enable memory write and invalidate ??? */ byte = pci_read_config8(dev, 0x04); -- cgit v1.2.3