Re: [PATCH 4/4] venus: helpers: drop setting of timestap invalid flag

[Alexandre Courbot]

On Wed, Jan 9, 2019 at 5:46 PM Stanimir Varbanov
<stanimir.varbanov@xxxxxxxxxx> wrote:
>
> The zero timestap is really a valid so not sure why I discarded it. Fix

s/timestap/timestamp, also in patch title.

> that mistake by drop the code which checks for zero timestamp.

s/drop/dropping


>
> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx>
> ---
>  drivers/media/platform/qcom/venus/helpers.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/drivers/media/platform/qcom/venus/helpers.c b/drivers/media/platform/qcom/venus/helpers.c
> index e436385bc5ab..5cad601d4c57 100644
> --- a/drivers/media/platform/qcom/venus/helpers.c
> +++ b/drivers/media/platform/qcom/venus/helpers.c
> @@ -439,9 +439,6 @@ session_process_buf(struct venus_inst *inst, struct vb2_v4l2_buffer *vbuf)
>         fdata.flags = 0;
>         fdata.clnt_data = vbuf->vb2_buf.index;
>
> -       if (!fdata.timestamp)
> -               fdata.flags |= HFI_BUFFERFLAG_TIMESTAMPINVALID;
> -
>         if (type == V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE) {
>                 fdata.buffer_type = HFI_BUFFER_INPUT;
>                 fdata.filled_len = vb2_get_plane_payload(vb, 0);
> --
> 2.17.1
>