On Mon, 2 Jul 2012 11:51:07 -0300 Eugeni Dodonov <eugeni.dodonov at intel.com> wrote: > It should be working so let's turn it on by default and catch any possible > issues faster. > > Signed-off-by: Eugeni Dodonov <eugeni.dodonov at intel.com> Reviewed-by: Ben Widawsky <ben at bwidawsk.net> > --- > drivers/gpu/drm/i915/intel_pm.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index 64cd97e..b00df1f 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -2332,9 +2332,11 @@ int intel_enable_rc6(const struct drm_device *dev) > if (INTEL_INFO(dev)->gen == 5) > return 0; > > - /* Sorry Haswell, no RC6 for you for now. */ > + /* On Haswell, only RC6 is available. So let's enable it by default to > + * provide better testing and coverage since the beginning. > + */ > if (IS_HASWELL(dev)) > - return 0; > + return INTEL_RC6_ENABLE; > > /* > * Disable rc6 on Sandybridge I think you can/should fold this in with the existing case. -- Ben Widawsky, Intel Open Source Technology Center
