From: Xiang, Haihao <haihao.xiang@intel.com>
Date: Fri, 15 Jul 2011 08:33:41 +0000 (+0800)
Subject: i965_drv_video: check the internal format of a surface before rendering
X-Git-Tag: 1.0_branch~140^2~19
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=93c00e667e11d04e8ace0289cd5224f3fae57af9;p=profile%2Fivi%2Fvaapi-intel-driver.git

i965_drv_video: check the internal format of a surface before rendering

Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
---

diff --git a/src/i965_render.c b/src/i965_render.c
index ad8de1e..b45edfc 100644
--- a/src/i965_render.c
+++ b/src/i965_render.c
@@ -975,26 +975,24 @@ i965_render_upload_vertex(
 }
 
 static void
-i965_render_upload_constants(VADriverContextP ctx)
+i965_render_upload_constants(VADriverContextP ctx,
+                             VASurfaceID surface)
 {
     struct i965_driver_data *i965 = i965_driver_data(ctx);
     struct i965_render_state *render_state = &i965->render_state;
     unsigned short *constant_buffer;
-
-    if (render_state->curbe.upload)
-        return;
+    struct object_surface *obj_surface = SURFACE(surface);
 
     dri_bo_map(render_state->curbe.bo, 1);
     assert(render_state->curbe.bo->virtual);
     constant_buffer = render_state->curbe.bo->virtual;
 
-    if (render_state->interleaved_uv)
+    if (obj_surface->fourcc == VA_FOURCC('N','V','1','2'))
         *constant_buffer = 1;
     else
         *constant_buffer = 0;
 
     dri_bo_unmap(render_state->curbe.bo);
-    render_state->curbe.upload = 1;
 }
 
 static void
@@ -1014,7 +1012,7 @@ i965_surface_render_state_setup(
     i965_render_cc_viewport(ctx);
     i965_render_cc_unit(ctx);
     i965_render_upload_vertex(ctx, surface, src_rect, dst_rect);
-    i965_render_upload_constants(ctx);
+    i965_render_upload_constants(ctx, surface);
 }
 static void
 i965_subpic_render_state_setup(
@@ -1696,7 +1694,7 @@ gen6_render_setup_states(
     gen6_render_color_calc_state(ctx);
     gen6_render_blend_state(ctx);
     gen6_render_depth_stencil_state(ctx);
-    i965_render_upload_constants(ctx);
+    i965_render_upload_constants(ctx, surface);
     i965_render_upload_vertex(ctx, surface, src_rect, dst_rect);
 }
 
@@ -2288,7 +2286,7 @@ gen7_render_setup_states(
     gen7_render_color_calc_state(ctx);
     gen7_render_blend_state(ctx);
     gen7_render_depth_stencil_state(ctx);
-    i965_render_upload_constants(ctx);
+    i965_render_upload_constants(ctx, surface);
     i965_render_upload_vertex(ctx, surface, src_rect, dst_rect);
 }
 
@@ -2984,7 +2982,6 @@ i965_render_init(VADriverContextP ctx)
                       "constant buffer",
                       4096, 64);
     assert(render_state->curbe.bo);
-    render_state->curbe.upload = 0;
 
     return True;
 }
diff --git a/src/i965_render.h b/src/i965_render.h
index a5034d9..96a1512 100644
--- a/src/i965_render.h
+++ b/src/i965_render.h
@@ -67,7 +67,6 @@ struct i965_render_state
 
     struct {
         dri_bo *bo;
-        int upload;
     } curbe;
 
     unsigned short interleaved_uv;