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: improve Inactive Source test
Author:  Hans Verkuil <hverkuil-ci...@xs4all.nl>
Date:    Wed Oct 16 09:27:06 2019 +0200

If Inactive Source is sent by a TV, then it should be ignored by
the remote device.

Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 utils/cec-compliance/cec-test.cpp | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=8d20035c12fe63d2c4b2dd9c0c5a71b6b13c3e4a
diff --git a/utils/cec-compliance/cec-test.cpp 
b/utils/cec-compliance/cec-test.cpp
index b6d8efee2048..0a2aec20cabb 100644
--- a/utils/cec-compliance/cec-test.cpp
+++ b/utils/cec-compliance/cec-test.cpp
@@ -454,13 +454,15 @@ static int routing_control_inactive_source(struct node 
*node, unsigned me, unsig
                                CEC_MSG_ACTIVE_SOURCE, CEC_MSG_SET_STREAM_PATH);
        mode = CEC_MODE_INITIATOR;
        doioctl(node, CEC_S_MODE, &mode);
-       fail_on_test(response < 0);
-       fail_on_test(interactive && !question("Did the TV switch away from or 
stop showing this source?"));
+       if (me == CEC_LOG_ADDR_TV) {
+               // Inactive Source should be ignored by all other devices
+               fail_on_test(response >= 0);
+       } else {
+               fail_on_test(response < 0);
+               fail_on_test(interactive && !question("Did the TV switch away 
from or stop showing this source?"));
+       }
 
-       if (interactive)
-               return 0;
-       else
-               return PRESUMED_OK;
+       return 0;
 }
 
 static int routing_control_active_source(struct node *node, unsigned me, 
unsigned la, bool interactive)

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

Reply via email to