From: Paul Elder <paul.elder@ideasonboard.com>
To: Benjamin Mugnier <benjamin.mugnier@foss.st.com>
Cc: linux-media@vger.kernel.org, alain.volmat@foss.st.com,
hugues.fruchet@foss.st.com, sylvain.petinot@foss.st.com,
dave.stevenson@raspberrypi.com, sakari.ailus@linux.intel.com,
laurent.pinchart@ideasonboard.com,
kieran.bingham@ideasonboard.com, nicolas@ndufresne.ca,
hverkuil@xs4all.nl
Subject: Re: [PATCH v7 1/5] media: v4l: Add 1X16 16-bit greyscale media bus code definition
Date: Tue, 8 Nov 2022 18:50:22 +0900 [thread overview]
Message-ID: <Y2omXpfvxxEUgMTt@pyrite.rasen.tech> (raw)
In-Reply-To: <20221011121605.30260-1-benjamin.mugnier@foss.st.com>
On Tue, Oct 11, 2022 at 02:16:01PM +0200, Benjamin Mugnier wrote:
> This extends the greyscale media bus family originally from
> MEDIA_BUS_FMT_Y8_1X8 up to MEDIA_BUS_FMT_Y14_1X14 by adding
> MEDIA_BUS_FMT_Y16_1X16, and behaves the same way with 16 bits.
> Add its documentation in subdev-formats.rst
>
> Signed-off-by: Benjamin Mugnier <benjamin.mugnier@foss.st.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
> ---
> .../media/v4l/subdev-formats.rst | 37 +++++++++++++++++++
> include/uapi/linux/media-bus-format.h | 3 +-
> 2 files changed, 39 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/userspace-api/media/v4l/subdev-formats.rst b/Documentation/userspace-api/media/v4l/subdev-formats.rst
> index d21d532eee15..16ef3b41932e 100644
> --- a/Documentation/userspace-api/media/v4l/subdev-formats.rst
> +++ b/Documentation/userspace-api/media/v4l/subdev-formats.rst
> @@ -6057,6 +6057,43 @@ the following codes.
> - y\ :sub:`2`
> - y\ :sub:`1`
> - y\ :sub:`0`
> + * .. _MEDIA-BUS-FMT-Y16-1X16:
> +
> + - MEDIA_BUS_FMT_Y16_1X16
> + - 0x202e
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + -
> + - y\ :sub:`15`
> + - y\ :sub:`14`
> + - y\ :sub:`13`
> + - y\ :sub:`12`
> + - y\ :sub:`11`
> + - y\ :sub:`10`
> + - y\ :sub:`9`
> + - y\ :sub:`8`
> + - y\ :sub:`7`
> + - y\ :sub:`6`
> + - y\ :sub:`5`
> + - y\ :sub:`4`
> + - y\ :sub:`3`
> + - y\ :sub:`2`
> + - y\ :sub:`1`
> + - y\ :sub:`0`
> * .. _MEDIA-BUS-FMT-UYVY8-1X16:
>
> - MEDIA_BUS_FMT_UYVY8_1X16
> diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
> index ec3323dbb927..ca9a24c874da 100644
> --- a/include/uapi/linux/media-bus-format.h
> +++ b/include/uapi/linux/media-bus-format.h
> @@ -69,7 +69,7 @@
> #define MEDIA_BUS_FMT_RGB121212_1X36 0x1019
> #define MEDIA_BUS_FMT_RGB161616_1X48 0x101a
>
> -/* YUV (including grey) - next is 0x202e */
> +/* YUV (including grey) - next is 0x202f */
> #define MEDIA_BUS_FMT_Y8_1X8 0x2001
> #define MEDIA_BUS_FMT_UV8_1X8 0x2015
> #define MEDIA_BUS_FMT_UYVY8_1_5X8 0x2002
> @@ -92,6 +92,7 @@
> #define MEDIA_BUS_FMT_YUYV12_2X12 0x201e
> #define MEDIA_BUS_FMT_YVYU12_2X12 0x201f
> #define MEDIA_BUS_FMT_Y14_1X14 0x202d
> +#define MEDIA_BUS_FMT_Y16_1X16 0x202e
> #define MEDIA_BUS_FMT_UYVY8_1X16 0x200f
> #define MEDIA_BUS_FMT_VYUY8_1X16 0x2010
> #define MEDIA_BUS_FMT_YUYV8_1X16 0x2011
> --
> 2.25.1
>
next prev parent reply other threads:[~2022-11-08 9:50 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-11 12:05 [PATCH v7 0/5] media: Add ST VGXY61 camera sensor driver Benjamin Mugnier
2022-10-11 12:16 ` [PATCH v7 1/5] media: v4l: Add 1X16 16-bit greyscale media bus code definition Benjamin Mugnier
2022-10-11 12:16 ` [PATCH v7 2/5] media: v4l: ctrls: Add a control for HDR mode Benjamin Mugnier
2022-11-08 9:50 ` Paul Elder [this message]
2022-10-11 12:30 ` [PATCH v7 3/5] media: dt-bindings: Add ST VGXY61 camera sensor binding Benjamin Mugnier
2022-10-11 12:30 ` [PATCH v7 4/5] media: Documentation: Add ST VGXY61 driver documentation Benjamin Mugnier
2022-10-25 7:35 ` Sakari Ailus
2022-10-11 12:30 ` [PATCH v7 5/5] media: i2c: Add driver for ST VGXY61 camera sensor Benjamin Mugnier
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=Y2omXpfvxxEUgMTt@pyrite.rasen.tech \
--to=paul.elder@ideasonboard.com \
--cc=alain.volmat@foss.st.com \
--cc=benjamin.mugnier@foss.st.com \
--cc=dave.stevenson@raspberrypi.com \
--cc=hugues.fruchet@foss.st.com \
--cc=hverkuil@xs4all.nl \
--cc=kieran.bingham@ideasonboard.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=nicolas@ndufresne.ca \
--cc=sakari.ailus@linux.intel.com \
--cc=sylvain.petinot@foss.st.com \
/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