diff options
author | Jean Delvare <jdelvare@suse.de> | 2016-01-29 04:45:30 -0500 |
---|---|---|
committer | Wolfram Sang <wsa@the-dreams.de> | 2016-02-24 09:59:05 -0500 |
commit | 62194e869a56bf9d6fc10b6bdf8f11b1c4386249 (patch) | |
tree | 07f396ffaed20b71faf1206bcd89da16e7911f00 | |
parent | 6befa3fde65fe437f588da490c07a114393ce229 (diff) |
i2c: piix4: Always use the same type for port
Sometimes u8 is used to store the port number, sometimes unsigned
short is used. Consistently stick to a single type, for consistency
and to avoid implicit casts.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
-rw-r--r-- | drivers/i2c/busses/i2c-piix4.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/i2c/busses/i2c-piix4.c b/drivers/i2c/busses/i2c-piix4.c index 78d4483d2d5c..fe2dd07d80f6 100644 --- a/drivers/i2c/busses/i2c-piix4.c +++ b/drivers/i2c/busses/i2c-piix4.c | |||
@@ -159,7 +159,7 @@ struct i2c_piix4_adapdata { | |||
159 | 159 | ||
160 | /* SB800 */ | 160 | /* SB800 */ |
161 | bool sb800_main; | 161 | bool sb800_main; |
162 | unsigned short port; | 162 | u8 port; |
163 | }; | 163 | }; |
164 | 164 | ||
165 | static int piix4_setup(struct pci_dev *PIIX4_dev, | 165 | static int piix4_setup(struct pci_dev *PIIX4_dev, |
@@ -655,7 +655,7 @@ static struct i2c_adapter *piix4_main_adapters[PIIX4_MAX_ADAPTERS]; | |||
655 | static struct i2c_adapter *piix4_aux_adapter; | 655 | static struct i2c_adapter *piix4_aux_adapter; |
656 | 656 | ||
657 | static int piix4_add_adapter(struct pci_dev *dev, unsigned short smba, | 657 | static int piix4_add_adapter(struct pci_dev *dev, unsigned short smba, |
658 | bool sb800_main, unsigned short port, | 658 | bool sb800_main, u8 port, |
659 | const char *name, struct i2c_adapter **padap) | 659 | const char *name, struct i2c_adapter **padap) |
660 | { | 660 | { |
661 | struct i2c_adapter *adap; | 661 | struct i2c_adapter *adap; |