Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Mirela Rabulea <mirela.rabulea@nxp.com>
To:     mchehab@kernel.org, hverkuil-cisco@xs4all.nl,
        robert.chiras@nxp.com, laurentiu.palcu@nxp.com
Cc:     linux-imx@nxp.com, shawnguo@kernel.org, s.hauer@pengutronix.de,
        kernel@pengutronix.de, festevam@gmail.com,
        linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Subject: [PATCH] media: imx-jpeg: Remove soft reset between frames encoding
Date:   Mon, 27 Sep 2021 21:57:19 +0300
Message-Id: <20210927185719.3094796-1-mirela.rabulea@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?6iGpX0hJrcEliPi5b6ccNgKia2VWAgo22aDoLa4kqEf+ipHRvOyCMFcZQVGs?=
 =?us-ascii?Q?oUlJeulElkmrLvEVV9caZOToOGIqxf84RYgE1uOz1WkBu46qcQbc1nCdgmtg?=
 =?us-ascii?Q?MhqAA621kOp+6axvRTujfR1tBE/CwGDW3S6VCbToogNW9JL2+K32WQq6SIDI?=
 =?us-ascii?Q?TM/G4JHaqe2vhBlFFhzh2DTNYstIjcoRCYlcfz6+lfKNV9C4j5cvwFrqa0yP?=
 =?us-ascii?Q?cw0/4hHq7DU4/oDqelkLuVHFQaSxc37JreT8K2bjdOphMvdgRkUihKsK/SGN?=
 =?us-ascii?Q?NR4ZF2SOu8fnNxY9dKbs5E2aEiJrMQT9BMMVH16af+dGYzWRraoQ7JEHBGc/?=
 =?us-ascii?Q?NlC2TXc/5V+cGQ4c3JNkqnZmubemUKigA5ZJKwkVfz0PAWCiehCsrK9po1W/?=
 =?us-ascii?Q?GVNk/sEKDN8I1lni3OyDIrlrr+tKJj8NBDkSTFmm62Y23N+cSDRjbRTAxP3j?=
 =?us-ascii?Q?62vsZagH/Hm+LIWndpKUuvuSa+luvJskMB9Q/vGVdFAwC4s2UJOskaPYruQE?=
 =?us-ascii?Q?36Lz79S+S7vv0IgIbolMwm0l8n9gumyFf0p0NP0Q8zfuLNW/bh3O44XW6cBF?=
 =?us-ascii?Q?Xg2mxtKnJmKZQ2Bbrz1bH0mS9kgiF8RYLn4JPmktqxts2wTSgc4WuHwDqids?=
 =?us-ascii?Q?jhzpqVvnzpn8dd0vdr6xvu29vn+qyoHACgyAxX2MvFqDun+Djimd0SPrsSxi?=
 =?us-ascii?Q?w+oazNgNuk4K8zQruCNHcrzOXcav92SNkl+bv7U/Cwq5XRxqnCJ/YegTXEhM?=
 =?us-ascii?Q?dr1fJjGkIq1FbtexiDbWspdCd8lJCDhsgL+lalvDSu6E/ODuQPGSSzkB56Ne?=
 =?us-ascii?Q?OcQ1mwsD9fWkKA8HgYwY32O4LV3825+namRuqoRZPVatK9Xvhe2Op60IfwBX?=
 =?us-ascii?Q?IsyDZ1b00oXCzNh/kYYoR+suiVi66rXxgsz+ZJr+wgvVwbpDasKrL0fgJqum?=
 =?us-ascii?Q?M+yjirPkokW+IYMxyFCy4JNaKOSYEyVU2Nu4mCcCZlJ7sWMhjWwf3hdn3p+G?=
 =?us-ascii?Q?jHDs5uzBky+VCd/2GKBc/8m2UdQ5sk/D05WsoYWS1ruP/WGC8xzBsie2N0mo?=
 =?us-ascii?Q?zrSyxXYaAn9ULtEHiVd92N1YAkWqHDqrcb9opW+OqxuPUkaAAxhx4MsMeub9?=
 =?us-ascii?Q?0PH1FZ7qJ1CGMtxMaD2GCl3QwMzPEFpbsGUbaAEGkRRaDRF2J3IRMflaGOcu?=
 =?us-ascii?Q?4oFOTu9jQ7xCyXfDuX6xZHnLYq6q3x3qf4z8W10vglAiJUSlzkdLma+JtfVJ?=
 =?us-ascii?Q?qHqgOgozTe6Nv7BCYtbiQraHrSjExYopC6XDVyaNnw/UwLY+T1WdPI0Vw/zs?=
 =?us-ascii?Q?Z6AppfYNUB2sD90HlTV61Ymo?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 56c0912d-35e9-46d8-807f-08d981e8a841
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0401MB2495.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Sep 2021 18:57:30.6715
 (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: JbTr370VAtVXTlOwvXv6kL2uQEIOqsfeUQJH9AyFjci2DVSggZt14wTdmhpRYuTonVpDQmSmWbTTg8QTvCfDVQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB2287
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Remove soft reset between frames, which was merely a workaround,
actually replace it with something less aggressive: clearing of jpeg stream
buffer pointer, via STM_CTRL[BITBUF_PTR_CLR].
According to the reference manual, the BITBUF_PTR_CLR is performed on
context switch, but, per hardware team recommendation, it is also working
without context switch, which is the case for this driver, for now.

Without soft reset and without BITBUF_PTR_CLR, successive encoding jobs
continue from where the previous left off, basically writing past the
v4l2 buffer.

Signed-off-by: Mirela Rabulea <mirela.rabulea@nxp.com>
Reviewed-by: Robert Chiras <robert.chiras@nxp.com>
Reviewed-by: Laurentiu Palcu <laurentiu.palcu@oss.nxp.com>
---
 drivers/media/platform/imx-jpeg/mxc-jpeg.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/media/platform/imx-jpeg/mxc-jpeg.c b/drivers/media/platform/imx-jpeg/mxc-jpeg.c
index bd19a4b1bf08..f18b67453152 100644
--- a/drivers/media/platform/imx-jpeg/mxc-jpeg.c
+++ b/drivers/media/platform/imx-jpeg/mxc-jpeg.c
@@ -817,6 +817,7 @@ static void mxc_jpeg_config_dec_desc(struct vb2_buffer *out_buf,
 	img_fmt = mxc_jpeg_fourcc_to_imgfmt(q_data_cap->fmt->fourcc);
 	desc->stm_ctrl &= ~STM_CTRL_IMAGE_FORMAT(0xF); /* clear image format */
 	desc->stm_ctrl |= STM_CTRL_IMAGE_FORMAT(img_fmt);
+	desc->stm_ctrl |= STM_CTRL_BITBUF_PTR_CLR(1);
 	desc->line_pitch = q_data_cap->bytesperline[0];
 	mxc_jpeg_addrs(desc, dst_buf, src_buf, 0);
 	mxc_jpeg_set_bufsize(desc, ALIGN(vb2_plane_size(src_buf, 0), 1024));
@@ -843,6 +844,7 @@ static void mxc_jpeg_config_dec_desc(struct vb2_buffer *out_buf,
 	cfg_desc->imgsize |= MXC_JPEG_MIN_HEIGHT;
 	cfg_desc->line_pitch = MXC_JPEG_MIN_WIDTH * 2;
 	cfg_desc->stm_ctrl = STM_CTRL_IMAGE_FORMAT(MXC_JPEG_YUV422);
+	cfg_desc->stm_ctrl |= STM_CTRL_BITBUF_PTR_CLR(1);
 	cfg_desc->stm_bufbase = cfg_stream_handle;
 	cfg_desc->stm_bufsize = ALIGN(*cfg_size, 1024);
 	print_descriptor_info(jpeg->dev, cfg_desc);
@@ -886,6 +888,7 @@ static void mxc_jpeg_config_enc_desc(struct vb2_buffer *out_buf,
 	cfg_desc->stm_bufsize = 0x0;
 	cfg_desc->imgsize = 0;
 	cfg_desc->stm_ctrl = STM_CTRL_CONFIG_MOD(1);
+	cfg_desc->stm_ctrl |= STM_CTRL_BITBUF_PTR_CLR(1);
 
 	desc->next_descpt_ptr = 0; /* end of chain */
 
@@ -900,6 +903,7 @@ static void mxc_jpeg_config_enc_desc(struct vb2_buffer *out_buf,
 		dev_err(jpeg->dev, "No valid image format detected\n");
 	desc->stm_ctrl = STM_CTRL_CONFIG_MOD(0) |
 			 STM_CTRL_IMAGE_FORMAT(img_fmt);
+	desc->stm_ctrl |= STM_CTRL_BITBUF_PTR_CLR(1);
 	mxc_jpeg_addrs(desc, src_buf, dst_buf, 0);
 	dev_dbg(jpeg->dev, "cfg_desc:\n");
 	print_descriptor_info(jpeg->dev, cfg_desc);
@@ -955,11 +959,6 @@ static void mxc_jpeg_device_run(void *priv)
 		return;
 	}
 
-	/*
-	 * TODO: this reset should be removed, once we figure out
-	 * how to overcome hardware issues both on encoder and decoder
-	 */
-	mxc_jpeg_sw_reset(reg);
 	mxc_jpeg_enable(reg);
 	mxc_jpeg_set_l_endian(reg, 1);
 
-- 
2.25.1