On 11/03/15 11:16, Junghak Sung wrote:
> Move things related with vb2 file I/O and vb2_thread without doing any
> functional changes. After that, videobuf2-internal.h is removed because
> it is not necessary any more.
> 
> Signed-off-by: Junghak Sung <jh1009.s...@samsung.com>
> Signed-off-by: Geunyoung Kim <nenggun....@samsung.com>
> Acked-by: Seung-Woo Kim <sw0312....@samsung.com>
> Acked-by: Inki Dae <inki....@samsung.com>

Acked-by: Hans Verkuil <hans.verk...@cisco.com>

Nice to see that videobuf2-internal.h has now disappeared!

One note (something for a separate patch):

> ---
>  drivers/media/v4l2-core/videobuf2-core.c     |  777 
> +++++++++++++++++++++++++-
>  drivers/media/v4l2-core/videobuf2-internal.h |  161 ------
>  drivers/media/v4l2-core/videobuf2-v4l2.c     |  630 +--------------------
>  include/media/videobuf2-core.h               |   43 ++
>  include/media/videobuf2-v4l2.h               |   38 +-
>  5 files changed, 824 insertions(+), 825 deletions(-)
>  delete mode 100644 drivers/media/v4l2-core/videobuf2-internal.h
> 
> diff --git a/drivers/media/v4l2-core/videobuf2-core.c 
> b/drivers/media/v4l2-core/videobuf2-core.c
> index 33bdd81..f62c548 100644
> --- a/drivers/media/v4l2-core/videobuf2-core.c
> +++ b/drivers/media/v4l2-core/videobuf2-core.c

<snip>

> +     q->threadio = NULL;
> +     return err;
> +}
> +EXPORT_SYMBOL_GPL(vb2_thread_stop);
> +
>  MODULE_DESCRIPTION("Driver helper framework for Video for Linux 2");

This description should be updated as it is no longer a v4l2 framework but a
more generic media framework.

>  MODULE_AUTHOR("Pawel Osciak <pa...@osciak.com>, Marek Szyprowski");
>  MODULE_LICENSE("GPL");

Regards,

        Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to