Re: [PATCH i-g-t v2] tests/intel/kms_cdclk: skip cdclk tests for bmg

["Sharma, Swati2" <swati2.sharma@intel.com>]

LGTM

Reviewed-by: Swati Sharma <swati2.sharma@intel.com

On 02-Sep-24 2:02 PM, Vinod Govindapillai wrote:
> As there is only one cdclk option in BMG, these cdclk tests
> which look for change in cdclk frequency when changing
> between high and low resolution mode will fail. So skip
> these tests for bmg
> 
> v2: Add WA info as a comment (Matt Roper)
>      fix commit header description (Swati)
> 
> Acked-by: Matt Roper <matthew.d.roper@intel.com>
> Signed-off-by: Vinod Govindapillai <vinod.govindapillai@intel.com>
> ---
>   tests/intel/kms_cdclk.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/tests/intel/kms_cdclk.c b/tests/intel/kms_cdclk.c
> index 6fe6e6c3f..382b3e9d1 100644
> --- a/tests/intel/kms_cdclk.c
> +++ b/tests/intel/kms_cdclk.c
> @@ -402,6 +402,9 @@ igt_main
>   			      "Hardware doesn't support crawling/squashing.\n");
>   		igt_display_require(&data.display, data.drm_fd);
>   		igt_display_require_output(&data.display);
> +
> +		/* Wa_15015413771: Only single possible cdclk value in XE2_HPD */
> +		igt_require(!IS_BATTLEMAGE(data.devid));
>   	}
>   
>   	igt_describe("Plane scaling test to validate cdclk frequency change.");