Dikshita, I cannot find new version of this patchset with Han's comments addressed? On 3/14/22 12:09, quic_dikshita@xxxxxxxxxxx wrote: > From: Dikshita Agarwal <quic_dikshita@xxxxxxxxxxx> > > Hi, > > This series add a new intra-refresh type control for encoders. > this can be used to specify which intra refresh to be enabled, random, cyclic or none. > > Change since v0: > Dropped INTRA_REFRESH_TYPE_NONE as it was not needed. > Intra refresh period value as zero will disable the intra refresh. > > Change since v1: > Updated the control name for better undestanding. > Also updated the documentation accordingly. > > Change since v2: > Updated the venus driver implementation as well to use the correct control name. Missed in v2. > > Change since v3: > Addressed comments from Hans in v4l2 patch. > Enabled the support for cyclic intra refresh in venus driver. > > Thanks, > Dikshita > > Dikshita Agarwal (2): > media: v4l2-ctrls: Add intra-refresh type control > venus: venc: Add support for intra-refresh mode > > .../userspace-api/media/v4l/ext-ctrls-codec.rst | 22 ++++++++++++++++++++++ > drivers/media/platform/qcom/venus/core.h | 1 + > drivers/media/platform/qcom/venus/venc.c | 6 +++++- > drivers/media/platform/qcom/venus/venc_ctrls.c | 10 ++++++++++ > drivers/media/v4l2-core/v4l2-ctrls-defs.c | 9 +++++++++ > include/uapi/linux/v4l2-controls.h | 5 +++++ > 6 files changed, 52 insertions(+), 1 deletion(-) > -- regards, Stan
