On Wed, Feb 22, 2012 at 8:56 PM, Dave Airlie <airlied at linux.ie> wrote:
> Eugeni Dodonov (4):
> ? ? ?drm/i915: gen7: implement rczunit workaround
> ? ? ?drm/i915: gen7: Implement an L3 caching workaround.
> ? ? ?drm/i915: gen7: work around a system hang on IVB
> ? ? ?drm/i915: do not enable RC6p on Sandy Bridge
That last patch about RC6p looks wrong.
It does:
GEN6_RC_CTL_RC6_ENABLE |
(IS_GEN7(dev_priv->dev)) ? GEN6_RC_CTL_RC6p_ENABLE : 0;
But I think this was meant:
GEN6_RC_CTL_RC6_ENABLE |
((IS_GEN7(dev_priv->dev)) ? GEN6_RC_CTL_RC6p_ENABLE :
0);
Or did I get the operator precedence wrong?
HTH
Torsten
