This is an automatic generated email to let you know that the following patch 
were queued at the tree:

Subject: cec-compliance: --test-adapter needs a valid PA
Author:  Hans Verkuil <>
Date:    Thu Dec 1 15:16:11 2016 +0100

If the --test-adapter test is run, and there is no valid physical address, then
bail out with a clear message instead of producing a bunch of weird failures.

Signed-off-by: Hans Verkuil <>

 utils/cec-compliance/cec-compliance.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/utils/cec-compliance/cec-compliance.cpp 
index bd96d968e21b..547247a07825 100644
--- a/utils/cec-compliance/cec-compliance.cpp
+++ b/utils/cec-compliance/cec-compliance.cpp
@@ -1307,6 +1307,11 @@ int main(int argc, char **argv)
                printf("\tCEC_ADAP_G/S_PHYS_ADDR: %s\n", 
                if (node.caps & CEC_CAP_PHYS_ADDR)
                        doioctl(&node, CEC_ADAP_S_PHYS_ADDR, &node.phys_addr);
+               if (node.phys_addr == CEC_PHYS_ADDR_INVALID) {
+                       fprintf(stderr, "FAIL: without a valid physical address 
this test cannot proceed.\n");
+                       fprintf(stderr, "Make sure that this CEC adapter is 
connected to another HDMI sink or source.\n");
+                       exit(1);
+               }
                printf("\tCEC_ADAP_G/S_LOG_ADDRS: %s\n", 
                fcntl(node.fd, F_SETFL, fcntl(node.fd, F_GETFL) & ~O_NONBLOCK);
                if (node.caps & CEC_CAP_LOG_ADDRS) {

linuxtv-commits mailing list

Reply via email to