lkml.org 
[lkml]   [2018]   [Nov]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] media: videodev2: add V4L2_FMT_FLAG_NO_SOURCE_CHANGE
On Thu, Nov 29, 2018 at 1:01 AM Hans Verkuil <hverkuil@xs4all.nl> wrote:
>
> On 10/04/2018 03:37 PM, Maxime Jourdan wrote:
> > When a v4l2 driver exposes V4L2_EVENT_SOURCE_CHANGE, some (usually
> > OUTPUT) formats may not be able to trigger this event.
> >
> > Add a enum_fmt format flag to tag those specific formats.
>
> I think I missed (or forgot) some discussion about this since I have no
> idea why this flag is needed. What's the use-case?

As far as I remember, the hardware/firmware Maxime has been working
with can't handle resolution changes for some coded formats. Perhaps
we should explain that better in the commit message and documentation
of the flag, though. Maxime, could you refresh my memory with the
details?

Best regards,
Tomasz

>
> Regards,
>
> Hans
>
> >
> > Signed-off-by: Maxime Jourdan <mjourdan@baylibre.com>
> > ---
> > Documentation/media/uapi/v4l/vidioc-enum-fmt.rst | 5 +++++
> > include/uapi/linux/videodev2.h | 5 +++--
> > 2 files changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/media/uapi/v4l/vidioc-enum-fmt.rst b/Documentation/media/uapi/v4l/vidioc-enum-fmt.rst
> > index 019c513df217..e0040b36ac43 100644
> > --- a/Documentation/media/uapi/v4l/vidioc-enum-fmt.rst
> > +++ b/Documentation/media/uapi/v4l/vidioc-enum-fmt.rst
> > @@ -116,6 +116,11 @@ one until ``EINVAL`` is returned.
> > - This format is not native to the device but emulated through
> > software (usually libv4l2), where possible try to use a native
> > format instead for better performance.
> > + * - ``V4L2_FMT_FLAG_NO_SOURCE_CHANGE``
> > + - 0x0004
> > + - The event ``V4L2_EVENT_SOURCE_CHANGE`` is not supported
> > + for this format.
> > +
> >
> >
> > Return Value
> > diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
> > index 3a65951ca51e..a28acee1cb52 100644
> > --- a/include/uapi/linux/videodev2.h
> > +++ b/include/uapi/linux/videodev2.h
> > @@ -723,8 +723,9 @@ struct v4l2_fmtdesc {
> > __u32 reserved[4];
> > };
> >
> > -#define V4L2_FMT_FLAG_COMPRESSED 0x0001
> > -#define V4L2_FMT_FLAG_EMULATED 0x0002
> > +#define V4L2_FMT_FLAG_COMPRESSED 0x0001
> > +#define V4L2_FMT_FLAG_EMULATED 0x0002
> > +#define V4L2_FMT_FLAG_NO_SOURCE_CHANGE 0x0004
> >
> > /* Frame Size and frame rate enumeration */
> > /*
> >
>

\
 
 \ /
  Last update: 2018-11-29 20:35    [W:0.206 / U:0.684 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site