Commit 3578a075 authored by Jean Delvare's avatar Jean Delvare Committed by Jean Delvare

i2c-piix4: Minor cleanups

* Remove a needless include.
* Remove a legacy comment in piix4_access.
* Minor optimization in piix4_access.
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
parent c5d21b7f
...@@ -38,7 +38,6 @@ ...@@ -38,7 +38,6 @@
#include <linux/ioport.h> #include <linux/ioport.h>
#include <linux/i2c.h> #include <linux/i2c.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/apm_bios.h>
#include <linux/dmi.h> #include <linux/dmi.h>
#include <asm/io.h> #include <asm/io.h>
...@@ -343,12 +342,7 @@ static s32 piix4_access(struct i2c_adapter * adap, u16 addr, ...@@ -343,12 +342,7 @@ static s32 piix4_access(struct i2c_adapter * adap, u16 addr,
switch (size) { switch (size) {
case PIIX4_BYTE: /* Where is the result put? I assume here it is in case PIIX4_BYTE:
SMBHSTDAT0 but it might just as well be in the
SMBHSTCMD. No clue in the docs */
data->byte = inb_p(SMBHSTDAT0);
break;
case PIIX4_BYTE_DATA: case PIIX4_BYTE_DATA:
data->byte = inb_p(SMBHSTDAT0); data->byte = inb_p(SMBHSTDAT0);
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment