Message ID | 741fd4cb-1c56-9546-36da-1993474caf49@xs4all.nl (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1jg3hR-00EjyV-7U; Tue, 02 Jun 2020 09:59:14 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726267AbgFBKDC (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Tue, 2 Jun 2020 06:03:02 -0400 Received: from lb2-smtp-cloud9.xs4all.net ([194.109.24.26]:57727 "EHLO lb2-smtp-cloud9.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726139AbgFBKDC (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 2 Jun 2020 06:03:02 -0400 Received: from cust-b5b5937f ([IPv6:fc0c:c16d:66b8:757f:c639:739b:9d66:799d]) by smtp-cloud9.xs4all.net with ESMTPA id g3l0jWljfCKzeg3l4jIjCB; Tue, 02 Jun 2020 12:03:00 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xs4all.nl; s=s1; t=1591092180; bh=aJj5kAnNbvdoWc+y8QdQsM9Lsb5y+OJNaY5Ima3ONew=; h=To:From:Subject:Message-ID:Date:MIME-Version:Content-Type:From: Subject; b=O1eYJl2ELKfTEgDFwOXprI63XnxuuYb6SVsjcwOYlol+tpD/HNjYR+RHX8LK7BDqq C5pju0NsAWBKNuN1qkzhXhtvn01tbFFBYJ2ewxmWVT7A3RgAkO+eJMeSy7QZlvPeFu 8kP6Yq9/Ge8Op9zKI50s/1CfIXRtzC1n2KJG+OZ1G7RbLe0XhxQxzW6MKH5c7P+r22 NNpyiIMbu72yxGxaFd8++VB/0obU7Q2gK1caPJ4oh5mqA1XPdy3RKBX3hzhPsHfT/H ew6kb7YQwq7pHq9BlKi4V87G7HVBm/NCnZZh8WNEz72vaIrw1zUN1bk30b/L1Ev4Sw WVkiag/OENvjw== To: Linux Media Mailing List <linux-media@vger.kernel.org> Cc: Michael Tretter <m.tretter@pengutronix.de>, Tomasz Figa <tfiga@chromium.org>, Nicolas Dufresne <nicolas@ndufresne.ca> From: Hans Verkuil <hverkuil-cisco@xs4all.nl> Subject: [GIT PULL FOR v5.9] Finalize stateful encoder support Message-ID: <741fd4cb-1c56-9546-36da-1993474caf49@xs4all.nl> Date: Tue, 2 Jun 2020 12:02:54 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4wfMTSnIyelNujJqCxYeuuWEc2VRbbGs4E/GLm0qV/1myHwTD/tv3MUvJyaaLz/IQDwvhoeo4VEF18n7yPbfCXsiZgh674QsRiXMLYSJA6xszR8C0LM6O0 9ZChTobk84kL2RsIf9RW7FtItqX4UCVqVhjOO0LRjxY2efzaZVlvMaTRqstWAp92UTa42cJ13eO7SkPjiOnCaSHt97ieyESyLMfIV9FQpEFI3EmCoSOE2nj3 tWfdiAoi3HZCceCXs98AtNcLfPe+uAF8V7XSWdeTk/Coj5lSILxkW+X48X9q4hUK Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.5 (--) X-LSpam-Report: No, score=-2.5 required=5.0 tests=BAYES_00=-1.9,DKIMWL_WL_MED=0.001,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
[GIT,PULL,FOR,v5.9] Finalize stateful encoder support
|
|
Pull-request
git://linuxtv.org/hverkuil/media_tree.git tags/br-stateful-encMessage
Hans Verkuil
June 2, 2020, 10:02 a.m. UTC
Hi Mauro, This PR finalizes the stateful encoder support by adding the stateful encoder spec. It was delayed quite a bit and the main reason was how framerates should be handled given the constraints of what existing stateful encoder drivers implemented. Michael, can you make a allegro patch that sets the V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL flag in that driver and have it behave according to this updated spec? Many thanks to all who contributed, with honorable mention of Tomasz who wrote the original version of this spec. Regards, Hans The following changes since commit 938b29db3aa9c293c7c1366b16e55e308f1a1ddd: media: Documentation: media: Refer to mbus format documentation from CSI-2 docs (2020-05-25 15:47:02 +0200) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/br-stateful-enc for you to fetch changes up to f73d82aca17259d7409c3bb675c1def0d71041e0: dev-encoder.rst: add reference to V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL (2020-06-02 11:05:51 +0200) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Hans Verkuil (4): vidioc-g-parm.rst: update the VIDIOC_G/S_PARM documentation dev-decoder.rst: small fixes videodev2.h: add V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL flag dev-encoder.rst: add reference to V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL Tomasz Figa (1): media: docs-rst: Document memory-to-memory video encoder interface Documentation/userspace-api/media/v4l/dev-decoder.rst | 6 +- Documentation/userspace-api/media/v4l/dev-encoder.rst | 729 ++++++++++++++++++++++++++++++++++++++++ Documentation/userspace-api/media/v4l/dev-mem2mem.rst | 1 + Documentation/userspace-api/media/v4l/pixfmt-v4l2.rst | 5 + Documentation/userspace-api/media/v4l/v4l2.rst | 2 + Documentation/userspace-api/media/v4l/vidioc-encoder-cmd.rst | 51 +-- Documentation/userspace-api/media/v4l/vidioc-enum-fmt.rst | 30 +- Documentation/userspace-api/media/v4l/vidioc-g-parm.rst | 51 +-- Documentation/userspace-api/media/videodev2.h.rst.exceptions | 1 + include/uapi/linux/videodev2.h | 1 + 10 files changed, 830 insertions(+), 47 deletions(-) create mode 100644 Documentation/userspace-api/media/v4l/dev-encoder.rst
Comments
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/patch/64176/
Build log: https://builder.linuxtv.org/job/patchwork/53633/
Build time: 00:23:59
Link: https://lore.kernel.org/linux-media/741fd4cb-1c56-9546-36da-1993474caf49@xs4all.nl
gpg: Signature made Tue 02 Jun 2020 09:53:49 AM UTC
gpg: using RSA key AAA7FFBA4D2D77EF4CAEA1421326E0CD23ABDCE5
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg: aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]
Summary: 2 patches and/or PDF generation with issues, being 0 at build time
Error/warnings:
Error #256 when running cat patches/0001-media-docs-rst-Document-memory-to-memory-video-encod.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict:
$ cat patches/0001-media-docs-rst-Document-memory-to-memory-video-encod.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:37: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
Em Tue, 2 Jun 2020 12:02:54 +0200 Hans Verkuil <hverkuil-cisco@xs4all.nl> escreveu: > Hi Mauro, > > This PR finalizes the stateful encoder support by adding the stateful encoder > spec. It was delayed quite a bit and the main reason was how framerates should > be handled given the constraints of what existing stateful encoder drivers > implemented. > > Michael, can you make a allegro patch that sets the V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL > flag in that driver and have it behave according to this updated spec? > > Many thanks to all who contributed, with honorable mention of Tomasz who wrote > the original version of this spec. > > Regards, > > Hans > > The following changes since commit 938b29db3aa9c293c7c1366b16e55e308f1a1ddd: > > media: Documentation: media: Refer to mbus format documentation from CSI-2 docs (2020-05-25 15:47:02 +0200) > > are available in the Git repository at: > > git://linuxtv.org/hverkuil/media_tree.git tags/br-stateful-enc > > for you to fetch changes up to f73d82aca17259d7409c3bb675c1def0d71041e0: > > dev-encoder.rst: add reference to V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL (2020-06-02 11:05:51 +0200) > > ---------------------------------------------------------------- > Tag branch > > ---------------------------------------------------------------- > Hans Verkuil (4): > vidioc-g-parm.rst: update the VIDIOC_G/S_PARM documentation > dev-decoder.rst: small fixes > videodev2.h: add V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL flag > dev-encoder.rst: add reference to V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL > > Tomasz Figa (1): > media: docs-rst: Document memory-to-memory video encoder interface > > Documentation/userspace-api/media/v4l/dev-decoder.rst | 6 +- > Documentation/userspace-api/media/v4l/dev-encoder.rst | 729 ++++++++++++++++++++++++++++++++++++++++ There's an issue here: 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 In order to be compatible with the license of the entire uAPI, we need to use here: .. SPDX-License-Identifier: GPL-2.0 OR GFDL-1.1-or-later-no-invariants E.g. dual GPLv2 and GFDL 1.1 or later. Thanks, Mauro