Received-SPF: pass (google.com: domain of linux-kernel+bounces-52138-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
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,
	eagle.zhou@nxp.com,
	tao.jiang_2@nxp.com,
	ming.zhou@nxp.com,
	ming.qian@oss.nxp.com,
	linux-imx@nxp.com,
	linux-media@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: [PATCH] media: imx-jpeg: Support for negotiating bytesperline with client
Date: Mon,  5 Feb 2024 16:25:56 +0900
Message-ID: <20240205072556.804809-1-ming.qian@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?z82RBD2iR4iHgI4U5CnctB/qQNf3XxRUEb4nCSAk7HHWu4MU48+TBAgx/5Or?=
 =?us-ascii?Q?GpOb2gV+BkxxbVXRzJMEwNTaR/cfghi9a3FgbyD0CYrKmAFVgrDgliJDIOso?=
 =?us-ascii?Q?KOdGM5Uz58m9Mjk4XkwAasG+cqnChPIgQz8LTVyfdBbjRyqcEa59Ouz3Iv5j?=
 =?us-ascii?Q?WNShkHyG7bJjlbhwAPKjUe8Lmk3i6E35o4cIr/TkWQuciT830ICuvMlB2Cyr?=
 =?us-ascii?Q?du51gjwcK6461nJGaZu+MaxYlzSP4AfB8Zt2bPvZn37N8gQk1gY6ZUUcSie8?=
 =?us-ascii?Q?cet7bxh1nwPqFOcz5Hm1vTa7DpU2Xl6By3PVPZaQ4XakH+1q7+SsoZn40cg7?=
 =?us-ascii?Q?2nQDmRAnfU9cVb1Ga9YzTPaCdmo4t3acoQ49UCSG5zRGjKbP4ix29MdUC4Dd?=
 =?us-ascii?Q?U+XrGFiWObsLHBBnZfiRZetQYQ86ufFftq6tnCODv2IqiuHYmXucxjEhEx41?=
 =?us-ascii?Q?2Bu86b8H26ttk7Gx8CyACqkM5hgUO/zG8F4jEz6nV1TWgiPtAfc2ZAZ2dsNX?=
 =?us-ascii?Q?tXDeCsjp1bSTWa0nikrmWvME4jDJSKpKSFE00Lf8m4ZjsDG2YcadC2bdePy6?=
 =?us-ascii?Q?N1fprNHDmyemvNVXQY1LLwKCq3rIUUPpOI82TbFjkM+n9l1fs1N/BmaV1P2t?=
 =?us-ascii?Q?xMTmruNbjC4VF7K9AHzln6ah6nBtKNmH/Epc4YMQUOyCaxEEue2RTXmxBwfO?=
 =?us-ascii?Q?FakpXfc3PTxCn/r3IRZPyDJAe1MnDXDpoAf/SwDQPZ42ihXXyW2zBNDGeYHp?=
 =?us-ascii?Q?8x3wNIzDUVP9w4N/yAIf4NpeRbv54HHELPEF8v3RIio3LjKa/zxHj+WY1xoG?=
 =?us-ascii?Q?R70phOblqnFL8o3MmyWSYUtbpSe1KGeDVoSUG8SbYrmcAVSOYC67eSdXg0E8?=
 =?us-ascii?Q?bFk+T0RtqHN4cfz1dzJI5CsObCv9R8Tf4r2qhlgDi4k8rjvHnNMQFAro9qp1?=
 =?us-ascii?Q?zRKF6uC15VnmRbzJCI7fZqCZUshwbFpWnP9GCCKUK8rY3V5sIX2+gVysdpGK?=
 =?us-ascii?Q?kt3gLVeD2mozdAjE9sCZRBO00kvgd8arbxMNnYPvg9XefXQKKjV0yQdUpwy1?=
 =?us-ascii?Q?z/gp5Hh6c5QbFih5h+l11WlavAArx3P4z1ZvliGnkLTc988Y42KgFgpibm/U?=
 =?us-ascii?Q?1BAL8fQyXOZiv0rYefqybrm5pST0IUEIxL2ZihmIiH2R46opmry0a4JbYrmZ?=
 =?us-ascii?Q?AgBZCWujAGjY9reuOokwH7GkuhErVcsQqQMk+AMvrToZYYgkWRT7X+oUuY12?=
 =?us-ascii?Q?yZPnla9FvoCe1OKZJ1HQkVN4JI6tDqWk6HshuAH3Fiw6znUqTzNkKKuv3RtT?=
 =?us-ascii?Q?hB+qlA+r4n6ie/Nal3w0HuKzPzjJSjgLk2X7QhYCHMcNnwTy6Ay8/AdNb299?=
 =?us-ascii?Q?RkgjbgLt7NFfLjYZooICIiRNx3uaz+4aXsfyrQ5kAaliSYgHXIVDjOzykcwK?=
 =?us-ascii?Q?1n6RFMf14KTh+fsEKsCVdud6VOjYLzFTWQjnvKEmrT43JMdEsCSXQe1MZhtc?=
 =?us-ascii?Q?GOPIyb/uHZ0JFjz/Q/yIGb0QccsWeTv6C5oqtIGuBES9IZcMwMIMHfkFPzkx?=
 =?us-ascii?Q?cap+yV1c+LdVNucBVS8kkKvuJ/b5NbhHYhoaBMRV?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7908d6b0-9279-4a1f-74fc-08dc261bc491
X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB8254.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Feb 2024 07:26:28.5268
 (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: 1Gq7SOlHC1EVNEEZ2wZMd6QAobpMhD/h3FDnRjr5Bl8lG+Fa6fgMqFR6/rrIwxIxtQ8+lMNfxwVAkj5KcXlRww==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB7082

This mxc-jpeg driver doesn't allow the client to set the bytesperline,
but for some android cts case, it need to negotiate the bytesperline
between decoder and display, and fail the case if driver doesn't support
negotiating bytesperline

The jpegdec and jpegenc does support to set bytesperline which is
multiple of 2, and greater than the value calulated by driver.

Signed-off-by: Ming Qian <ming.qian@nxp.com>
---
 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c | 16 ++++++++++++++++
 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h |  1 +
 2 files changed, 17 insertions(+)

diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
index 64112b63298c..cc97790ed30f 100644
--- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
+++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
@@ -1373,6 +1373,8 @@ static bool mxc_jpeg_source_change(struct mxc_jpeg_ctx *ctx,
 		q_data_cap->crop.top = 0;
 		q_data_cap->crop.width = jpeg_src_buf->w;
 		q_data_cap->crop.height = jpeg_src_buf->h;
+		q_data_cap->bytesperline[0] = 0;
+		q_data_cap->bytesperline[1] = 0;
 
 		/*
 		 * align up the resolution for CAST IP,
@@ -1752,6 +1754,14 @@ static u32 mxc_jpeg_get_image_format(struct device *dev,
 
 static void mxc_jpeg_bytesperline(struct mxc_jpeg_q_data *q, u32 precision)
 {
+	u32 bytesperline[2];
+
+	bytesperline[0] = q->bytesperline[0];
+	bytesperline[1] = q->bytesperline[0];	/*imx-jpeg only support the same line pitch*/
+	v4l_bound_align_image(&bytesperline[0], 0, MXC_JPEG_MAX_LINE, 2,
+			      &bytesperline[1], 0, MXC_JPEG_MAX_LINE, 2,
+			      0);
+
 	/* Bytes distance between the leftmost pixels in two adjacent lines */
 	if (q->fmt->fourcc == V4L2_PIX_FMT_JPEG) {
 		/* bytesperline unused for compressed formats */
@@ -1775,6 +1785,12 @@ static void mxc_jpeg_bytesperline(struct mxc_jpeg_q_data *q, u32 precision)
 		q->bytesperline[0] = q->w_adjusted * DIV_ROUND_UP(precision, 8);
 		q->bytesperline[1] = 0;
 	}
+
+	if (q->fmt->fourcc != V4L2_PIX_FMT_JPEG) {
+		q->bytesperline[0] = max(q->bytesperline[0], bytesperline[0]);
+		if (q->fmt->mem_planes > 1)
+			q->bytesperline[1] = max(q->bytesperline[1], bytesperline[1]);
+	}
 }
 
 static void mxc_jpeg_sizeimage(struct mxc_jpeg_q_data *q)
diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h
index dc4afeeff5b6..86e324b21aed 100644
--- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h
+++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.h
@@ -22,6 +22,7 @@
 #define MXC_JPEG_MIN_HEIGHT		64
 #define MXC_JPEG_MAX_WIDTH		0x2000
 #define MXC_JPEG_MAX_HEIGHT		0x2000
+#define MXC_JPEG_MAX_LINE		0x8000
 #define MXC_JPEG_MAX_CFG_STREAM		0x1000
 #define MXC_JPEG_H_ALIGN		3
 #define MXC_JPEG_W_ALIGN		3
-- 
2.43.0-rc1