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>

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);