[igt-dev] [PATCH i-g-t] tests/kms_invalid_dotclock: Skip the test if big joiner is supported

Petri Latvala petri.latvala at intel.com
Thu Feb 25 09:23:21 UTC 2021


On Thu, Feb 25, 2021 at 02:39:54PM +0530, Ankit Nautiyal wrote:
> Currently the test assumes that the modes with clock more than the
> maximum dot clock will be rejected. This fails in case of
> platforms that combine multiple pipes and modes higher than the
> maximum dotclock can still be supported.
> 
> This patch adds a check to skip the test if big joiner can be
> supported for a given platform and connector.
> 
> Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
> ---
>  tests/kms_invalid_dotclock.c | 51 ++++++++++++++++++++++++++++++++++--
>  1 file changed, 49 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/kms_invalid_dotclock.c b/tests/kms_invalid_dotclock.c
> index 402629ab..5692736b 100644
> --- a/tests/kms_invalid_dotclock.c
> +++ b/tests/kms_invalid_dotclock.c
> @@ -44,6 +44,44 @@ static bool has_scaling_mode_prop(data_t *data)
>  				    NULL, NULL, NULL);
>  }
>  
> +static bool
> +is_dsc_capable(int drm_fd, drmModeConnector *connector)
> +{
> +	int debugfs_fd = igt_debugfs_dir(drm_fd);
> +	char filename[128];
> +	char buf[512];
> +	bool dsc = false;
> +
> +	sprintf(filename, "%s-%d",
> +		kmstest_connector_type_str(connector->connector_type),
> +		connector->connector_type_id);
> +
> +	strcat(filename, "/i915_dsc_fec_support");
> +
> +	if (igt_debugfs_simple_read(debugfs_fd, filename, buf,
> +				    sizeof(buf)) > 0)
> +		dsc = strstr(buf, "DSC_Sink_Support: yes") ? true : false;
> +
> +	close(debugfs_fd);
> +
> +	return dsc;
> +}

Consider moving this to lib/. There's another such function in
kms_dp_dsc.c already.


-- 
Petri Latvala


More information about the igt-dev mailing list