From:   Ming Qian <ming.qian@nxp.com>
To:     mchehab@kernel.org, mirela.rabulea@oss.nxp.com,
        hverkuil-cisco@xs4all.nl
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: [PATCH v2 04/10] media: Add YUV24_12 video format
Date:   Wed,  1 Feb 2023 14:02:26 +0800
Message-Id: <70b55bc46cd3cce59637d384013785c9efe444db.1675230665.git.ming.qian@nxp.com>
In-Reply-To: <cover.1675230665.git.ming.qian@nxp.com>
References: <cover.1675230665.git.ming.qian@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?DVFtSlXon7BudJKlGqjoQWHHZ4yg4094Mjf6oX+Wz0DoxdhY4Rj1qVevT30N?=
 =?us-ascii?Q?LU/wfMZLPNDa0j6WDJe3fSua1AzFFyzU5CIjE10VcSuCvD0Y2uVz7p8Nr/ci?=
 =?us-ascii?Q?plgriD8otyqpUV0BrGKAMJKC9I4UG8Q3Ka5D5S4NZhbOi+R0XJvfQEp6B6XT?=
 =?us-ascii?Q?hC2tEfVEFpApphuQxB/XxMrmHquPbOISVPovKQkt9k36RJyyUYwdzVb2tfxA?=
 =?us-ascii?Q?gn1OFSypsIHhimA5/l4z3THWJYS2qj5Pa0VUxlcybwcjIPdhrhikiRsFkWzv?=
 =?us-ascii?Q?ciArwVvm3hLstCY40DYdU+mxhiTuqoKS/TlV0bcUcyBxtiEEthtgYJhwS73H?=
 =?us-ascii?Q?gX93rVeswnROeSkDRylt5TRsS3cV3IvoqORkhHg2e8xAbVlmOguVuKQev2s2?=
 =?us-ascii?Q?it6WvSuiD2uH2tdHmN+K6zQu5NZF3zgIYA19OySNgK7C0MrRpXBnGnRYUfe0?=
 =?us-ascii?Q?Es+lx174Cp7N7n8WEXRz/efP0JRjuFsHyHDYpkZGq3ziqoENXzFPDRM5qcVW?=
 =?us-ascii?Q?pnOwgHt2NNFKUt2ho3tQ3nTAvmUSh9RIr5VJJoKJNMFCDe+H0H2rJhIwk7tZ?=
 =?us-ascii?Q?DFKmrcSbLhX9MOB45xc1ygOc73LLChAqT3+GIxfCqOjEecrmNeh9ycfiHfad?=
 =?us-ascii?Q?I6Hi0Kfxx6cY70niJjoewV4ovc1qQXM5H0M2qvLfhRPLhWd2MjKZB48h9qQP?=
 =?us-ascii?Q?l8y6Z67esheDCMS1O8xIlWdDxOSIu5YAwU27JWQsrpUYzIZv9vhl4jaMrT/x?=
 =?us-ascii?Q?jXSaKv6/tXBjOOhl5eo9Mq94k+nlDz6jSKM0kq7afPF/g0BMU7x/Uu7E+mNC?=
 =?us-ascii?Q?+GKvSAFM7qRvbG9tU/rUTtCCNoprIPH29KxISbfbJMJrnului1FKip8rUKo7?=
 =?us-ascii?Q?iOyIDkDoS1Xryk2WtvjQjWtZVL8CHA5623SioVvC0lzoZfdhGgDnjp+BSVR3?=
 =?us-ascii?Q?0bATLc5g6g4PZFvnGx1BWAOyYK/NtR15K32kbnIAz+qyRAOjW9vcSuNsRh+/?=
 =?us-ascii?Q?M1kyDy3y4zZ5y0bd9UNJ0EdF8o+8gcOM1vQs65nuJCzVFvygex+VcVf6D033?=
 =?us-ascii?Q?GbjW4VKhOHtLDj3gzGLJboYZBTtXnGs4FrMEC9eLI/chHtZHFcHJiFWtdcfA?=
 =?us-ascii?Q?dGNi8AZ4KBaM3Va7CQ2POgP1ryIpU4q30d3Gy/hLtYxN4UBsNDt4pHs1Uy03?=
 =?us-ascii?Q?T0JNKi3pDI4ZLz7P+0dc0xIctcupilsyPQsYHqQX9bCjWtp4wHLFpL5BTp5h?=
 =?us-ascii?Q?rf7Lt9A+SP0sEABKjApcysy0612rY8McUa/It62S2Gi53R/BC6UKLZRk+MoT?=
 =?us-ascii?Q?GrUI9Te0/btUstjeYwIrhc4yr/T7p5tHnwgDk4X9QF89yxhSpShmXSiCc69/?=
 =?us-ascii?Q?Uq1gm1ick5PpyDkMp1SEsfg/3W0I0GKjt5aRS8RzkRwti7P0sHGn6MXJC2wR?=
 =?us-ascii?Q?Nd+hrL2dm+hUIj6bi3BJevl3CIHnSRQ4jOSZapcRbyRCRS1i8RlnXX2mXX8+?=
 =?us-ascii?Q?t/CIEo5tZI4ggatPq7hJjKLfsYHz2I2N3ZJyQLzhtW9jL6QnGqiXWyea/565?=
 =?us-ascii?Q?nICn2H3h/FcD/7HzIdPhjuAfD8zPiSrjN96e0Y9z?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f716eb89-652e-4d30-ae88-08db041a0cdd
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB6341.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Feb 2023 06:03:33.6291
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: BX3RS5kM6o7LP/bf7JXnzK4SGYWSUFcUmD+Iyy6/uqnBP9oRTFbyQ60UkNq7aDPofYbvkdLqHFRbrBtE+GZspA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB8429
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

