On Thu, 23 Feb 2012 23:57:06 -0200 Eugeni Dodonov <eugeni.dodo...@intel.com> wrote:
> As noticed by Torsten Kaiser, the operator precedence can play tricks with > us here. > > CC: Dave Airlie <airl...@redhat.com> > CC: Jesse Barnes <jbar...@virtuousgeek.org> > Signed-off-by: Eugeni Dodonov <eugeni.dodo...@intel.com> > --- > drivers/gpu/drm/i915/intel_display.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index e654f32..4871ba0 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -8194,7 +8194,7 @@ void gen6_enable_rps(struct drm_i915_private *dev_priv) > > if (intel_enable_rc6(dev_priv->dev)) > rc6_mask = GEN6_RC_CTL_RC6_ENABLE | > - (IS_GEN7(dev_priv->dev)) ? GEN6_RC_CTL_RC6p_ENABLE : 0; > + ((IS_GEN7(dev_priv->dev)) ? GEN6_RC_CTL_RC6p_ENABLE : > 0); > > I915_WRITE(GEN6_RC_CONTROL, > rc6_mask | ugg yeah, with the old code we'd never end up enabling rc6 at all. Pushed. Thanks, -- Jesse Barnes, Intel Open Source Technology Center
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
