HACK: media: qcom: venus: avoid setting HEADER_MODE_SEPARATE

Using HEADER_MODE_SEPARATE causes output to be invalid.
10or-G · Jul 25, 2024 · ec4db89 · ec4db89
1 parent bfe0820
commit ec4db89
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/drivers/media/platform/qcom/venus/venc.c b/drivers/media/platform/qcom/venus/venc.c
@@ -847,6 +847,10 @@ static int venc_set_properties(struct venus_inst *inst)
 		else
 			en.enable = 1;
 
+		/* Seems to be broken */
+		if (IS_V3(inst->core) && is_fw_rev_or_older(inst->core, 4, 4, 60))
+			en.enable = 1;
+
 		ret = hfi_session_set_property(inst, ptype, &en);
 		if (ret)
 			return ret;

diff --git a/drivers/media/platform/qcom/venus/venc_ctrls.c b/drivers/media/platform/qcom/venus/venc_ctrls.c
@@ -218,6 +218,11 @@ static int venc_op_s_ctrl(struct v4l2_ctrl *ctrl)
 				en.enable = 0;
 			else
 				en.enable = 1;
+
+			/* Seems to be broken */
+			if (IS_V3(inst->core) && is_fw_rev_or_older(inst->core, 4, 4, 60))
+				en.enable = 1;
+
 			ptype = HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER;
 			ret = hfi_session_set_property(inst, ptype, &en);
 			if (ret) {