On Tue, Jun 23, 2020 at 12:37 PM Hans Verkuil <hverkuil-cisco@xxxxxxxxx> wrote: > > Hi Tomasz, > > On 26/05/2020 12:09, Hans Verkuil wrote: > > From: Tomasz Figa <tfiga@xxxxxxxxxxxx> > > > > Due to complexity of the video encoding process, the V4L2 drivers of > > stateful encoder hardware require specific sequences of V4L2 API calls > > to be followed. These include capability enumeration, initialization, > > encoding, encode parameters change, drain and reset. > > > > Specifics of the above have been discussed during Media Workshops at > > LinuxCon Europe 2012 in Barcelona and then later Embedded Linux > > Conference Europe 2014 in Düsseldorf. The de facto Codec API that > > originated at those events was later implemented by the drivers we already > > have merged in mainline, such as s5p-mfc or coda. > > > > The only thing missing was the real specification included as a part of > > Linux Media documentation. Fix it now and document the encoder part of > > the Codec API. > > > > Signed-off-by: Tomasz Figa <tfiga@xxxxxxxxxxxx> > > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> > > --- > > .../userspace-api/media/v4l/dev-encoder.rst | 728 ++++++++++++++++++ > > .../userspace-api/media/v4l/dev-mem2mem.rst | 1 + > > .../userspace-api/media/v4l/pixfmt-v4l2.rst | 5 + > > .../userspace-api/media/v4l/v4l2.rst | 2 + > > .../media/v4l/vidioc-encoder-cmd.rst | 51 +- > > 5 files changed, 767 insertions(+), 20 deletions(-) > > create mode 100644 Documentation/userspace-api/media/v4l/dev-encoder.rst > > > > diff --git a/Documentation/userspace-api/media/v4l/dev-encoder.rst b/Documentation/userspace-api/media/v4l/dev-encoder.rst > > new file mode 100644 > > index 000000000000..aace7b812a9c > > --- /dev/null > > +++ b/Documentation/userspace-api/media/v4l/dev-encoder.rst > > @@ -0,0 +1,728 @@ > > +.. SPDX-License-Identifier: GPL-2.0 > > + > > New rst docs should be dual licensed: > > SPDX-License-Identifier: GPL-2.0 OR GFDL-1.1-or-later-no-invariants > > It is OK to make this change? No need to repost, if you agree with this > I or Mauro will make the change. > > If you agree, just reply with your Signed-off-by line. Fine with me. Signed-off-by: Tomasz Figa <tfiga@xxxxxxxxxxxx> Best regards, Tomasz
