aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohan Fjeldtvedt <jaffe1@gmail.com>2016-08-11 13:32:18 +0200
committerHans Verkuil <hans.verkuil@cisco.com>2016-08-12 13:27:10 +0200
commite9017b0275813b5eb84ea46f62cd631f07139ae9 (patch)
tree2e7f95c95b559aa68c9deebd2126212726d0e6d2
parent5be09ccd41c481fed13f45f0635c1807196fc86c (diff)
cec-compliance: fix wrong abort reason
We should treat "Unable to determine" as not supported, not "Refused". Signed-off-by: Johan Fjeldtvedt <jaffe1@gmail.com> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
-rw-r--r--utils/cec-compliance/cec-compliance.h10
1 files changed, 5 insertions, 5 deletions
diff --git a/utils/cec-compliance/cec-compliance.h b/utils/cec-compliance/cec-compliance.h
index 2403c3c7..1eeb912d 100644
--- a/utils/cec-compliance/cec-compliance.h
+++ b/utils/cec-compliance/cec-compliance.h
@@ -300,8 +300,8 @@ static inline bool unrecognized_op(const struct cec_msg *msg)
return false;
if (abort_reason(msg) == CEC_OP_ABORT_UNRECOGNIZED_OP)
return true;
- if (abort_reason(msg) == CEC_OP_ABORT_REFUSED) {
- warn("Opcode %x was refused and is treated as not supported.\n", msg->msg[2]);
+ if (abort_reason(msg) == CEC_OP_ABORT_UNDETERMINED) {
+ warn("Opcode %x was undetermined and is treated as not supported.\n", msg->msg[2]);
return true;
}
return false;
@@ -348,7 +348,7 @@ static inline bool transmit_timeout(struct node *node, struct cec_msg *msg,
switch (abort_reason(msg)) {
case CEC_OP_ABORT_UNRECOGNIZED_OP:
- case CEC_OP_ABORT_REFUSED:
+ case CEC_OP_ABORT_UNDETERMINED:
return true;
case CEC_OP_ABORT_INVALID_OP:
reason = "Invalid operand";
@@ -356,8 +356,8 @@ static inline bool transmit_timeout(struct node *node, struct cec_msg *msg,
case CEC_OP_ABORT_NO_SOURCE:
reason = "Cannot provide source";
break;
- case CEC_OP_ABORT_UNDETERMINED:
- reason = "Unable to determine";
+ case CEC_OP_ABORT_REFUSED:
+ reason = "Refused";
break;
case CEC_OP_ABORT_INCORRECT_MODE:
reason = "Incorrect mode";

Privacy Policy