aboutsummaryrefslogtreecommitdiffstats
path: root/utils/cec-compliance
diff options
context:
space:
mode:
authorHans Verkuil <hans.verkuil@cisco.com>2018-07-24 09:58:02 +0200
committerHans Verkuil <hans.verkuil@cisco.com>2018-07-24 09:58:02 +0200
commite5632f2521e69a1a3210fe7316c1655e1c0967e7 (patch)
tree1e3e61240532063fa02d8c84310997135c035b0c /utils/cec-compliance
parent3ad1c7c075c46b4ac069be6f75d59d722f135bcc (diff)
cec-compliance: OK & MAX_RETRIES: report that workaround was applied
Previous versions of cec-compliance just reported that OK and MAX_RETRIES were both set. But now this is worked around, so mention that as well. Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Diffstat (limited to 'utils/cec-compliance')
-rw-r--r--utils/cec-compliance/cec-compliance.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/utils/cec-compliance/cec-compliance.cpp b/utils/cec-compliance/cec-compliance.cpp
index f062f452..ad6c5744 100644
--- a/utils/cec-compliance/cec-compliance.cpp
+++ b/utils/cec-compliance/cec-compliance.cpp
@@ -704,7 +704,6 @@ int cec_named_ioctl(struct node *node, const char *name,
if (!retval && request == CEC_TRANSMIT &&
(msg->tx_status & CEC_TX_STATUS_OK) && ((msg->tx_status & CEC_TX_STATUS_MAX_RETRIES))) {
- warn("Both OK and MAX_RETRIES were set in tx_status!\n");
/*
* Workaround this bug in the CEC framework. This bug was solved
* in kernel 4.18 but older versions still can produce this incorrect
@@ -717,6 +716,7 @@ int cec_named_ioctl(struct node *node, const char *name,
msg->tx_error_cnt--;
msg->rx_status = CEC_RX_STATUS_TIMEOUT;
msg->rx_ts = msg->tx_ts;
+ warn("Both OK and MAX_RETRIES were set in tx_status! Applied workaround.\n");
}
if (!retval && request == CEC_TRANSMIT && show_info) {

Privacy Policy