diff options
author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2022-05-13 08:54:43 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@kernel.org> | 2022-06-20 10:30:30 +0100 |
commit | 691c3db0dc7616b3cc4ff0f52f956c9afa71b1cd (patch) | |
tree | 1eefda93d12d0ab23599e9155ed14d26ac59212f /drivers/media/cec | |
parent | b9629c550f01f9864fa644b2ba6dae9c9a2be98d (diff) | |
download | linux-691c3db0dc7616b3cc4ff0f52f956c9afa71b1cd.tar.gz |
media: cec-adap.c: log when claiming LA fails unexpectedly
If there is a hardware problem such as someone pulling the CEC line low
continuously, then the POLL message will fail with an error other than
OK, NACK, ABORTED or TIMEOUT. Log the tx_status value in that case to
help debug this.
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers/media/cec')
-rw-r--r-- | drivers/media/cec/core/cec-adap.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/media/cec/core/cec-adap.c b/drivers/media/cec/core/cec-adap.c index 8bf91b5a7d0e..41a79293ee02 100644 --- a/drivers/media/cec/core/cec-adap.c +++ b/drivers/media/cec/core/cec-adap.c @@ -1309,8 +1309,11 @@ static int cec_config_log_addr(struct cec_adapter *adap, * we assume that something is really weird and that it is not a * good idea to try and claim this logical address. */ - if (i == max_retries) + if (i == max_retries) { + dprintk(0, "polling for LA %u failed with tx_status=0x%04x\n", + log_addr, msg.tx_status); return 0; + } /* * Message not acknowledged, so this logical |