Re: [PATCH 1/2] [media] v4l2-mem2mem: set the queue owner field just as vb2_ioctl_reqbufs does

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Philipp,

On 25 June 2015 at 11:01, Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote:
> Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>

Please add the patch description no matter how simple it is and how
well the subject covers the content of the patch.

Best wishes,
Kamil

> ---
>  drivers/media/v4l2-core/v4l2-mem2mem.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/media/v4l2-core/v4l2-mem2mem.c b/drivers/media/v4l2-core/v4l2-mem2mem.c
> index dc853e5..511caaa 100644
> --- a/drivers/media/v4l2-core/v4l2-mem2mem.c
> +++ b/drivers/media/v4l2-core/v4l2-mem2mem.c
> @@ -357,9 +357,16 @@ int v4l2_m2m_reqbufs(struct file *file, struct v4l2_m2m_ctx *m2m_ctx,
>                      struct v4l2_requestbuffers *reqbufs)
>  {
>         struct vb2_queue *vq;
> +       int ret;
>
>         vq = v4l2_m2m_get_vq(m2m_ctx, reqbufs->type);
> -       return vb2_reqbufs(vq, reqbufs);
> +       ret = vb2_reqbufs(vq, reqbufs);
> +       /* If count == 0, then the owner has released all buffers and he
> +          is no longer owner of the queue. Otherwise we have a new owner. */
> +       if (ret == 0)
> +               vq->owner = reqbufs->count ? file->private_data : NULL;
> +
> +       return ret;
>  }
>  EXPORT_SYMBOL_GPL(v4l2_m2m_reqbufs);
>
> --
> 2.1.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux