| Message ID | 1341240671-5843-7-git-send-email-eugeni.dodonov@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, 2 Jul 2012 11:51:07 -0300 Eugeni Dodonov <eugeni.dodonov@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@intel.com> Reviewed-by: Ben Widawsky <ben@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.
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
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@intel.com> --- drivers/gpu/drm/i915/intel_pm.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)