diff options
author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2021-05-21 10:28:21 +0200 |
---|---|---|
committer | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2021-05-21 10:28:21 +0200 |
commit | bf461ab00c6e3dad0d38c85b3b1292e45195c8f5 (patch) | |
tree | 1b839b411aae4751d6578fd96e20ccb787bdcfe3 | |
parent | f0c7e3d71eaf4182bae7eb3ee0e43b4eeb047ea9 (diff) | |
download | v4l-utils-bf461ab00c6e3dad0d38c85b3b1292e45195c8f5.tar.gz |
cec-compliance: clarify a warning
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 <hverkuil-cisco@xs4all.nl>
-rw-r--r-- | utils/cec-compliance/cec-test-audio.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/utils/cec-compliance/cec-test-audio.cpp b/utils/cec-compliance/cec-test-audio.cpp index bdbcd178..9471865e 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); |