Re: [igt-dev] [PATCH] lib/intel_batchbuffer: Skip media-fill test for Graphics version 12.50 and above

[Karolina Stolarek <karolina.stolarek@intel.com>]

On 2.03.2023 06:54, Vikas Srivastava wrote:
> From: Sinjan Kumar <sinjan.kumar@intel.com>
> 
> Xe_HP and beyond platforms does not have media pipeline.
> So, this test is not applicable for those platforms.
> 
> Cc: Chris Wilson <chris.p.wilson@linux.intel.com>
> Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
> Cc: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
> Cc: Zbigniew Kempczy��ski <zbigniew.kempczynski@intel.com>

Not sure if it's your git send-email playing tricks or something, but 
char coding is off. You can either try to force git to use UTF-8 or just 
write "Kempczynski" here.

With all tags corrected:

Reviewed-by: Karolina Stolarek <karolina.stolarek@intel.com>

> Cc: Priyanka Dandamudi <priyanka.dandamudi@intel.com>
> Cc: Rahul Kumar <janga.rahul.kumar@intel.com>
> Reviewed-by: Chris Wilson <chris.p.wilson@linux.intel.com>
> Signed-off-by: Sinjan Kumar <sinjan.kumar@intel.com>
> Signed-off-by: Vikas Srivastava <vikas.srivastava@intel.com>
> ---
>   lib/intel_batchbuffer.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/intel_batchbuffer.c b/lib/intel_batchbuffer.c
> index 59c788e68..cbb513f79 100644
> --- a/lib/intel_batchbuffer.c
> +++ b/lib/intel_batchbuffer.c
> @@ -643,7 +643,9 @@ igt_fillfunc_t igt_get_media_fillfunc(int devid)
>   {
>   	igt_fillfunc_t fill = NULL;
>   
> -	if (IS_GEN12(devid))
> +	if (intel_graphics_ver(devid) >= IP_VER(12, 50)) {
> +		/* current implementation defeatured PIPELINE_MEDIA */
> +	} else if (IS_GEN12(devid))
>   		fill = gen12_media_fillfunc;
>   	else if (IS_GEN9(devid) || IS_GEN10(devid) || IS_GEN11(devid))
>   		fill = gen9_media_fillfunc;