Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/xe: Prevent runtime PM wake while reading rp0 frequency
@ 2025-10-15  9:46 Badal Nilawar
  2025-10-15 11:38 ` Poosa, Karthik
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Badal Nilawar @ 2025-10-15  9:46 UTC (permalink / raw)
  To: intel-xe; +Cc: rodrigo.vivi, vinay.belgaumkar, sk.anirban

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);
 
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2025-10-16 19:06 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-15  9:46 [PATCH] drm/xe: Prevent runtime PM wake while reading rp0 frequency Badal Nilawar
2025-10-15 11:38 ` Poosa, Karthik
2025-10-16 19:05   ` Rodrigo Vivi
2025-10-15 14:41 ` ✗ CI.checkpatch: warning for " Patchwork
2025-10-15 14:42 ` ✓ CI.KUnit: success " Patchwork
2025-10-15 15:20 ` ✓ Xe.CI.BAT: " Patchwork
2025-10-15 19:19 ` [PATCH] " Belgaumkar, Vinay
2025-10-16  0:10 ` ✗ Xe.CI.Full: failure for " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox