Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 832d4671 authored by Stanimir Varbanov's avatar Stanimir Varbanov Committed by Mauro Carvalho Chehab
Browse files

[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: default avatarStanimir Varbanov <stanimir.varbanov@linaro.org>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 4cb3548a

drivers/media/platform/qcom/venus/hfi_msgs.c

+1 −3
Original line number Diff line number Diff line
@@ -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;