summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStanimir Varbanov <stanimir.varbanov@linaro.org>2017-06-15 13:31:55 -0300
committerMauro Carvalho Chehab <mchehab@s-opensource.com>2017-06-20 08:53:40 -0300
commit832d467115787472140fb254f24f17398aed8d39 (patch)
tree77387829b51c4462d2195043e2e3a81d010d3352
parent4cb3548a87c4a3650ab9c4587abfcf5fe6fa7542 (diff)
[media] media: venus: hfi_msgs: fix set but not used variables
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>
-rw-r--r--drivers/media/platform/qcom/venus/hfi_msgs.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/media/platform/qcom/venus/hfi_msgs.c b/drivers/media/platform/qcom/venus/hfi_msgs.c
index 88898118f6af..f8841713e417 100644
--- a/drivers/media/platform/qcom/venus/hfi_msgs.c
+++ b/drivers/media/platform/qcom/venus/hfi_msgs.c
@@ -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;