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: get_power_status(): fix valid RX test
Author:  Hans Verkuil <[email protected]>
Date:    Thu Jun 22 11:39:59 2023 +0200

Explicitly test for a valid reply to the <Give Device Power Status>
message. The current test just checks that it isn't a FEATURE ABORT
or a time out, but that misses the ABORT status where the HPD goes
low while waiting for a reply.

Just check that the rx_status is OK instead.

Signed-off-by: Hans Verkuil <[email protected]>

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

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=357f28d55419dd8658e12079581a4a021a962d30
diff --git a/utils/cec-compliance/cec-test-power.cpp 
b/utils/cec-compliance/cec-test-power.cpp
index ed1e1fbc26b4..1e485f42a4f4 100644
--- a/utils/cec-compliance/cec-test-power.cpp
+++ b/utils/cec-compliance/cec-test-power.cpp
@@ -26,7 +26,7 @@ static bool get_power_status(struct node *node, unsigned me, 
unsigned la, __u8 &
                power_status = CEC_OP_POWER_STATUS_STANDBY;
                return true;
        }
-       if (res || !(msg.tx_status & CEC_TX_STATUS_OK) || 
timed_out_or_abort(&msg))
+       if (res || !(msg.tx_status & CEC_TX_STATUS_OK) || !(msg.rx_status & 
CEC_RX_STATUS_OK))
                return false;
        cec_ops_report_power_status(&msg, &power_status);
        return true;

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to