[igt-dev] [PATCH] tests/amdgpu/amd_bypass: skip if DSC is enabled on the connector

[David Zhang]

[why & how]
Since the DSC is visually lossless but loss still exists after
compression/decompression at source/destination sides. So for any
bypass mode validation, which requires what received at sink is
consistent to what is transmitted at source, DSC is required to be
turned off.

skip the test run if the DSC is enabled on the connector (output).

Signed-off-by: David Zhang <dingchen.zhang at amd.com>
---
 tests/amdgpu/amd_bypass.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/tests/amdgpu/amd_bypass.c b/tests/amdgpu/amd_bypass.c
index 75ad8521..5ec90884 100644
--- a/tests/amdgpu/amd_bypass.c
+++ b/tests/amdgpu/amd_bypass.c
@@ -21,6 +21,7 @@
  *
  */
 #include "igt.h"
+#include "igt_amd.h"
 
 /*
  * internal use
@@ -315,6 +316,14 @@ static void bypass_8bpc_test(data_t *data)
 
 	test_init(data);
 
+	/**
+	 * 8bpc bypass only makes sense without DSC.
+	 * DSC is visually lossless but actually still loss exists and any bypass mode
+	 * cannot be run w/ DSC enabled.
+	 */
+	igt_skip_on_f(igt_amd_read_dsc_clock_status(data->drm_fd, data->output->name) == 1,
+		      "DSC enabled on %s and no sense to validate bypass mode\n", data->output->name);
+
 	igt_create_fb(data->drm_fd, data->width, data->height,
 		      DRM_FORMAT_XRGB8888, DRM_FORMAT_MOD_LINEAR, &fb);
 
-- 
2.25.1