This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:

Subject: cec-compliance: OK & MAX_RETRIES: report that workaround was applied
Author:  Hans Verkuil <hans.verk...@cisco.com>
Date:    Tue Jul 24 09:58:02 2018 +0200

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.verk...@cisco.com>

 utils/cec-compliance/cec-compliance.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=e5632f2521e69a1a3210fe7316c1655e1c0967e7
diff --git a/utils/cec-compliance/cec-compliance.cpp 
b/utils/cec-compliance/cec-compliance.cpp
index f062f4526cd1..ad6c5744c597 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) {

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to