Sorry, the autorship of the other patch changed when I moved the hunk there and didn't notice it. :( Marek On Mon, Nov 18, 2013 at 10:29 AM, Michel Dänzer <michel@xxxxxxxxxxx> wrote: > On Fre, 2013-11-15 at 18:55 +0100, Marek Olšák wrote: >> From: Michel Dänzer <michel.daenzer@xxxxxxx> >> >> Signed-off-by: Marek Olšák <marek.olsak@xxxxxxx> > > [...] > >> @@ -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@xxxxxxx> > > 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 > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
