These always happen together; the extra atom just means another item to iterate through, flags to check, and a call through a function pointer.
Signed-off-by: Kenneth Graunke <kenn...@whitecape.org> --- src/mesa/drivers/dri/i965/brw_cc.c | 9 ++++++++- src/mesa/drivers/dri/i965/brw_state.h | 1 - src/mesa/drivers/dri/i965/brw_state_upload.c | 2 -- src/mesa/drivers/dri/i965/gen7_viewport_state.c | 19 ------------------- 4 files changed, 8 insertions(+), 23 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_cc.c b/src/mesa/drivers/dri/i965/brw_cc.c index da72995..86ab503 100644 --- a/src/mesa/drivers/dri/i965/brw_cc.c +++ b/src/mesa/drivers/dri/i965/brw_cc.c @@ -62,7 +62,14 @@ brw_upload_cc_vp(struct brw_context *brw) } } - brw->state.dirty.cache |= CACHE_NEW_CC_VP; + if (brw->gen >= 7) { + BEGIN_BATCH(2); + OUT_BATCH(_3DSTATE_VIEWPORT_STATE_POINTERS_CC << 16 | (2 - 2)); + OUT_BATCH(brw->cc.vp_offset); + ADVANCE_BATCH(); + } else { + brw->state.dirty.cache |= CACHE_NEW_CC_VP; + } } const struct brw_tracked_state brw_cc_vp = { diff --git a/src/mesa/drivers/dri/i965/brw_state.h b/src/mesa/drivers/dri/i965/brw_state.h index 209fab1..399347c 100644 --- a/src/mesa/drivers/dri/i965/brw_state.h +++ b/src/mesa/drivers/dri/i965/brw_state.h @@ -117,7 +117,6 @@ extern const struct brw_tracked_state gen6_vs_state; extern const struct brw_tracked_state gen6_wm_push_constants; extern const struct brw_tracked_state gen6_wm_state; extern const struct brw_tracked_state gen7_depthbuffer; -extern const struct brw_tracked_state gen7_cc_viewport_state_pointer; extern const struct brw_tracked_state gen7_clip_state; extern const struct brw_tracked_state gen7_disable_stages; extern const struct brw_tracked_state gen7_gs_push_constants; diff --git a/src/mesa/drivers/dri/i965/brw_state_upload.c b/src/mesa/drivers/dri/i965/brw_state_upload.c index 1235d49..6cc2770 100644 --- a/src/mesa/drivers/dri/i965/brw_state_upload.c +++ b/src/mesa/drivers/dri/i965/brw_state_upload.c @@ -190,7 +190,6 @@ static const struct brw_tracked_state *gen7_atoms[] = &brw_state_base_address, &brw_cc_vp, - &gen7_cc_viewport_state_pointer, /* must do after brw_cc_vp */ &gen7_sf_clip_viewport, &gen7_push_constant_space, @@ -265,7 +264,6 @@ static const struct brw_tracked_state *gen8_atoms[] = &gen8_state_base_address, &brw_cc_vp, - &gen7_cc_viewport_state_pointer, /* must do after brw_cc_vp */ &gen8_sf_clip_viewport, &gen7_push_constant_space, diff --git a/src/mesa/drivers/dri/i965/gen7_viewport_state.c b/src/mesa/drivers/dri/i965/gen7_viewport_state.c index 193ead7..01af044 100644 --- a/src/mesa/drivers/dri/i965/gen7_viewport_state.c +++ b/src/mesa/drivers/dri/i965/gen7_viewport_state.c @@ -99,22 +99,3 @@ const struct brw_tracked_state gen7_sf_clip_viewport = { }, .emit = gen7_upload_sf_clip_viewport, }; - -/* ----------------------------------------------------- */ - -static void upload_cc_viewport_state_pointer(struct brw_context *brw) -{ - BEGIN_BATCH(2); - OUT_BATCH(_3DSTATE_VIEWPORT_STATE_POINTERS_CC << 16 | (2 - 2)); - OUT_BATCH(brw->cc.vp_offset); - ADVANCE_BATCH(); -} - -const struct brw_tracked_state gen7_cc_viewport_state_pointer = { - .dirty = { - .mesa = 0, - .brw = BRW_NEW_BATCH, - .cache = CACHE_NEW_CC_VP - }, - .emit = upload_cc_viewport_state_pointer, -}; -- 2.1.3 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev