drm/i915/gvt: Fix shift for cmd data size
authorZhenyu Wang <zhenyuw@linux.intel.com>
Thu, 27 Oct 2016 09:30:13 +0000 (17:30 +0800)
committerZhenyu Wang <zhenyuw@linux.intel.com>
Mon, 7 Nov 2016 06:16:56 +0000 (14:16 +0800)
This is to fix smatch warning on

     drivers/gpu/drm/i915/gvt/cmd_parser.c:1421 cmd_handler_mi_op_2f()
     warn: shift has higher precedence than mask

We need bits 20-19 mask for data size.

Cc: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
drivers/gpu/drm/i915/gvt/cmd_parser.c

index 0084ece8d8ffd17b24225e64a14384c9eec416bb..1238b75fe3f8966d021eb4081b1366029559c65a 100644 (file)
@@ -1418,8 +1418,8 @@ static int cmd_handler_mi_op_2e(struct parser_exec_state *s)
 static int cmd_handler_mi_op_2f(struct parser_exec_state *s)
 {
        int gmadr_bytes = s->vgpu->gvt->device_info.gmadr_bytes_in_cmd;
-       int op_size = ((1 << (cmd_val(s, 0) & GENMASK(20, 19) >> 19)) *
-                       sizeof(u32));
+       int op_size = (1 << ((cmd_val(s, 0) & GENMASK(20, 19)) >> 19)) *
+                       sizeof(u32);
        unsigned long gma, gma_high;
        int ret = 0;