On Fre, 2013-11-15 at 18:55 +0100, Marek Ol??k wrote:
> From: Michel D?nzer <michel.daenzer at amd.com>
> 
> Signed-off-by: Marek Ol??k <marek.olsak at amd.com>

[...]

> @@ -1657,10 +1659,7 @@ static int si_surface_init_2d(struct 
> radeon_surface_manager *surf_man,
>                  tile_mode = SI_TILE_MODE_COLOR_1D_SCANOUT;
>                  break;
>              case SI_TILE_MODE_DEPTH_STENCIL_2D:
> -                if (surf_man->family >= CHIP_BONAIRE)
> -                    tile_mode = CIK_TILE_MODE_DEPTH_STENCIL_1D;
> -                else
> -                    tile_mode = SI_TILE_MODE_DEPTH_STENCIL_1D;
> +                tile_mode = SI_TILE_MODE_DEPTH_STENCIL_1D;
>                  break;
>              default:
>                  return -EINVAL;

This hunk should be in patch 2.


Other than that, the series is

Reviewed-and-Tested-by: Michel D?nzer <michel.daenzer at amd.com>

However, patch 2 also depends on the corresponding kernel support, which
I just submitted.


Thanks for ironing out the kinks of this, Marek!


-- 
Earthling Michel D?nzer            |                  http://www.amd.com
Libre software enthusiast          |                Mesa and X developer

Reply via email to