Re: [PATCH i-g-t 1/7] lib/igt_kms: don't run on unsupported platform

["Nautiyal, Ankit K" <ankit.k.nautiyal@intel.com>]

On 4/18/2024 8:58 PM, Kunal Joshi wrote:
> kmd exposes debugfs for all the platforms, so add
> a check in igt
>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
> Cc: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
> Suggested-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
> ---
>   lib/igt_kms.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 3216fe7e4..6f093fed7 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -6246,6 +6246,14 @@ bool igt_has_force_joiner_debugfs(int drmfd, igt_output_t *output)
>   	char buf[512];
>   	int debugfs_fd, ret;
>   
> +	/*
> +	 * bigjoiner is supported on display>= 12 and only on
> +	 * Pipe A for Display 11
> +	 * For simplicity avoid Display 11 and check for >= 12

For display 12 joiner needs DSC also. Due to this we cannot force 
bigjoiner on machines that are not connected to DSC sinks.

So perhaps I guess it would be better to test for 13 and above, as we 
will not require DSC for that.


Regards,

Ankit


> +	 */
> +	if (intel_display_ver(intel_get_drm_devid(drmfd)) < 12)
> +		return false;
> +
>   	igt_assert_f(output->name, "Connector name cannot be NULL\n");
>   	debugfs_fd = igt_debugfs_connector_dir(drmfd, output->name, O_RDONLY);
>   	if (debugfs_fd < 0)