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: remove obsolete cec_msg_record_on/off workaround + fix typo Author: Hans Verkuil <hans.verk...@cisco.com> Date: Wed Oct 12 19:29:40 2016 +0200 Remove the obsolete workaround for cec_msg_record_on/off where the reply argument was missing: this has now been fixed in cec-funcs.h. Also fix a spelling mistake. Signed-off-by: Hans Verkuil <hans.verk...@cisco.com> utils/cec-compliance/cec-test.cpp | 44 +-------------------------------------- 1 file changed, 1 insertion(+), 43 deletions(-) --- http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=9f5f5e4692e316ed7cb2bb0d5e0a7c72f132f4d3 diff --git a/utils/cec-compliance/cec-test.cpp b/utils/cec-compliance/cec-test.cpp index e261453e875a..7c6ac2e157dd 100644 --- a/utils/cec-compliance/cec-test.cpp +++ b/utils/cec-compliance/cec-test.cpp @@ -31,48 +31,6 @@ #include "cec-compliance.h" -/* - * cec-funcs.h doesn't have the reply argument for these CEC messages. - * A fix is pending but not yet merged. Once it is merged this workaround - * can be removed. - */ -static inline void cec_msg_record_on_reply(struct cec_msg *msg, - bool reply, - const struct cec_op_record_src *rec_src) -{ - switch (rec_src->type) { - case CEC_OP_RECORD_SRC_OWN: - cec_msg_record_on_own(msg); - break; - case CEC_OP_RECORD_SRC_DIGITAL: - cec_msg_record_on_digital(msg, &rec_src->digital); - break; - case CEC_OP_RECORD_SRC_ANALOG: - cec_msg_record_on_analog(msg, - rec_src->analog.ana_bcast_type, - rec_src->analog.ana_freq, - rec_src->analog.bcast_system); - break; - case CEC_OP_RECORD_SRC_EXT_PLUG: - cec_msg_record_on_plug(msg, rec_src->ext_plug.plug); - break; - case CEC_OP_RECORD_SRC_EXT_PHYS_ADDR: - cec_msg_record_on_phys_addr(msg, - rec_src->ext_phys_addr.phys_addr); - break; - } - msg->reply = reply ? CEC_MSG_RECORD_STATUS : 0; -} -#define cec_msg_record_on cec_msg_record_on_reply - -static inline void cec_msg_record_off_reply(struct cec_msg *msg, bool reply) -{ - msg->len = 2; - msg->msg[1] = CEC_MSG_RECORD_OFF; - msg->reply = reply ? CEC_MSG_RECORD_STATUS : 0; -} -#define cec_msg_record_off cec_msg_record_off_reply - #define test_case(name, tags, subtests) {name, tags, subtests, ARRAY_SIZE(subtests)} #define test_case_ext(name, tags, subtests) {name, tags, subtests, subtests##_size} @@ -1412,7 +1370,7 @@ static struct remote_test tests[] = { test_case("One Touch Record feature", TAG_ONE_TOUCH_RECORD, one_touch_rec_subtests), - test_case("Timer Progrmaming feature", + test_case("Timer Programming feature", TAG_TIMER_PROGRAMMING, timer_prog_subtests), test_case("Capability Discovery and Control feature", _______________________________________________ linuxtv-commits mailing list linuxtv-commits@linuxtv.org https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits