aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHans Verkuil <hverkuil-cisco@xs4all.nl>2021-05-21 10:28:21 +0200
committerHans Verkuil <hverkuil-cisco@xs4all.nl>2021-05-21 10:28:21 +0200
commitbf461ab00c6e3dad0d38c85b3b1292e45195c8f5 (patch)
tree1b839b411aae4751d6578fd96e20ccb787bdcfe3
parentf0c7e3d71eaf4182bae7eb3ee0e43b4eeb047ea9 (diff)
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.cpp7
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);

Privacy Policy