[Intel-gfx] [PATCH] drm/i915: make default minimum frequency RP1 instead of RPN

Jesse Barnes jbarnes at virtuousgeek.org
Fri Oct 26 21:24:51 CEST 2012


On Fri, 26 Oct 2012 11:32:08 -0700
Eric Anholt <eric at anholt.net> wrote:

> Jesse Barnes <jbarnes at virtuousgeek.org> writes:
> 
> > We should only ever go below RP1 for thermal reasons.  Users can still
> > control this through sysfs by manually reducing the minimum frequency
> > (and max if they want to clamp their frequency for power reasons, e.g.
> > to keep fans off or something).
> 
> I think I know what the reasoning is, but this commit message could use
> some justification on why you wouldn't want to use the lowest frequency
> available.

Yeah I should have been more verbose.

RP1 is the lowest "normal" frequency, in that it's the highest we can
run at the base voltage level for the part.  This means that any lower
frequencies will simply make things take longer and not make up for it
in power savings because the voltage is already at the minimum.

Daniel, feel free to add the above to the changelog if you want.

-- 
Jesse Barnes, Intel Open Source Technology Center



More information about the Intel-gfx mailing list