diff options
author | Jean Delvare <khali@linux-fr.org> | 2010-10-24 12:16:59 -0400 |
---|---|---|
committer | Jean Delvare <khali@endymion.delvare> | 2010-10-24 12:16:59 -0400 |
commit | bf5d95c82692ead9ba7876af73dac2edcc8a6191 (patch) | |
tree | 546ff11c11534062477bb5ec73c27a98907e5e72 /drivers/i2c/busses/i2c-viapro.c | |
parent | db79f2a1d259a9026241dd0f192c20d6264d5afa (diff) |
i2c-viapro: Don't log nacks
Transactions not acked can happen every now and then, in particular
during device detection, and various transaction types can be used for
this purpose. So stop logging this event, except when debugging is
enabled. This is what other similar drivers (e.g. i2c-i801 or
i2c-piix4) do.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'drivers/i2c/busses/i2c-viapro.c')
-rw-r--r-- | drivers/i2c/busses/i2c-viapro.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/drivers/i2c/busses/i2c-viapro.c b/drivers/i2c/busses/i2c-viapro.c index 4c6fff5f330d..0b012f1f8ac5 100644 --- a/drivers/i2c/busses/i2c-viapro.c +++ b/drivers/i2c/busses/i2c-viapro.c | |||
@@ -185,14 +185,8 @@ static int vt596_transaction(u8 size) | |||
185 | } | 185 | } |
186 | 186 | ||
187 | if (temp & 0x04) { | 187 | if (temp & 0x04) { |
188 | int read = inb_p(SMBHSTADD) & 0x01; | ||
189 | result = -ENXIO; | 188 | result = -ENXIO; |
190 | /* The quick and receive byte commands are used to probe | 189 | dev_dbg(&vt596_adapter.dev, "No response\n"); |
191 | for chips, so errors are expected, and we don't want | ||
192 | to frighten the user. */ | ||
193 | if (!((size == VT596_QUICK && !read) || | ||
194 | (size == VT596_BYTE && read))) | ||
195 | dev_err(&vt596_adapter.dev, "Transaction error!\n"); | ||
196 | } | 190 | } |
197 | 191 | ||
198 | /* Resetting status register */ | 192 | /* Resetting status register */ |