From: c0d1n61at3 at gmail.com (Jiunn Chang) Subject: [Linux-kernel-mentees] [PATCH v5 2/2] Add test for new features in cec-follower Date: Wed, 2 Oct 2019 22:18:16 -0500 [thread overview] Message-ID: <20191003031816.168985-3-c0d1n61at3@gmail.com> (raw) In-Reply-To: <20191001031823.312020-1-c0d1n61at3@gmail.com> Analog tuner control test tuner_ctl_test(): - give analog tuner status - select tuner analog service - analog tuner step features Signed-off-by: Jiunn Chang <c0d1n61at3 at gmail.com> --- utils/cec-compliance/cec-test.cpp | 74 +++++++++++++++++++++++++++++++ 1 file changed, 74 insertions(+) diff --git a/utils/cec-compliance/cec-test.cpp b/utils/cec-compliance/cec-test.cpp index 017d1ef8..669df4b7 100644 --- a/utils/cec-compliance/cec-test.cpp +++ b/utils/cec-compliance/cec-test.cpp @@ -16,6 +16,7 @@ #include <sys/ioctl.h> #include <config.h> #include <sstream> +#include <vector> #include "cec-compliance.h" @@ -722,6 +723,78 @@ static struct remote_subtest deck_ctl_subtests[] = { TODO: These are very rudimentary tests which should be expanded. */ +static int tuner_ctl_test(struct node *node, unsigned me, unsigned la, bool interactive) +{ + struct cec_msg msg = {}; + struct cec_op_tuner_device_info info = {}; + __u16 start_freq; + std::vector<struct cec_op_tuner_device_info> info_vec; + + cec_msg_init(&msg, me, la); + cec_msg_give_tuner_device_status(&msg, true, CEC_OP_STATUS_REQ_ONCE); + fail_on_test(!transmit_timeout(node, &msg)); + if (timed_out(&msg) || unrecognized_op(&msg)) + return NOTSUPPORTED; + if (refused(&msg)) + return REFUSED; + cec_ops_tuner_device_status(&msg, &info); + start_freq = info.analog.ana_freq; + info_vec.push_back(info); + + while (1) { + cec_msg_init(&msg, me, la); + cec_msg_tuner_step_increment(&msg); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(cec_msg_status_is_abort(&msg)); + if (cec_msg_status_is_abort(&msg)) { + fail_on_test(abort_reason(&msg) == CEC_OP_ABORT_UNRECOGNIZED_OP); + if (abort_reason(&msg) == CEC_OP_ABORT_REFUSED) { + warn("Tuner step increment does not wrap.\n"); + break; + } else { + warn("Tuner at end of service list did not receive feature abort refused.\n"); + break; + } + } + cec_msg_init(&msg, me, la); + cec_msg_give_tuner_device_status(&msg, true, CEC_OP_STATUS_REQ_ONCE); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(timed_out_or_abort(&msg)); + cec_ops_tuner_device_status(&msg, &info); + if (info.analog.ana_freq == start_freq) + break; + info_vec.push_back(info); + } + + for (std::vector<struct cec_op_tuner_device_info>::iterator iter = info_vec.begin(); + iter != info_vec.end(); iter++) { + cec_msg_init(&msg, me, la); + cec_msg_select_analogue_service(&msg, iter->analog.ana_bcast_type, + iter->analog.ana_freq, iter->analog.bcast_system); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(cec_msg_status_is_abort(&msg)); + cec_msg_init(&msg, me, la); + cec_msg_give_tuner_device_status(&msg, true, CEC_OP_STATUS_REQ_ONCE); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(timed_out_or_abort(&msg)); + cec_ops_tuner_device_status(&msg, &info); + fail_on_test(info.analog.ana_freq != iter->analog.ana_freq); + if (iter->is_analog) { + double freq_mhz = (iter->analog.ana_freq * 625) / 10000.0; + info("Analog channel freq: %.2f MHz\n", freq_mhz); + } + } + + cec_msg_init(&msg, me, la); + cec_msg_select_analogue_service(&msg, 3, 16000, 9); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(!cec_msg_status_is_abort(&msg)); + fail_on_test(unrecognized_op(&msg)); + fail_on_test(abort_reason(&msg) != CEC_OP_ABORT_INVALID_OP); + + return 0; +} + static int tuner_ctl_give_status(struct node *node, unsigned me, unsigned la, bool interactive) { struct cec_msg msg = {}; @@ -885,6 +958,7 @@ static int tuner_ctl_step_inc(struct node *node, unsigned me, unsigned la, bool } static struct remote_subtest tuner_ctl_subtests[] = { + { "Tuner Control", CEC_LOG_ADDR_MASK_TUNER | CEC_LOG_ADDR_MASK_TV, tuner_ctl_test }, { "Give Tuner Device Status", CEC_LOG_ADDR_MASK_TUNER, tuner_ctl_give_status }, { "Select Analogue Service", CEC_LOG_ADDR_MASK_TUNER, tuner_ctl_sel_analog_service }, { "Select Digital Service", CEC_LOG_ADDR_MASK_TUNER, tuner_ctl_sel_digital_service }, -- 2.23.0
WARNING: multiple messages have this Message-ID (diff)
From: c0d1n61at3@gmail.com (Jiunn Chang) Subject: [Linux-kernel-mentees] [PATCH v5 2/2] Add test for new features in cec-follower Date: Wed, 2 Oct 2019 22:18:16 -0500 [thread overview] Message-ID: <20191003031816.168985-3-c0d1n61at3@gmail.com> (raw) Message-ID: <20191003031816.VoGy2F3POOpKVWgpKHMehcVqQeDzHBRVVa84ptNC9HA@z> (raw) In-Reply-To: <20191001031823.312020-1-c0d1n61at3@gmail.com> Analog tuner control test tuner_ctl_test(): - give analog tuner status - select tuner analog service - analog tuner step features Signed-off-by: Jiunn Chang <c0d1n61at3 at gmail.com> --- utils/cec-compliance/cec-test.cpp | 74 +++++++++++++++++++++++++++++++ 1 file changed, 74 insertions(+) diff --git a/utils/cec-compliance/cec-test.cpp b/utils/cec-compliance/cec-test.cpp index 017d1ef8..669df4b7 100644 --- a/utils/cec-compliance/cec-test.cpp +++ b/utils/cec-compliance/cec-test.cpp @@ -16,6 +16,7 @@ #include <sys/ioctl.h> #include <config.h> #include <sstream> +#include <vector> #include "cec-compliance.h" @@ -722,6 +723,78 @@ static struct remote_subtest deck_ctl_subtests[] = { TODO: These are very rudimentary tests which should be expanded. */ +static int tuner_ctl_test(struct node *node, unsigned me, unsigned la, bool interactive) +{ + struct cec_msg msg = {}; + struct cec_op_tuner_device_info info = {}; + __u16 start_freq; + std::vector<struct cec_op_tuner_device_info> info_vec; + + cec_msg_init(&msg, me, la); + cec_msg_give_tuner_device_status(&msg, true, CEC_OP_STATUS_REQ_ONCE); + fail_on_test(!transmit_timeout(node, &msg)); + if (timed_out(&msg) || unrecognized_op(&msg)) + return NOTSUPPORTED; + if (refused(&msg)) + return REFUSED; + cec_ops_tuner_device_status(&msg, &info); + start_freq = info.analog.ana_freq; + info_vec.push_back(info); + + while (1) { + cec_msg_init(&msg, me, la); + cec_msg_tuner_step_increment(&msg); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(cec_msg_status_is_abort(&msg)); + if (cec_msg_status_is_abort(&msg)) { + fail_on_test(abort_reason(&msg) == CEC_OP_ABORT_UNRECOGNIZED_OP); + if (abort_reason(&msg) == CEC_OP_ABORT_REFUSED) { + warn("Tuner step increment does not wrap.\n"); + break; + } else { + warn("Tuner at end of service list did not receive feature abort refused.\n"); + break; + } + } + cec_msg_init(&msg, me, la); + cec_msg_give_tuner_device_status(&msg, true, CEC_OP_STATUS_REQ_ONCE); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(timed_out_or_abort(&msg)); + cec_ops_tuner_device_status(&msg, &info); + if (info.analog.ana_freq == start_freq) + break; + info_vec.push_back(info); + } + + for (std::vector<struct cec_op_tuner_device_info>::iterator iter = info_vec.begin(); + iter != info_vec.end(); iter++) { + cec_msg_init(&msg, me, la); + cec_msg_select_analogue_service(&msg, iter->analog.ana_bcast_type, + iter->analog.ana_freq, iter->analog.bcast_system); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(cec_msg_status_is_abort(&msg)); + cec_msg_init(&msg, me, la); + cec_msg_give_tuner_device_status(&msg, true, CEC_OP_STATUS_REQ_ONCE); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(timed_out_or_abort(&msg)); + cec_ops_tuner_device_status(&msg, &info); + fail_on_test(info.analog.ana_freq != iter->analog.ana_freq); + if (iter->is_analog) { + double freq_mhz = (iter->analog.ana_freq * 625) / 10000.0; + info("Analog channel freq: %.2f MHz\n", freq_mhz); + } + } + + cec_msg_init(&msg, me, la); + cec_msg_select_analogue_service(&msg, 3, 16000, 9); + fail_on_test(!transmit_timeout(node, &msg)); + fail_on_test(!cec_msg_status_is_abort(&msg)); + fail_on_test(unrecognized_op(&msg)); + fail_on_test(abort_reason(&msg) != CEC_OP_ABORT_INVALID_OP); + + return 0; +} + static int tuner_ctl_give_status(struct node *node, unsigned me, unsigned la, bool interactive) { struct cec_msg msg = {}; @@ -885,6 +958,7 @@ static int tuner_ctl_step_inc(struct node *node, unsigned me, unsigned la, bool } static struct remote_subtest tuner_ctl_subtests[] = { + { "Tuner Control", CEC_LOG_ADDR_MASK_TUNER | CEC_LOG_ADDR_MASK_TV, tuner_ctl_test }, { "Give Tuner Device Status", CEC_LOG_ADDR_MASK_TUNER, tuner_ctl_give_status }, { "Select Analogue Service", CEC_LOG_ADDR_MASK_TUNER, tuner_ctl_sel_analog_service }, { "Select Digital Service", CEC_LOG_ADDR_MASK_TUNER, tuner_ctl_sel_digital_service }, -- 2.23.0
next prev parent reply other threads:[~2019-10-03 3:18 UTC|newest] Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-09-24 14:17 [Linux-kernel-mentees] [PATCH 0/2] cec-compliance: tuner control c0d1n61at3 2019-09-24 14:17 ` Jiunn Chang 2019-09-24 14:17 ` [Linux-kernel-mentees] [PATCH 1/2] cec-follower: add tuner step increment/decrement c0d1n61at3 2019-09-24 14:17 ` Jiunn Chang 2019-09-24 15:20 ` hverkuil 2019-09-24 15:20 ` Hans Verkuil 2019-09-24 14:17 ` [Linux-kernel-mentees] [PATCH 2/2] cec-compliance: add/refactor tuner control tests c0d1n61at3 2019-09-24 14:17 ` Jiunn Chang 2019-09-24 19:24 ` [Linux-kernel-mentees] [PATCH v2 0/2] cec-compliance: tuner control c0d1n61at3 2019-09-24 19:24 ` Jiunn Chang 2019-09-30 4:30 ` [Linux-kernel-mentees] [PATCH v3 0/3] " c0d1n61at3 2019-09-30 4:30 ` Jiunn Chang 2019-10-01 3:18 ` [Linux-kernel-mentees] [PATCH v4 " c0d1n61at3 2019-10-01 3:18 ` Jiunn Chang 2019-10-03 3:18 ` [Linux-kernel-mentees] [PATCH v5 0/2] " c0d1n61at3 2019-10-03 3:18 ` Jiunn Chang 2019-10-04 4:05 ` [Linux-kernel-mentees] [PATCH v6 0/1] " c0d1n61at3 2019-10-04 4:05 ` Jiunn Chang 2019-10-04 4:05 ` [Linux-kernel-mentees] [PATCH v6 1/1] Add test for new features in cec-follower c0d1n61at3 2019-10-04 4:05 ` Jiunn Chang 2019-10-03 3:18 ` [Linux-kernel-mentees] [PATCH v5 1/2] cec-follower: add tuner step increment/decrement c0d1n61at3 2019-10-03 3:18 ` Jiunn Chang 2019-10-03 3:18 ` c0d1n61at3 [this message] 2019-10-03 3:18 ` [Linux-kernel-mentees] [PATCH v5 2/2] Add test for new features in cec-follower Jiunn Chang 2019-10-03 6:51 ` hverkuil 2019-10-03 6:51 ` Hans Verkuil 2019-10-01 3:18 ` [Linux-kernel-mentees] [PATCH v4 1/3] cec-follower: fix bugs for tuner emulation c0d1n61at3 2019-10-01 3:18 ` Jiunn Chang 2019-10-01 3:18 ` [Linux-kernel-mentees] [PATCH v4 2/3] cec-follower: add tuner step increment/decrement c0d1n61at3 2019-10-01 3:18 ` Jiunn Chang 2019-10-01 7:12 ` hverkuil 2019-10-01 7:12 ` Hans Verkuil 2019-10-01 3:18 ` [Linux-kernel-mentees] [PATCH v4 3/3] cec-compliance: add tuner control test c0d1n61at3 2019-10-01 3:18 ` Jiunn Chang 2019-10-01 7:51 ` hverkuil 2019-10-01 7:51 ` Hans Verkuil 2019-09-30 4:30 ` [Linux-kernel-mentees] [PATCH v3 1/3] cec-follower: fix bugs for tuner emulation c0d1n61at3 2019-09-30 4:30 ` Jiunn Chang 2019-09-30 9:12 ` hverkuil 2019-09-30 9:12 ` Hans Verkuil 2019-09-30 4:30 ` [Linux-kernel-mentees] [PATCH v3 2/3] cec-follower: add tuner step increment/decrement c0d1n61at3 2019-09-30 4:30 ` Jiunn Chang 2019-09-30 9:35 ` hverkuil 2019-09-30 9:35 ` Hans Verkuil 2019-09-30 4:30 ` [Linux-kernel-mentees] [PATCH v3 3/3] cec-compliance: refactor tuner control tests c0d1n61at3 2019-09-30 4:30 ` Jiunn Chang 2019-09-30 9:43 ` hverkuil 2019-09-30 9:43 ` Hans Verkuil 2019-09-24 19:24 ` [Linux-kernel-mentees] [PATCH v2 1/2] cec-follower: add tuner step increment/decrement c0d1n61at3 2019-09-24 19:24 ` Jiunn Chang 2019-09-24 19:31 ` skhan 2019-09-24 19:31 ` Shuah Khan 2019-09-25 6:54 ` hverkuil 2019-09-25 6:54 ` Hans Verkuil 2019-09-25 7:12 ` hverkuil 2019-09-25 7:12 ` Hans Verkuil 2019-09-24 19:24 ` [Linux-kernel-mentees] [PATCH v2 2/2] cec-compliance: add/refactor tuner control tests c0d1n61at3 2019-09-24 19:24 ` Jiunn Chang 2019-09-24 19:44 ` skhan 2019-09-24 19:44 ` Shuah Khan 2019-09-25 7:22 ` hverkuil 2019-09-25 7:22 ` Hans Verkuil
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20191003031816.168985-3-c0d1n61at3@gmail.com \ --to=linux-kernel-mentees@lists.linux-foundation.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).