[igt-dev] [PATCH i-g-t] tests/intel: Add changes to use new sysfs frequency entries by xe

[Gupta, Anshuman]

> -----Original Message-----
> From: Poosa, Karthik <karthik.poosa at intel.com>
> Sent: Monday, November 20, 2023 8:07 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Sundaresan, Sujaritha <sujaritha.sundaresan at intel.com>; Gupta, Anshuman
> <anshuman.gupta at intel.com>; Nilawar, Badal <badal.nilawar at intel.com>;
> Poosa, Karthik <karthik.poosa at intel.com>
> Subject: [PATCH i-g-t] tests/intel: Add changes to use new sysfs frequency entries
> by xe
> 
> Updated xe set_freq and get_freq definitions to use new sysfs entries.
> The new entries will be under sub-directory freq.
> Names of the entries have also changed. They will be suffixed with _freq i.e
> /device/tile<n>/gt<n>/freq/*_freq instead of prefix.
> 
> Signed-off-by: Karthik Poosa <karthik.poosa at intel.com>
[Gupta, Anshuman] LGTM.
Reviewed-by: Anshuman Gupta <anshuman.gupta at intel.com>
> ---
>  tests/intel/xe_guc_pc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/intel/xe_guc_pc.c b/tests/intel/xe_guc_pc.c index
> fa2f20cca..bb6344449 100644
> --- a/tests/intel/xe_guc_pc.c
> +++ b/tests/intel/xe_guc_pc.c
> @@ -141,7 +141,7 @@ static int set_freq(int fd, int gt_id, const char *freq_name,
> uint32_t freq)
>  	char freq_attr[16];
>  	int gt_fd;
> 
> -	snprintf(freq_attr, sizeof(freq_attr), "freq_%s", freq_name);
> +	snprintf(freq_attr, sizeof(freq_attr), "freq/%s_freq", freq_name);
>  	gt_fd = xe_sysfs_gt_open(fd, gt_id);
>  	igt_assert(gt_fd >= 0);
> 
> @@ -159,7 +159,7 @@ static uint32_t get_freq(int fd, int gt_id, const char
> *freq_name)
>  	char freq_attr[16];
>  	int gt_fd;
> 
> -	snprintf(freq_attr, sizeof(freq_attr), "freq_%s", freq_name);
> +	snprintf(freq_attr, sizeof(freq_attr), "freq/%s_freq", freq_name);
>  	gt_fd = xe_sysfs_gt_open(fd, gt_id);
>  	igt_assert(gt_fd >= 0);
> 
> --
> 2.25.1