diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2017-07-06 16:27:18 +0200 |
|---|---|---|
| committer | Hans Verkuil <hans.verkuil@cisco.com> | 2017-07-06 16:27:18 +0200 |
| commit | 8e68406dae2233e811032dc8e7714c09c818e893 (patch) | |
| tree | a8c850d551242300032fd146b921262657436af3 | |
| parent | 9e965f0dc0acf869a0525ad82acf54b636c17092 (diff) | |
cec-compliance: don't set app_result to -1 if the ioctl fails
There are many tests that expect an ioctl to fail, so that is not
a reason to set app_result to -1. Only do this when a test fails.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
| -rw-r--r-- | utils/cec-compliance/cec-compliance.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/utils/cec-compliance/cec-compliance.cpp b/utils/cec-compliance/cec-compliance.cpp index 2210e12c..6101e3d3 100644 --- a/utils/cec-compliance/cec-compliance.cpp +++ b/utils/cec-compliance/cec-compliance.cpp @@ -921,9 +921,6 @@ int cec_named_ioctl(struct node *node, const char *name, printf("\t\t%s: Sequence: %u Rx Timestamp: %s Length: %u\n", opname.c_str(), msg->sequence, ts2s(msg->rx_ts).c_str(), msg->len); - if (retval < 0) - app_result = -1; - if (!retval) { __u8 la = cec_msg_initiator(msg); |