YUV24_12 is a YUV format with 12-bits per component like YUV24,
expanded to 16bits.
Data in the 12 high bits, zeros in the 4 low bits,
arranged in little endian order.

Signed-off-by: Ming Qian <ming.qian@nxp.com>
---
 .../media/v4l/pixfmt-packed-yuv.rst           | 28 +++++++++++++++++++
 drivers/media/v4l2-core/v4l2-common.c         |  1 +
 drivers/media/v4l2-core/v4l2-ioctl.c          |  1 +
 include/uapi/linux/videodev2.h                |  1 +
 4 files changed, 31 insertions(+)

diff --git a/Documentation/userspace-api/media/v4l/pixfmt-packed-yuv.rst b/Documentation/userspace-api/media/v4l/pixfmt-packed-yuv.rst
index bb7169b2cc8d..a098c5e8e609 100644
--- a/Documentation/userspace-api/media/v4l/pixfmt-packed-yuv.rst
+++ b/Documentation/userspace-api/media/v4l/pixfmt-packed-yuv.rst
@@ -257,6 +257,34 @@ the second byte and Y'\ :sub:`7-0` in the third byte.
     - The padding bits contain undefined values that must be ignored by all
       applications and drivers.
 
+The next lists the packed YUV 4:4:4 formats with more than 8 bits per component.
+expand the bits per component to 16 bits, data in the high bits, zeros in the low bits,
+arranged in little endian order. storing 1 pixels in 6 bytes.
+
+.. flat-table:: Packed YUV 4:4:4 Image Formats (more than 8bpc)
+    :header-rows: 1
+    :stub-columns: 0
+
+    * - Identifier
+      - Code
+      - Byte 1-0
+      - Byte 3-2
+      - Byte 5-4
+      - Byte 7-6
+      - Byte 9-8
+      - Byte 11-10
+
+    * .. _V4L2-PIX-FMT-YUV24-12:
+
+      - ``V4L2_PIX_FMT_YUV24_12``
+      - 'Y312'
+
+      - Y'\ :sub:`0`
+      - Cb\ :sub:`0`
+      - Cr\ :sub:`0`
+      - Y'\ :sub:`1`
+      - Cb\ :sub:`1`
+      - Cr\ :sub:`1`
 
 4:2:2 Subsampling
 =================
