[igt-dev] [v2 4/9] tests/i915/kms_dsc: add get_status()
Nautiyal, Ankit K
ankit.k.nautiyal at intel.com
Mon Jul 24 05:29:46 UTC 2023
On 7/4/2023 11:01 PM, Swati Sharma wrote:
> Use get_status() to get current bpc when TEST_DSC_BPC test
> flag is used. If input bpc is not equal to current bpc,
> skip test.
>
> Signed-off-by: Swati Sharma <swati2.sharma at intel.com>
> ---
> tests/i915/kms_dsc.c | 14 +++++++++++++-
> 1 file changed, 13 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/kms_dsc.c b/tests/i915/kms_dsc.c
> index 7b9bae91a..df9b27225 100644
> --- a/tests/i915/kms_dsc.c
> +++ b/tests/i915/kms_dsc.c
> @@ -111,12 +111,18 @@ static void test_cleanup(data_t *data)
> igt_remove_fb(data->drm_fd, &data->fb_test_pattern);
> }
>
> +static int get_status(int drmfd, enum pipe pipe)
> +{
> + return igt_get_pipe_current_bpc(drmfd, pipe);
> +}
I think we can directly call the lib function, so this is not required.
Otherwise the change looks good to me.
Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
> +
> /* re-probe connectors and do a modeset with DSC */
> static void update_display(data_t *data, uint32_t test_type)
> {
> int ret;
> bool enabled;
> int index = 0;
> + int current_bpc = 0;
> igt_plane_t *primary;
> drmModeModeInfo *mode;
> igt_output_t *output = data->output;
> @@ -202,11 +208,17 @@ static void update_display(data_t *data, uint32_t test_type)
>
> restore_force_dsc_en();
>
> + if (test_type & TEST_DSC_BPC) {
> + current_bpc = get_status(data->drm_fd, data->pipe);
> + igt_skip_on_f(data->input_bpc != current_bpc,
> + "Input bpc = %d is not equal to current bpc = %d\n",
> + data->input_bpc, current_bpc);
> + }
> +
> igt_assert_f(enabled,
> "Default DSC enable failed on connector: %s pipe: %s\n",
> output->name,
> kmstest_pipe_name(data->pipe));
> -
> reset:
> test_reset(data);
>
More information about the igt-dev
mailing list