From 3f111b0b11cf3d14b9aded601b6a1dd84f35a8dc Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Thu, 9 Mar 2017 12:02:52 +0100 Subject: southbridge/intel/i82801gx: Fix problems found by checkpatch.pl Change-Id: Iddc67e7c126ce19429afc24b021e385353564cb8 Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/18705 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/southbridge/intel/i82801gx/ac97.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/southbridge/intel/i82801gx/ac97.c') diff --git a/src/southbridge/intel/i82801gx/ac97.c b/src/southbridge/intel/i82801gx/ac97.c index ed800ff79d..b86891bddf 100644 --- a/src/southbridge/intel/i82801gx/ac97.c +++ b/src/southbridge/intel/i82801gx/ac97.c @@ -96,9 +96,8 @@ static int ac97_semaphore(void) reg8 = inb(nabmbar + CAS); timeout--; } while ((reg8 & 1) && timeout); - if (! timeout) { + if (!timeout) printk(BIOS_DEBUG, "Timeout!\n"); - } return (!timeout); } @@ -220,7 +219,7 @@ static void ac97_modem_init(struct device *dev) mbar = pci_read_config16(dev, MBAR) & 0xfffe; reg16 = inw(mmbar + EXT_MODEM_ID1); - if ((reg16 & 0xc000) != 0xc000 ) { + if ((reg16 & 0xc000) != 0xc000) { if (reg16 & (1 << 0)) { reg32 = inw(mmbar + VENDOR_ID2); reg32 <<= 16; @@ -247,7 +246,8 @@ static void ac97_modem_init(struct device *dev) } } -static void ac97_set_subsystem(device_t dev, unsigned vendor, unsigned device) +static void ac97_set_subsystem(device_t dev, unsigned int vendor, + unsigned int device) { if (!vendor || !device) { pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID, -- cgit v1.2.3