diff options
author | Jean Delvare <khali@linux-fr.org> | 2008-04-29 17:11:37 -0400 |
---|---|---|
committer | Jean Delvare <khali@hyperion.delvare> | 2008-04-29 17:11:37 -0400 |
commit | 3578a0759ed2f0ea1f2409144e628dad4d748059 (patch) | |
tree | f47a03d4cd63eeb7c0be69932d537da6f4fe22ce | |
parent | c5d21b7fb747042cb2155698649cffccfd77d1f3 (diff) |
i2c-piix4: Minor cleanups
* Remove a needless include.
* Remove a legacy comment in piix4_access.
* Minor optimization in piix4_access.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
-rw-r--r-- | drivers/i2c/busses/i2c-piix4.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/drivers/i2c/busses/i2c-piix4.c b/drivers/i2c/busses/i2c-piix4.c index 4ae3a2cb6ace..fdc9ad805e35 100644 --- a/drivers/i2c/busses/i2c-piix4.c +++ b/drivers/i2c/busses/i2c-piix4.c | |||
@@ -38,7 +38,6 @@ | |||
38 | #include <linux/ioport.h> | 38 | #include <linux/ioport.h> |
39 | #include <linux/i2c.h> | 39 | #include <linux/i2c.h> |
40 | #include <linux/init.h> | 40 | #include <linux/init.h> |
41 | #include <linux/apm_bios.h> | ||
42 | #include <linux/dmi.h> | 41 | #include <linux/dmi.h> |
43 | #include <asm/io.h> | 42 | #include <asm/io.h> |
44 | 43 | ||
@@ -343,12 +342,7 @@ static s32 piix4_access(struct i2c_adapter * adap, u16 addr, | |||
343 | 342 | ||
344 | 343 | ||
345 | switch (size) { | 344 | switch (size) { |
346 | case PIIX4_BYTE: /* Where is the result put? I assume here it is in | 345 | case PIIX4_BYTE: |
347 | SMBHSTDAT0 but it might just as well be in the | ||
348 | SMBHSTCMD. No clue in the docs */ | ||
349 | |||
350 | data->byte = inb_p(SMBHSTDAT0); | ||
351 | break; | ||
352 | case PIIX4_BYTE_DATA: | 346 | case PIIX4_BYTE_DATA: |
353 | data->byte = inb_p(SMBHSTDAT0); | 347 | data->byte = inb_p(SMBHSTDAT0); |
354 | break; | 348 | break; |