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

[Riana Tauro]

On 11/21/2023 3:09 PM, Karthik Poosa wrote:
> 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.
> 
> v2: Updated commit message based on review comments. [Riana]
> 
> Signed-off-by: Karthik Poosa <karthik.poosa at intel.com>
Looks good to me

Reviewed-by: Riana Tauro <riana.tauro 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);
>