On Thu, May 24, 2012 at 1:22 PM, Jesse Barnes <[email protected]>wrote:
> static ssize_t
> +i915_min_freq_read(struct file *filp,
> + char __user *ubuf,
> + size_t max,
> + loff_t *ppos)
> +{
> + struct drm_device *dev = filp->private_data;
> + drm_i915_private_t *dev_priv = dev->dev_private;
> + char buf[80];
>
<bikeshed>
I think 20 characters, as in min_freq_write, would be enough here.
</bikeshed>
> + /*
> + * Turbo will still be enabled, but won't go above the set value.
> + */
> + dev_priv->min_delay = val / 50;
>
I guess the comment here is meant to say that turbo won't go below the set
value.
But other than that,
Reviewed-by: Eugeni Dodonov <[email protected]>
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
