On Tue, May 15, 2018 at 5:01 PM Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx> wrote: > > Make a new function vdec_output_conf() for decoder output > configuration. vdec_output_conf() will set properties via > HFI interface related to the output configuration, and > keep vdec_set_properties() which will set properties > related to decoding parameters. > > Signed-off-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx> > --- > drivers/media/platform/qcom/venus/vdec.c | 35 ++++++++++++++++++-------------- > 1 file changed, 20 insertions(+), 15 deletions(-) > > diff --git a/drivers/media/platform/qcom/venus/vdec.c b/drivers/media/platform/qcom/venus/vdec.c > index 5a5e3e2fece4..3a699af0ab58 100644 > --- a/drivers/media/platform/qcom/venus/vdec.c > +++ b/drivers/media/platform/qcom/venus/vdec.c > @@ -545,6 +545,23 @@ static const struct v4l2_ioctl_ops vdec_ioctl_ops = { > static int vdec_set_properties(struct venus_inst *inst) > { > struct vdec_controls *ctr = &inst->controls.dec; > + struct hfi_enable en = { .enable = 1 }; > + u32 ptype; > + int ret; > + > + if (ctr->post_loop_deb_mode) { > + ptype = HFI_PROPERTY_CONFIG_VDEC_POST_LOOP_DEBLOCKER; > + en.enable = 1; en.enable was already set to 1 in the definition. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html