From: Benjamin Gaignard <benjamin.gaignard@collabora.com>
To: mchehab@kernel.org, ezequiel@vanguardiasur.com.ar,
hverkuil-cisco@xs4all.nl
Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-rockchip@lists.infradead.org, kernel@collabora.com,
Benjamin Gaignard <benjamin.gaignard@collabora.com>
Subject: [PATCH v5 0/3] Enumerate all pixels formats
Date: Mon, 22 Jul 2024 17:05:20 +0200 [thread overview]
Message-ID: <20240722150523.149667-1-benjamin.gaignard@collabora.com> (raw)
The goal of this series is to help userspace applications, like Gstreamer
or Chromium, to categorize decoders and so avoid trying to use decoder
that can't do the require task because it won't support the needed pixel
format.
As example, in today implementation we need to simulate 10 bit header
control to know if the driver support 10 bits pixel formats. With this
new flag it will simpler for userspace applications know if driver
support 10 bit pixel formats and if it is supported by userspace.
An example of how it can be used in GStreamer to discover the
supported pixels formats for stateless decoders is available here:
https://gitlab.freedesktop.org/benjamin.gaignard1/gstreamer/-/commits/v4l2codecs_enum_all_supported_formats?ref_type=heads
v4l2-compliance test of this flag:
https://gitlab.collabora.com/benjamin.gaignard/v4l-utils/-/tree/enum_all_formats?ref_type=heads
changes in version 5:
- Reset the proposal to follow Hans's advices
- Add new flag to be used with index field.
- Make vicodec and visl test driver use the new flag
- Doing the same for Verisilicon driver.
changes in version 4:
- Explicitly document that the new flags are targeting mem2mem devices.
changes in version 3:
- Add a flag to inform userspace application that driver
as take care of the flag.
changes in version 2:
- Clarify documentation.
- Only keep V4L2_FMT_FLAG_ALL_FORMATS flag in ioctl.
Benjamin
Benjamin Gaignard (3):
media: videodev2: Add flag to unconditionnaly enumerate pixels formats
media: test-drivers: Use V4L2_FMT_FLAG_ENUM_ALL flag
media: verisilicon: Use V4L2_FMT_FLAG_ENUM_ALL flag
.../userspace-api/media/v4l/vidioc-enum-fmt.rst | 12 +++++++++++-
.../media/videodev2.h.rst.exceptions | 1 +
.../media/platform/verisilicon/hantro_v4l2.c | 17 ++++++++++++-----
.../media/test-drivers/vicodec/vicodec-core.c | 7 ++++---
drivers/media/test-drivers/visl/visl-video.c | 11 +++++++----
include/uapi/linux/videodev2.h | 3 +++
6 files changed, 38 insertions(+), 13 deletions(-)
--
2.43.0
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
next reply other threads:[~2024-07-22 15:05 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-22 15:05 Benjamin Gaignard [this message]
2024-07-22 15:05 ` [PATCH v5 1/3] media: videodev2: Add flag to unconditionnaly enumerate pixels formats Benjamin Gaignard
2024-07-23 9:00 ` Sebastian Fricke
2024-07-30 7:08 ` Hans Verkuil
2024-07-30 7:19 ` Hans Verkuil
2024-07-31 7:00 ` Benjamin Gaignard
2024-07-31 6:59 ` Benjamin Gaignard
2024-07-22 15:05 ` [PATCH v5 2/3] media: test-drivers: Use V4L2_FMT_FLAG_ENUM_ALL flag Benjamin Gaignard
2024-07-23 9:49 ` Sebastian Fricke
2024-07-30 7:15 ` Hans Verkuil
2024-07-30 7:13 ` Hans Verkuil
2024-07-31 7:05 ` Benjamin Gaignard
2024-07-22 15:05 ` [PATCH v5 3/3] media: verisilicon: " Benjamin Gaignard
2024-07-23 10:03 ` Sebastian Fricke
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240722150523.149667-1-benjamin.gaignard@collabora.com \
--to=benjamin.gaignard@collabora.com \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=hverkuil-cisco@xs4all.nl \
--cc=kernel@collabora.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mchehab@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox