aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHans Verkuil <hans.verkuil@cisco.com>2016-07-22 10:38:48 +0200
committerHans Verkuil <hans.verkuil@cisco.com>2016-07-22 10:38:48 +0200
commit8f6e3ca355f6329115cc0c959ab703d0961d9301 (patch)
tree96f684b7989b64ce2ee8897af2bca1ca98d4c28d
parent2f488c5b159d3f6da225e359075c12f3dbf6ab56 (diff)
cec-compliance: improve device features test
The last byte of the features was 0 in the original test. Because of that decision I missed a cec framework bug. Make sure the last byte isn't 0. Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
-rw-r--r--utils/cec-compliance/cec-compliance.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/utils/cec-compliance/cec-compliance.cpp b/utils/cec-compliance/cec-compliance.cpp
index beede374..cc2893ba 100644
--- a/utils/cec-compliance/cec-compliance.cpp
+++ b/utils/cec-compliance/cec-compliance.cpp
@@ -728,7 +728,7 @@ static int testAdapLogAddrs(struct node *node)
static const __u8 all_dev_types =
CEC_OP_ALL_DEVTYPE_TV | CEC_OP_ALL_DEVTYPE_RECORD;
static const __u8 features[12] = {
- 0x90, 0x00, 0x8e, 0x00,
+ 0x80, 0x00, 0x10, 0xff,
0xff, 0xff, 0xff, 0xff,
0xff, 0xff, 0xff, 0xff
};
@@ -785,8 +785,8 @@ static int testAdapLogAddrs(struct node *node)
fail_on_test(laddrs.log_addr_mask == 0);
for (unsigned i = 0; i < laddrs.num_log_addrs; i++) {
fail_on_test(laddrs.log_addr[i] == CEC_LOG_ADDR_INVALID);
- fail_on_test(memcmp(laddrs.features[i], features, 4));
- fail_on_test(check_0(laddrs.features[i] + 4, 8));
+ fail_on_test(memcmp(laddrs.features[i], features, 3));
+ fail_on_test(check_0(laddrs.features[i] + 3, 9));
}
for (unsigned i = laddrs.num_log_addrs; i < CEC_MAX_LOG_ADDRS; i++) {
fail_on_test(laddrs.log_addr_type[i] ||

Privacy Policy