]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
[media] media: venus: hfi_msgs: fix set but not used variables
authorStanimir Varbanov <stanimir.varbanov@linaro.org>
Thu, 15 Jun 2017 16:31:55 +0000 (13:31 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 20 Jun 2017 11:53:40 +0000 (08:53 -0300)
This fixes a warning found when building with gcc7:

drivers/media/platform/qcom/venus/hfi_msgs.c:465:40:
warning: variable 'domain' set but not used [-Wunused-but-set-variable]
  u32 rem_bytes, num_props, codecs = 0, domain = 0;
                                        ^~~~~~
drivers/media/platform/qcom/venus/hfi_msgs.c:465:28:
warning: variable 'codecs' set but not used [-Wunused-but-set-variable]
  u32 rem_bytes, num_props, codecs = 0, domain = 0;

The warning is avoided by deleting the variables declaration.

Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/platform/qcom/venus/hfi_msgs.c

index 88898118f6af4c061468ebd90f3f8a0e99a2eeb3..f8841713e417268a1955f902d04519a47daf49fb 100644 (file)
@@ -462,7 +462,7 @@ static u32 init_done_read_prop(struct venus_core *core, struct venus_inst *inst,
                               struct hfi_msg_session_init_done_pkt *pkt)
 {
        struct device *dev = core->dev;
-       u32 rem_bytes, num_props, codecs = 0, domain = 0;
+       u32 rem_bytes, num_props;
        u32 ptype, next_offset = 0;
        u32 err;
        u8 *data;
@@ -490,8 +490,6 @@ static u32 init_done_read_prop(struct venus_core *core, struct venus_inst *inst,
                                (struct hfi_codec_mask_supported *)
                                (data + next_offset);
 
-                       codecs = masks->codecs;
-                       domain = masks->video_domains;
                        next_offset += sizeof(*masks);
                        num_props--;
                        break;