Commit 78f420ac authored by Heiner Kallweit's avatar Heiner Kallweit Committed by Wolfram Sang
Browse files

i2c: i801: Remove unneeded warning after wait_event_timeout timeout 



When passing -ETIMEDOUT to i801_check_post() it will emit a timeout
error message. I don't see much benefit in an additional warning
stating more or less the same.

Signed-off-by: default avatarHeiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: default avatarJean Delvare <jdelvare@suse.de>
Tested-by: default avatarJean Delvare <jdelvare@suse.de>
Signed-off-by: default avatarWolfram Sang <wsa@kernel.org>
parent 3fb2e2ae

drivers/i2c/busses/i2c-i801.c

+4 −8
Original line number Diff line number Diff line
@@ -509,11 +509,9 @@ static int i801_transaction(struct i801_priv *priv, int xact) 
		result = wait_event_timeout(priv->waitq,

					    (status = priv->status),

					    adap->timeout);

		if (!result) {

		if (!result)

			status = -ETIMEDOUT;

			dev_warn(&priv->pci_dev->dev,

				 "Timeout waiting for interrupt!\n");

		}



		priv->status = 0;

		return i801_check_post(priv, status);

	}
@@ -732,11 +730,9 @@ static int i801_block_transaction_byte_by_byte(struct i801_priv *priv, 
		result = wait_event_timeout(priv->waitq,

					    (status = priv->status),

					    adap->timeout);

		if (!result) {

		if (!result)

			status = -ETIMEDOUT;

			dev_warn(&priv->pci_dev->dev,

				 "Timeout waiting for interrupt!\n");

		}



		priv->status = 0;

		return i801_check_post(priv, status);

	}