diff options
author | Christopher James Halse Rogers <raof@ubuntu.com> | 2013-09-23 12:25:29 -0700 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2013-10-01 15:26:29 -0400 |
commit | f1dc677e79cd7a88d7379a934ebc7d87a3b18805 (patch) | |
tree | 63bacb465cbc76a698042b22a5be1047208f125a /src/evergreen_exa.c | |
parent | 67fb82a3f0759b171fea21b475a70fa825693570 (diff) |
EXA/evergreen: Paranoia around linear tiling. (v2)
The last two bytes of tiling_mode contain the actual tiling mode; the rest are
extra tiling configuration bits. These configuration bits are not necessarily
zero for a linear buffer, so mask them out before checking for linearity
v2: Also fix up evergreen_textured_videofuncs.c
agd5f: remove trailing whitespace
Signed-off-by: Christopher James Halse Rogers <raof@ubuntu.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'src/evergreen_exa.c')
-rw-r--r-- | src/evergreen_exa.c | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/src/evergreen_exa.c b/src/evergreen_exa.c index ccd102d5..abb5076f 100644 --- a/src/evergreen_exa.c +++ b/src/evergreen_exa.c @@ -156,7 +156,8 @@ EVERGREENPrepareSolid(PixmapPtr pPix, int alu, Pixel pm, Pixel fg) if (accel_state->planemask & 0xff000000) cb_conf.pmask |= 8; /* A */ cb_conf.rop = accel_state->rop; - if (accel_state->dst_obj.tiling_flags == 0) { + if ((accel_state->dst_obj.tiling_flags & RADEON_TILING_MASK) == + RADEON_TILING_LINEAR) { cb_conf.array_mode = 0; cb_conf.non_disp_tiling = 1; } @@ -335,7 +336,8 @@ EVERGREENDoPrepareCopy(ScrnInfoPtr pScrn) tex_res.base_level = 0; tex_res.last_level = 0; tex_res.perf_modulation = 0; - if (accel_state->src_obj[0].tiling_flags == 0) + if ((accel_state->src_obj[0].tiling_flags & RADEON_TILING_MASK) == + RADEON_TILING_LINEAR) tex_res.array_mode = 0; evergreen_set_tex_resource(pScrn, &tex_res, accel_state->src_obj[0].domain); @@ -378,7 +380,8 @@ EVERGREENDoPrepareCopy(ScrnInfoPtr pScrn) if (accel_state->planemask & 0xff000000) cb_conf.pmask |= 8; /* A */ cb_conf.rop = accel_state->rop; - if (accel_state->dst_obj.tiling_flags == 0) { + if ((accel_state->dst_obj.tiling_flags & RADEON_TILING_MASK) == + RADEON_TILING_LINEAR) { cb_conf.array_mode = 0; cb_conf.non_disp_tiling = 1; } @@ -1001,7 +1004,8 @@ static Bool EVERGREENTextureSetup(PicturePtr pPict, PixmapPtr pPix, tex_res.base_level = 0; tex_res.last_level = 0; tex_res.perf_modulation = 0; - if (accel_state->src_obj[unit].tiling_flags == 0) + if ((accel_state->src_obj[unit].tiling_flags & RADEON_TILING_MASK) == + RADEON_TILING_LINEAR) tex_res.array_mode = 0; evergreen_set_tex_resource (pScrn, &tex_res, accel_state->src_obj[unit].domain); @@ -1449,7 +1453,8 @@ static Bool EVERGREENPrepareComposite(int op, PicturePtr pSrcPicture, cb_conf.blendcntl |= CB_BLEND0_CONTROL__ENABLE_bit; cb_conf.rop = 3; cb_conf.pmask = 0xf; - if (accel_state->dst_obj.tiling_flags == 0) { + if ((accel_state->dst_obj.tiling_flags & RADEON_TILING_MASK) == + RADEON_TILING_LINEAR) { cb_conf.array_mode = 0; cb_conf.non_disp_tiling = 1; } |