Re: drm/xe: Prevent runtime PM wake while reading rp0 frequency

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Oct 15, 2025 at 11:38:48AM +0000, Poosa, Karthik wrote:
> -----Original Message-----
> From: Badal Nilawar <badal.nilawar@intel.com> 
> Sent: 15 October 2025 15:16
> To: intel-xe@lists.freedesktop.org
> Cc: rodrigo.vivi@intel.com; vinay.belgaumkar@intel.com; sk.anirban@intel.com
> Subject: drm/xe: Prevent runtime PM wake while reading rp0 frequency
> 
> The rp0 frequency is a fused value that is read once during probe and then cached, so there’s no need to trigger a runtime wake when accessing rp0.
> 
> Signed-off-by: Badal Nilawar <badal.nilawar@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_gt_freq.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_gt_freq.c b/drivers/gpu/drm/xe/xe_gt_freq.c index 4ff1b6b58d6b..701349251bbc 100644
> --- a/drivers/gpu/drm/xe/xe_gt_freq.c
> +++ b/drivers/gpu/drm/xe/xe_gt_freq.c
> @@ -99,13 +99,8 @@ static ssize_t rp0_freq_show(struct kobject *kobj,  {
>  	struct device *dev = kobj_to_dev(kobj);
>  	struct xe_guc_pc *pc = dev_to_pc(dev);
> -	u32 freq;
>  
> -	xe_pm_runtime_get(dev_to_xe(dev));
> -	freq = xe_guc_pc_get_rp0_freq(pc);
> -	xe_pm_runtime_put(dev_to_xe(dev));
> -
> -	return sysfs_emit(buf, "%d\n", freq);
> +	return sysfs_emit(buf, "%d\n", xe_guc_pc_get_rp0_freq(pc));
>  }
>  static struct kobj_attribute attr_rp0_freq = __ATTR_RO(rp0_freq);
> 
> LGTM.
> Reviewed-by: Karthik Poosa <karthik.poosa@intel.com>

pushed to drm-xe-next, thanks for the patch and reviews

>