Remove the redundant if () from gen8_pp_upload_constants
authorXiang, Haihao <haihao.xiang@intel.com>
Mon, 20 Jan 2014 03:13:11 +0000 (11:13 +0800)
committerXiang, Haihao <haihao.xiang@intel.com>
Thu, 27 Feb 2014 02:30:55 +0000 (10:30 +0800)
This fixed the issue reported by Klockwork

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

index 45d5561..bbcf1c7 100755 (executable)
@@ -5114,14 +5114,12 @@ static void
 gen8_pp_upload_constants(VADriverContextP ctx,
                          struct i965_post_processing_context *pp_context)
 {
-    struct i965_driver_data *i965 = i965_driver_data(ctx);
     unsigned char *constant_buffer;
     int param_size;
 
     assert(sizeof(struct gen7_pp_static_parameter) == 192);
 
-    if (IS_GEN8(i965->intel.device_id))
-        param_size = sizeof(struct gen7_pp_static_parameter);
+    param_size = sizeof(struct gen7_pp_static_parameter);
 
     dri_bo_map(pp_context->dynamic_state.bo, 1);
     assert(pp_context->dynamic_state.bo->virtual);