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: clarify a warning Author: Hans Verkuil <[email protected]> Date: Fri May 21 10:28:21 2021 +0200 The warning when a reserved audio out delay value was used didn't include that value 1 should be used instead. Clarify this. Signed-off-by: Hans Verkuil <[email protected]> utils/cec-compliance/cec-test-audio.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) --- http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=bf461ab00c6e3dad0d38c85b3b1292e45195c8f5 diff --git a/utils/cec-compliance/cec-test-audio.cpp b/utils/cec-compliance/cec-test-audio.cpp index bdbcd1789956..9471865e4765 100644 --- a/utils/cec-compliance/cec-test-audio.cpp +++ b/utils/cec-compliance/cec-test-audio.cpp @@ -51,9 +51,10 @@ static int dal_request_current_latency(struct node *node, unsigned me, unsigned } else { // Although this value will be ignored, it shouldn't use // reserved values. - warn_on_test(audio_out_delay == 0 || audio_out_delay > 251); - if (audio_out_delay > 1 && audio_out_delay <= 251) - warn("Audio out delay is %d (%dms), but value 1 is recommended when this field is unused\n", + if (audio_out_delay == 0 || audio_out_delay > 251) + warn("Audio out delay is set to a reserved value (%d), set it to 1 instead (recommended value when this field is unused).\n", audio_out_delay); + else + warn("Audio out delay is %d (%dms), but value 1 is recommended when this field is unused.\n", audio_out_delay, (audio_out_delay - 1) * 2); } fail_on_test(video_latency == 0 || video_latency > 251); _______________________________________________ linuxtv-commits mailing list [email protected] https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits
