Re: [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

[Marcin Slusarz <marcin.slusarz@gmail.com>]

On Mon, May 02, 2011 at 03:11:00PM +0200, Daniel Vetter wrote:
> On Mon, May 2, 2011 at 2:56 PM, Benjamin Franzke
> <benjaminfranzke@googlemail.com> wrote:
> > I think in i915g the CURSOR flag should be used in i9x5_scanout_layout
> > for the "special case for cursors" as well, instead of only checking
> > only pt->width0 == 64 && pt->height0 == 64.
> 
> Oops, so much for actually re-checking the code. On the other hand, that
> part is broken, it needs a
>    tex->tiling = I915_TILE_NONE;
> and perhaps some check that width == height == 64 indeed holds. Then
> move it out as the first if clause and it'd start to make sense ...

Could you fix it in separate patch?

I couldn't even test it...

Marcin