[PATCH] i2c: nvidia-gpu: Log transfer timeouts at debug level

[PATCH] i2c: nvidia-gpu: Log transfer timeouts at debug level

[Hex Lindqvist]

GPUs without a USB Type-C port, such as the GTX 1660 Ti, have no CCGx
controller behind the ccgx-ucsi client, so every transfer to it times
out and logs:

  nvidia-gpu 0000:01:00.3: i2c timeout error e0000000

The caller already gets -ETIMEDOUT and decides whether it matters. Log
the timeout at debug level.

Tested on an ASUS TUF Gaming A17 (FA706IU, GTX 1660 Ti): the error is
gone, and the timeout still shows up with dynamic debug.

Assisted by LLM.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=206653
Link: https://lore.kernel.org/all/20200921180840.GA18032@ninjato/
Signed-off-by: Hex Lindqvist <hex@freenixos.com>
---
The ucsi_ccg errors that follow are fixed by a separate patch to
linux-usb. Wolfram suggested dropping this message in 2020; debug level
keeps the status value.

 drivers/i2c/busses/i2c-nvidia-gpu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/i2c-nvidia-gpu.c b/drivers/i2c/busses/i2c-nvidia-gpu.c
index 14c059b03945..987dd3edb0bb 100644
--- a/drivers/i2c/busses/i2c-nvidia-gpu.c
+++ b/drivers/i2c/busses/i2c-nvidia-gpu.c
@@ -88,7 +88,7 @@ static int gpu_i2c_check_status(struct gpu_i2c_dev *i2cd)
 				 500, 1000 * USEC_PER_MSEC);
 
 	if (ret) {
-		dev_err(i2cd->dev, "i2c timeout error %x\n", val);
+		dev_dbg(i2cd->dev, "i2c timeout, status %#x\n", val);
 		return -ETIMEDOUT;
 	}
 

base-commit: 6628e7d4c6b25c8d18f1345e3a20d21829aae7e4
-- 
2.56.0