From: Nicolas Dufresne <nicolas@ndufresne.ca> To: Hans Verkuil <hverkuil-cisco@xs4all.nl>, Ming Qian <ming.qian@nxp.com>, mchehab@kernel.org, mirela.rabulea@oss.nxp.com Cc: shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, xiahong.bao@nxp.com, linux-imx@nxp.com, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 03/10] media: Add Y212 video format Date: Wed, 22 Feb 2023 10:31:14 -0500 [thread overview] Message-ID: <605689aa181770ad6c7fca2a55967ab5fc5a3699.camel@ndufresne.ca> (raw) In-Reply-To: <93c8ae69-c765-f88e-23c3-9c6b8cf67b40@xs4all.nl> Hi Hans, Ming, Le vendredi 27 janvier 2023 à 15:37 +0100, Hans Verkuil a écrit : > On 20/12/2022 04:11, Ming Qian wrote: > > Y212 is a YUV format with 12-bits per component like YUYV, > > expanded to 16bits. > > Data in the 12 high bits, zeros in the 4 low bits, > > arranged in little endian order. > > > > [...] > > @@ -618,6 +618,7 @@ struct v4l2_pix_format { > > #define V4L2_PIX_FMT_YUVA32 v4l2_fourcc('Y', 'U', 'V', 'A') /* 32 YUVA-8-8-8-8 */ > > #define V4L2_PIX_FMT_YUVX32 v4l2_fourcc('Y', 'U', 'V', 'X') /* 32 YUVX-8-8-8-8 */ > > #define V4L2_PIX_FMT_M420 v4l2_fourcc('M', '4', '2', '0') /* 12 YUV 4:2:0 2 lines y, 1 line uv interleaved */ > > +#define V4L2_PIX_FMT_Y212 v4l2_fourcc('Y', '2', '1', '2') /* 32 YUYV 12-bit per component */ > > This definitely needs to be renamed to prevent repeating the 'P010' mistake. > > It's a YUYV format, so perhaps: _YUYV_12? > > You definitely need to have 'YUYV' in the name in order to deal with different > YUV orders. Hans, it seems you already accepted this exact same format from Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> / [PATCH v3 2/7] media: Add Y210, Y212 and Y216 formats. Tomi omitted to update v4l2-common.c file. Ming, do you mind keeping this part of your patch (adding Y210/Y216 support), this way we'd get that properly implemented. Nicolas p.s. We can alternatively change our mind as this is not final yet I think. [...]
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Dufresne <nicolas@ndufresne.ca> To: Hans Verkuil <hverkuil-cisco@xs4all.nl>, Ming Qian <ming.qian@nxp.com>, mchehab@kernel.org, mirela.rabulea@oss.nxp.com Cc: shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, xiahong.bao@nxp.com, linux-imx@nxp.com, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 03/10] media: Add Y212 video format Date: Wed, 22 Feb 2023 10:31:14 -0500 [thread overview] Message-ID: <605689aa181770ad6c7fca2a55967ab5fc5a3699.camel@ndufresne.ca> (raw) In-Reply-To: <93c8ae69-c765-f88e-23c3-9c6b8cf67b40@xs4all.nl> Hi Hans, Ming, Le vendredi 27 janvier 2023 à 15:37 +0100, Hans Verkuil a écrit : > On 20/12/2022 04:11, Ming Qian wrote: > > Y212 is a YUV format with 12-bits per component like YUYV, > > expanded to 16bits. > > Data in the 12 high bits, zeros in the 4 low bits, > > arranged in little endian order. > > > > [...] > > @@ -618,6 +618,7 @@ struct v4l2_pix_format { > > #define V4L2_PIX_FMT_YUVA32 v4l2_fourcc('Y', 'U', 'V', 'A') /* 32 YUVA-8-8-8-8 */ > > #define V4L2_PIX_FMT_YUVX32 v4l2_fourcc('Y', 'U', 'V', 'X') /* 32 YUVX-8-8-8-8 */ > > #define V4L2_PIX_FMT_M420 v4l2_fourcc('M', '4', '2', '0') /* 12 YUV 4:2:0 2 lines y, 1 line uv interleaved */ > > +#define V4L2_PIX_FMT_Y212 v4l2_fourcc('Y', '2', '1', '2') /* 32 YUYV 12-bit per component */ > > This definitely needs to be renamed to prevent repeating the 'P010' mistake. > > It's a YUYV format, so perhaps: _YUYV_12? > > You definitely need to have 'YUYV' in the name in order to deal with different > YUV orders. Hans, it seems you already accepted this exact same format from Tomi Valkeinen <tomi.valkeinen+renesas@ideasonboard.com> / [PATCH v3 2/7] media: Add Y210, Y212 and Y216 formats. Tomi omitted to update v4l2-common.c file. Ming, do you mind keeping this part of your patch (adding Y210/Y216 support), this way we'd get that properly implemented. Nicolas p.s. We can alternatively change our mind as this is not final yet I think. [...] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-22 15:31 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-12-20 3:11 [PATCH 00/10] media: imx-jpeg: Add support for 12 bit extended Ming Qian 2022-12-20 3:11 ` Ming Qian 2022-12-20 3:11 ` [PATCH 01/10] media: Add P012 and P012M video format Ming Qian 2022-12-20 3:11 ` Ming Qian 2023-01-27 14:31 ` Hans Verkuil 2023-01-27 14:31 ` Hans Verkuil 2023-01-30 8:15 ` [EXT] " Ming Qian 2023-01-30 8:15 ` Ming Qian 2023-02-10 15:08 ` Nicolas Dufresne 2023-02-10 15:08 ` Nicolas Dufresne 2023-02-13 9:16 ` Ming Qian 2023-02-13 9:16 ` Ming Qian 2022-12-20 3:11 ` [PATCH 02/10] media: Add Y012 " Ming Qian 2022-12-20 3:11 ` Ming Qian 2022-12-20 3:11 ` [PATCH 03/10] media: Add Y212 " Ming Qian 2022-12-20 3:11 ` Ming Qian 2023-01-27 14:37 ` Hans Verkuil 2023-01-27 14:37 ` Hans Verkuil 2023-02-22 15:31 ` Nicolas Dufresne [this message] 2023-02-22 15:31 ` Nicolas Dufresne 2023-02-23 1:35 ` [EXT] " Ming Qian 2023-02-23 1:35 ` Ming Qian 2022-12-20 3:11 ` [PATCH 04/10] media: Add Y312 " Ming Qian 2022-12-20 3:11 ` Ming Qian 2023-01-27 14:42 ` Hans Verkuil 2023-01-27 14:42 ` Hans Verkuil 2022-12-20 3:11 ` [PATCH 05/10] media: Add B312 " Ming Qian 2022-12-20 3:11 ` Ming Qian 2023-01-27 14:46 ` Hans Verkuil 2023-01-27 14:46 ` Hans Verkuil 2022-12-20 3:11 ` [PATCH 06/10] media: Add B412 " Ming Qian 2022-12-20 3:11 ` Ming Qian 2023-01-27 14:53 ` Hans Verkuil 2023-01-27 14:53 ` Hans Verkuil 2022-12-20 3:11 ` [PATCH 07/10] media: imx-jpeg: Refine the function mxc_jpeg_find_format Ming Qian 2022-12-20 3:11 ` Ming Qian 2022-12-20 3:11 ` [PATCH 08/10] media: imx-jpeg: Clear slot next desc ptr if config error Ming Qian 2022-12-20 3:11 ` Ming Qian 2022-12-20 3:11 ` [PATCH 09/10] media: imx-jpeg: Decoder add support for 12bit jpeg Ming Qian 2022-12-20 3:11 ` Ming Qian 2022-12-20 3:11 ` [PATCH 10/10] media: imx-jpeg: Encoder " Ming Qian 2022-12-20 3:11 ` Ming Qian 2022-12-22 21:47 ` [PATCH 00/10] media: imx-jpeg: Add support for 12 bit extended mirela.rabulea 2022-12-22 21:47 ` mirela.rabulea
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=605689aa181770ad6c7fca2a55967ab5fc5a3699.camel@ndufresne.ca \ --to=nicolas@ndufresne.ca \ --cc=devicetree@vger.kernel.org \ --cc=festevam@gmail.com \ --cc=hverkuil-cisco@xs4all.nl \ --cc=kernel@pengutronix.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-imx@nxp.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=mchehab@kernel.org \ --cc=ming.qian@nxp.com \ --cc=mirela.rabulea@oss.nxp.com \ --cc=s.hauer@pengutronix.de \ --cc=shawnguo@kernel.org \ --cc=xiahong.bao@nxp.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.