diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2017-03-09 12:02:52 +0100 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-03-22 17:55:53 +0100 |
| commit | 3f111b0b11cf3d14b9aded601b6a1dd84f35a8dc (patch) | |
| tree | a11b02549c808bd7a59ee762a73bd46561731ea3 /src/southbridge/intel/i82801gx/i82801gx.h | |
| parent | 70a8e34853d4b01ab7a2089821c35715c59b4415 (diff) | |
southbridge/intel/i82801gx: Fix problems found by checkpatch.pl
Change-Id: Iddc67e7c126ce19429afc24b021e385353564cb8
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/18705
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/southbridge/intel/i82801gx/i82801gx.h')
| -rw-r--r-- | src/southbridge/intel/i82801gx/i82801gx.h | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/southbridge/intel/i82801gx/i82801gx.h b/src/southbridge/intel/i82801gx/i82801gx.h index d1441e85b1..85cf6dbd53 100644 --- a/src/southbridge/intel/i82801gx/i82801gx.h +++ b/src/southbridge/intel/i82801gx/i82801gx.h @@ -49,7 +49,7 @@ void i82801gx_enable(device_t dev); void gpi_route_interrupt(u8 gpi, u8 mode); #else void enable_smbus(void); -int smbus_read_byte(unsigned device, unsigned address); +int smbus_read_byte(unsigned int device, unsigned int address); int southbridge_detect_s3_resume(void); #endif #endif @@ -200,9 +200,9 @@ int southbridge_detect_s3_resume(void); /* Root Complex Register Block */ #define RCBA 0xf0 -#define RCBA8(x) *((volatile u8 *)(DEFAULT_RCBA + x)) -#define RCBA16(x) *((volatile u16 *)(DEFAULT_RCBA + x)) -#define RCBA32(x) *((volatile u32 *)(DEFAULT_RCBA + x)) +#define RCBA8(x) (*((volatile u8 *)(DEFAULT_RCBA + (x)))) +#define RCBA16(x) (*((volatile u16 *)(DEFAULT_RCBA + (x)))) +#define RCBA32(x) (*((volatile u32 *)(DEFAULT_RCBA + (x)))) #define VCH 0x0000 /* 32bit */ #define VCAP1 0x0004 /* 32bit */ @@ -295,9 +295,9 @@ int southbridge_detect_s3_resume(void); * If UHCI controllers get disabled, EHCI * must know about it, too! */ #define FD_UHCI4 (1 << 11) -#define FD_UHCI34 (1 << 10) | FD_UHCI4 -#define FD_UHCI234 (1 << 9) | FD_UHCI3 -#define FD_UHCI1234 (1 << 8) | FD_UHCI2 +#define FD_UHCI34 ((1 << 10) | FD_UHCI4) +#define FD_UHCI234 ((1 << 9) | FD_UHCI3) +#define FD_UHCI1234 ((1 << 8) | FD_UHCI2) #define FD_INTLAN (1 << 7) #define FD_ACMOD (1 << 6) |