diff --git a/drivers/media/v4l2-core/v4l2-common.c b/drivers/media/v4l2-core/v4l2-common.c
index 3a882fb71227..b3ad02f8cf11 100644
--- a/drivers/media/v4l2-core/v4l2-common.c
+++ b/drivers/media/v4l2-core/v4l2-common.c
@@ -259,6 +259,7 @@ const struct v4l2_format_info *v4l2_format_info(u32 format)
 		{ .format = V4L2_PIX_FMT_UYVY,    .pixel_enc = V4L2_PIXEL_ENC_YUV, .mem_planes = 1, .comp_planes = 1, .bpp = { 2, 0, 0, 0 }, .hdiv = 2, .vdiv = 1 },
 		{ .format = V4L2_PIX_FMT_VYUY,    .pixel_enc = V4L2_PIXEL_ENC_YUV, .mem_planes = 1, .comp_planes = 1, .bpp = { 2, 0, 0, 0 }, .hdiv = 2, .vdiv = 1 },
 		{ .format = V4L2_PIX_FMT_YUYV_12, .pixel_enc = V4L2_PIXEL_ENC_YUV, .mem_planes = 1, .comp_planes = 1, .bpp = { 4, 0, 0, 0 }, .hdiv = 2, .vdiv = 1 },
+		{ .format = V4L2_PIX_FMT_YUV24_12, .pixel_enc = V4L2_PIXEL_ENC_YUV, .mem_planes = 1, .comp_planes = 1, .bpp = { 6, 0, 0, 0 }, .hdiv = 1, .vdiv = 1 },
 
 		/* YUV planar formats */
 		{ .format = V4L2_PIX_FMT_NV12,    .pixel_enc = V4L2_PIXEL_ENC_YUV, .mem_planes = 1, .comp_planes = 2, .bpp = { 1, 2, 0, 0 }, .hdiv = 2, .vdiv = 2 },
diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c b/drivers/media/v4l2-core/v4l2-ioctl.c
index 928acb9d13ec..711d1b0a8184 100644
--- a/drivers/media/v4l2-core/v4l2-ioctl.c
+++ b/drivers/media/v4l2-core/v4l2-ioctl.c
@@ -1343,6 +1343,7 @@ static void v4l_fill_fmtdesc(struct v4l2_fmtdesc *fmt)
 	case V4L2_PIX_FMT_HI240:	descr = "8-bit Dithered RGB (BTTV)"; break;
 	case V4L2_PIX_FMT_M420:		descr = "YUV 4:2:0 (M420)"; break;
 	case V4L2_PIX_FMT_YUYV_12:	descr = "12-bit Depth YUYV 4:2:2"; break;
+	case V4L2_PIX_FMT_YUV24_12:	descr = "12-bit Depth YUV 4:4:4"; break;
 	case V4L2_PIX_FMT_NV12:		descr = "Y/UV 4:2:0"; break;
 	case V4L2_PIX_FMT_NV21:		descr = "Y/VU 4:2:0"; break;
 	case V4L2_PIX_FMT_NV16:		descr = "Y/UV 4:2:2"; break;
diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
index 01fd233ff681..3eb188581b83 100644
--- a/include/uapi/linux/videodev2.h
+++ b/include/uapi/linux/videodev2.h
@@ -619,6 +619,7 @@ struct v4l2_pix_format {
 #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_YUYV_12     v4l2_fourcc('Y', '2', '1', '2') /* 32  YUYV 12-bit per component */
+#define V4L2_PIX_FMT_YUV24_12    v4l2_fourcc('Y', '3', '1', '2') /* 48  YUV 4:4:4 12-bit per component */
 
 /* two planes -- one Y, one Cr + Cb interleaved  */
 #define V4L2_PIX_FMT_NV12    v4l2_fourcc('N', 'V', '1', '2') /* 12  Y/CbCr 4:2:0  */
-- 
2.38.1