On Thu, 23 Feb 2012 23:57:06 -0200 Eugeni Dodonov <eugeni.dodonov@xxxxxxxxx> wrote: > As noticed by Torsten Kaiser, the operator precedence can play tricks with > us here. > > CC: Dave Airlie <airlied@xxxxxxxxxx> > CC: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> > Signed-off-by: Eugeni Dodonov <eugeni.dodonov@xxxxxxxxx> > --- > 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
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
