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 at daenzer.net> wrote: > 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 >
