[Intel-gfx] [PATCH] drm/i915/sprite: Always enable the scaler on IronLake

[Jesse Barnes]

On Mon, 16 Apr 2012 16:14:25 +0100
Chris Wilson <chris at chris-wilson.co.uk> wrote:

> On Mon, 16 Apr 2012 08:00:17 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> > Funky... do you notice any difference in output (fuzziness or anything)
> > with the scaler enabled?  Wonder if it's safe to enable for 1:1 scaling
> > on SNB as well.
> 
> Hard to tell. Without the scaler being enabled, it is black. :)

Oh I misread "doesn't update" as "you only see the first frame".  I
guess we can compare on SNB at least and potentially enable the scaler
all the time.

> The content I was using was regular compressed mpeg-ts, so not the best
> for juding aesthetic quality. We'd have to render some test geometry,
> such as testdisplay to be sure. (testplanes? ;)

There's a sprite_on test in there somewhere, but a new testplanes
wouldn't hurt.

> The docs do suggest that we disable the scaler to save power, so
> probably best if we keep the code as is.

Ok, sounds good.

-- 
Jesse Barnes, Intel Open Source Technology Center