Hi Malathi, On Mon, Oct 22, 2018 at 9:06 PM Malathi Gottam <mgottam@xxxxxxxxxxxxxx> wrote: > > Max bitrate property is not supported for venus version 4xx. > Return unsupported from packetization layer. Handle it in > hfi_venus layer to exit gracefully to venc layer. > > Signed-off-by: Malathi Gottam <mgottam@xxxxxxxxxxxxxx> This seems to work, thanks! Tested-by: Alexandre Courbot <acourbot@xxxxxxxxxxxx> > --- > drivers/media/platform/qcom/venus/hfi_cmds.c | 2 +- > drivers/media/platform/qcom/venus/hfi_venus.c | 2 ++ > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/platform/qcom/venus/hfi_cmds.c b/drivers/media/platform/qcom/venus/hfi_cmds.c > index e8389d8..87a4414 100644 > --- a/drivers/media/platform/qcom/venus/hfi_cmds.c > +++ b/drivers/media/platform/qcom/venus/hfi_cmds.c > @@ -1215,7 +1215,7 @@ static int pkt_session_set_property_1x(struct hfi_session_set_property_pkt *pkt, > } > case HFI_PROPERTY_CONFIG_VENC_MAX_BITRATE: > /* not implemented on Venus 4xx */ > - break; > + return -ENOTSUPP; > default: > return pkt_session_set_property_3xx(pkt, cookie, ptype, pdata); > } > diff --git a/drivers/media/platform/qcom/venus/hfi_venus.c b/drivers/media/platform/qcom/venus/hfi_venus.c > index 1240855..9d086b9 100644 > --- a/drivers/media/platform/qcom/venus/hfi_venus.c > +++ b/drivers/media/platform/qcom/venus/hfi_venus.c > @@ -1355,6 +1355,8 @@ static int venus_session_set_property(struct venus_inst *inst, u32 ptype, > pkt = (struct hfi_session_set_property_pkt *)packet; > > ret = pkt_session_set_property(pkt, inst, ptype, pdata); > + if (ret == -ENOTSUPP) > + return 0; > if (ret) > return ret; > > -- > 1.9.1 >
