diff options
author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-06 18:01:04 -0800 |
---|---|---|
committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-09 17:16:09 +0100 |
commit | 0ca2a0654ca4b403e8a54d558bce07a862820a9d (patch) | |
tree | 86c845b441c8ae4824bf6178943148ec94441c6d /src/include/device/i2c.h | |
parent | 75d8d8da47a3cc759d7395f5b0ef91ba13a59e51 (diff) |
src/include: Fix unsigned warnings
Fix warning detected by checkpatch.pl:
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
BRANCH=none
BUG=None
TEST=Build and run on Galileo Gen2
Change-Id: I23d9b4b715aa74acc387db8fb8d3c73bd5cabfaa
Signed-off-by: Lee Leahy <Leroy.P.Leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18607
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/include/device/i2c.h')
-rw-r--r-- | src/include/device/i2c.h | 36 |
1 files changed, 19 insertions, 17 deletions
diff --git a/src/include/device/i2c.h b/src/include/device/i2c.h index d8a793c716..6316748b85 100644 --- a/src/include/device/i2c.h +++ b/src/include/device/i2c.h @@ -40,27 +40,29 @@ struct i2c_seg int len; }; -int platform_i2c_transfer(unsigned bus, struct i2c_seg *segments, int count); +int platform_i2c_transfer(unsigned int bus, struct i2c_seg *segments, + int count); #define SOFTWARE_I2C_MAX_BUS 10 /* increase as necessary */ struct software_i2c_ops { - void (*set_sda)(unsigned bus, int high); - void (*set_scl)(unsigned bus, int high); - int (*get_sda)(unsigned bus); - int (*get_scl)(unsigned bus); + void (*set_sda)(unsigned int bus, int high); + void (*set_scl)(unsigned int bus, int high); + int (*get_sda)(unsigned int bus); + int (*get_scl)(unsigned int bus); }; extern struct software_i2c_ops *software_i2c[]; -int software_i2c_transfer(unsigned bus, struct i2c_seg *segments, int count); -void software_i2c_wedge_ack(unsigned bus, u8 chip); -void software_i2c_wedge_read(unsigned bus, u8 chip, u8 reg, int bit_count); -void software_i2c_wedge_write(unsigned bus, u8 chip, u8 reg, int bit_count); +int software_i2c_transfer(unsigned int bus, struct i2c_seg *segments, + int count); +void software_i2c_wedge_ack(unsigned int bus, u8 chip); +void software_i2c_wedge_read(unsigned int bus, u8 chip, u8 reg, int bit_count); +void software_i2c_wedge_write(unsigned int bus, u8 chip, u8 reg, int bit_count); -int i2c_read_field(unsigned bus, uint8_t chip, uint8_t reg, uint8_t *data, +int i2c_read_field(unsigned int bus, uint8_t chip, uint8_t reg, uint8_t *data, uint8_t mask, uint8_t shift); -int i2c_write_field(unsigned bus, uint8_t chip, uint8_t reg, uint8_t data, +int i2c_write_field(unsigned int bus, uint8_t chip, uint8_t reg, uint8_t data, uint8_t mask, uint8_t shift); /* @@ -69,7 +71,7 @@ int i2c_write_field(unsigned bus, uint8_t chip, uint8_t reg, uint8_t data, * Need this ugly stub to arbitrate since I2C device drivers hardcode * 'i2c_transfer()' as their entry point. */ -static inline int i2c_transfer(unsigned bus, struct i2c_seg *segments, +static inline int i2c_transfer(unsigned int bus, struct i2c_seg *segments, int count) { if (CONFIG_SOFTWARE_I2C) @@ -84,7 +86,7 @@ static inline int i2c_transfer(unsigned bus, struct i2c_seg *segments, * * [start][slave addr][r][data][stop] */ -static inline int i2c_read_raw(unsigned bus, uint8_t chip, uint8_t *data, +static inline int i2c_read_raw(unsigned int bus, uint8_t chip, uint8_t *data, int len) { struct i2c_seg seg = @@ -97,7 +99,7 @@ static inline int i2c_read_raw(unsigned bus, uint8_t chip, uint8_t *data, * * [start][slave addr][w][data][stop] */ -static inline int i2c_write_raw(unsigned bus, uint8_t chip, uint8_t *data, +static inline int i2c_write_raw(unsigned int bus, uint8_t chip, uint8_t *data, int len) { struct i2c_seg seg = @@ -110,7 +112,7 @@ static inline int i2c_write_raw(unsigned bus, uint8_t chip, uint8_t *data, * * [start][slave addr][w][register addr][start][slave addr][r][data...][stop] */ -static inline int i2c_read_bytes(unsigned bus, uint8_t chip, uint8_t reg, +static inline int i2c_read_bytes(unsigned int bus, uint8_t chip, uint8_t reg, uint8_t *data, int len) { struct i2c_seg seg[2]; @@ -132,7 +134,7 @@ static inline int i2c_read_bytes(unsigned bus, uint8_t chip, uint8_t reg, * * [start][slave addr][w][register addr][start][slave addr][r][data][stop] */ -static inline int i2c_readb(unsigned bus, uint8_t chip, uint8_t reg, +static inline int i2c_readb(unsigned int bus, uint8_t chip, uint8_t reg, uint8_t *data) { struct i2c_seg seg[2]; @@ -154,7 +156,7 @@ static inline int i2c_readb(unsigned bus, uint8_t chip, uint8_t reg, * * [start][slave addr][w][register addr][data][stop] */ -static inline int i2c_writeb(unsigned bus, uint8_t chip, uint8_t reg, +static inline int i2c_writeb(unsigned int bus, uint8_t chip, uint8_t reg, uint8_t data) { struct i2c_seg seg; |