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: use the actual audio_out_delay value Author: Hans Verkuil <hverkuil-ci...@xs4all.nl> Date: Fri May 21 10:48:00 2021 +0200 cec_ops_report_current_latency() hardcodes the audio_out_delay value if it is unused, ignoring the actual value. Which is fine, except when running a compliance test, then you want the actual value. Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl> utils/cec-compliance/cec-test-audio.cpp | 5 +++++ 1 file changed, 5 insertions(+) --- http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=3ba75420a6e29eb85a0bccd23ef7381e15e3efa9 diff --git a/utils/cec-compliance/cec-test-audio.cpp b/utils/cec-compliance/cec-test-audio.cpp index 9471865e4765..bf7b39209c27 100644 --- a/utils/cec-compliance/cec-test-audio.cpp +++ b/utils/cec-compliance/cec-test-audio.cpp @@ -39,6 +39,11 @@ static int dal_request_current_latency(struct node *node, unsigned me, unsigned cec_ops_report_current_latency(&msg, &phys_addr, &video_latency, &low_latency_mode, &audio_out_compensated, &audio_out_delay); + // cec_ops_report_current_latency will hardcode audio_out_delay + // if it is unused, but for this test we want the real value, so + // get it from the actual message. + if (msg.len >= 7) + audio_out_delay = msg.msg[6]; fail_on_test(phys_addr != node->remote[la].phys_addr); info("Video latency: %d (%dms)\n", video_latency, (video_latency - 1) * 2); info("Low latency mode: %d\n", low_latency_mode); _______________________________________________ linuxtv-commits mailing list linuxtv-commits@linuxtv.org https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits
