Re: [igt-dev] [PATCH] [i-g-t] tests/amdgpu/amd_bypass: Skip test when DP Rx output bpc < 8 bpc

[Alex Hung <alex.hung@amd.com>]

Reviewed-by: Alex Hung <alex.hung@amd.com>

On 2023-06-20 15:20, Hersen Wu wrote:
> Check DP or eDP Rx output bpc, skip test when output bpc is
> less than 8bpc.
> 
> This test compares CRC of CRTC and Rx. It needs both CRTC_CRC
> and Rx-CRC with 8bpc. If CRTC-CRC with 8bpc, but Rx-CRC with
> 6bpc, then their values  will not match.
> 
> Signed-off-by: Hersen Wu <hersenxs.wu@amd.com>
> ---
>   tests/amdgpu/amd_bypass.c | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/tests/amdgpu/amd_bypass.c b/tests/amdgpu/amd_bypass.c
> index d191779b0..d192a30ea 100644
> --- a/tests/amdgpu/amd_bypass.c
> +++ b/tests/amdgpu/amd_bypass.c
> @@ -331,6 +331,13 @@ static void bypass_8bpc_test(data_t *data)
>   	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);
>   
> +	/* 8bpc bypass only valid when DP or eDP Rx supports 8bpc or up. When
> +	 * Rx supports only up to 6bpc, Rx-crc will different from crtc-crc
> +	 * with 8bpc.
> +	 */
> +	igt_skip_on_f(igt_get_output_max_bpc(data->drm_fd, data->output->name) <= 6,
> +		      "check /sys/kernel/debug/dri/0/eDP-1 (connector)/output_bpc\n");
> +
>   	igt_create_fb(data->drm_fd, data->width, data->height,
>   		      DRM_FORMAT_XRGB8888, DRM_FORMAT_MOD_LINEAR, &fb);
>