diff mbox

[1/1] drm/i915: Return stored value from max freq sysfs entry

Message ID 1365682079-5349-1-git-send-email-mika.kuoppala@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mika Kuoppala April 11, 2013, 12:07 p.m. UTC
commit 4f9b2fe0441d4bdf5666a306156b5d6755de2584
Author: Ben Widawsky <ben@bwidawsk.net>
Date:   Fri Apr 5 14:29:22 2013 -0700

    drm/i915: Better overclock support

changed the sysfs read semantics for 'gt_max_freq_mhz'. By
always returning overclock max instead of stored value.

Fix this by returning the stored value. Separate sysfs entry
should be considered for overclocking max freq.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=63415
Cc: Ben Widawsky <ben@bwidawsk.net>
Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
---
 drivers/gpu/drm/i915/i915_sysfs.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Daniel Vetter April 11, 2013, 3:26 p.m. UTC | #1
On Thu, Apr 11, 2013 at 03:07:59PM +0300, Mika Kuoppala wrote:
> commit 4f9b2fe0441d4bdf5666a306156b5d6755de2584
> Author: Ben Widawsky <ben@bwidawsk.net>
> Date:   Fri Apr 5 14:29:22 2013 -0700
> 
>     drm/i915: Better overclock support
> 
> changed the sysfs read semantics for 'gt_max_freq_mhz'. By
> always returning overclock max instead of stored value.
> 
> Fix this by returning the stored value. Separate sysfs entry
> should be considered for overclocking max freq.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=63415
> Cc: Ben Widawsky <ben@bwidawsk.net>
> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
Queued for -next, thanks for the patch.
-Daniel
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/i915_sysfs.c b/drivers/gpu/drm/i915/i915_sysfs.c
index fa4b4e8..d5e1890 100644
--- a/drivers/gpu/drm/i915/i915_sysfs.c
+++ b/drivers/gpu/drm/i915/i915_sysfs.c
@@ -226,7 +226,7 @@  static ssize_t gt_max_freq_mhz_show(struct device *kdev, struct device_attribute
 	int ret;
 
 	mutex_lock(&dev_priv->rps.hw_lock);
-	ret = dev_priv->rps.hw_max * GT_FREQUENCY_MULTIPLIER;
+	ret = dev_priv->rps.max_delay * GT_FREQUENCY_MULTIPLIER;
 	mutex_unlock(&dev_priv->rps.hw_lock);
 
 	return snprintf(buf, PAGE_SIZE, "%d\n", ret);