Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
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, 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] media: imx-jpeg: Don't clear stop state in handling dynamic resolution change
Date:   Tue, 26 Jul 2022 17:04:27 +0800
Message-Id: <20220726090427.28093-1-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?Vdze6Fe51TDH+SyWzhCJ1qWjRVxbtfQrTsVx5oTBDBpbluNBM9ke3sowWfy8?=
 =?us-ascii?Q?RO+UQjI4Nsx4lotbhtVlsbmyK70qUynZ2/9T2i4X8uDL2QHmiRxeqL6zCE8G?=
 =?us-ascii?Q?azdX//dWyX9mjS9GNW9T2/UImp6v0U4UbWntr7W5M0QO53AuCwAhYGBwORx1?=
 =?us-ascii?Q?eyYl5xGVoKFdekcKgUBvu+DSeOvaijO4jk9xDn3bJcyqgvVdJUqsDpR6vIcw?=
 =?us-ascii?Q?Vdjz8Nk3TveZx3tR9fwSs2FXsdG9XrmXJHfpQjvgCIj7Zn6kTeDg6GOKyff2?=
 =?us-ascii?Q?JeNV5yvXKPz5phuVvdTaPNuLSKSRXISqZsJdNObXjkWvtwdbwt1f3XbxkrsX?=
 =?us-ascii?Q?1e6f8AU1O4vb6MpQRDbyUEGVVD+UhvJdY8nYgq49Nn8FpyMKSwoWBgQJOAIi?=
 =?us-ascii?Q?LHu9X0n6XRVeMb6jJTY7sWIiVBpqE9D+fN2fTO0RmnQzN97S63EgtrTSLpV+?=
 =?us-ascii?Q?XGp9qRXJQPScA63AYhx8arrHwL+fd1UTavgfWspfNAZ5szXZ9fPsBDy6rhuY?=
 =?us-ascii?Q?o0H+L0e1s/A3gW6Rb9Fmth6RLYLVdhaFLfTXP6ZdRGLJbIMXmbxEZgPuPErO?=
 =?us-ascii?Q?gxlOeIWLUI+ieK1TGAOokXNWfhpHvc39vqUo7eTqZOYN14jtU/ZFNjfLYwkt?=
 =?us-ascii?Q?C7I+S8dJy7I9hGgxil+TFITIz2rxI5A2xz2ZDgB5ikd8xJQ00KtxDoCjsLd3?=
 =?us-ascii?Q?utiR1pfjXR8lTCyJnZnVNAnwvVJ+MnzsfkiNcaLaBEgKNcKTw5Jt6Tuhz3FY?=
 =?us-ascii?Q?QQsItCinOjG2RO3gayauOzwB/jt1A7q8NJqdK6Zpihnrjv5pSS6nNp0tTnPb?=
 =?us-ascii?Q?joOfKalbZepfcrW3D157GhJQxiioRow+RiXa/bPYFo5Fd8RmfAZ9iJTWut0w?=
 =?us-ascii?Q?+nbs6I0Ovozc6aM3MGMvOBnlPujpRtjnQU6IZwwjNasiPAGkMRIJdBZTDUXH?=
 =?us-ascii?Q?7uNrV6waQDt0ZS7Bf3PWE4rUufeu8oVYvdJJpPZ1Ti20pnK8lbe/da/JPPER?=
 =?us-ascii?Q?hswfJpqp6/y5pPr8SZrxdVMYOJx1NlKpyFLhTFxNoIidmIh5jdAPXP0CxQkR?=
 =?us-ascii?Q?ig92wA3A/Fv4yos0mntQtgZptZN7EaVzW5vD2luFMvzJ5rEH28s+Dblw22L6?=
 =?us-ascii?Q?Be/iaAg3OkbSXw/V6tHfjU0pbDFBg4wJ1O+uBS/B2NCiqcD+5iOkk/jP1OQV?=
 =?us-ascii?Q?HtB3QLvd1ENvFu0TP4fJPAIxgno9woq9fv2YVSxKwaLqDh6mS8w3HY8tnr5G?=
 =?us-ascii?Q?XRgJz22765JLvVzEDrm1HguyJ6Zf32dLuJhiV+AnYCZ4FLtaaUp0rbwIu3q0?=
 =?us-ascii?Q?6gaA7L06a+cKdHjSAXdF1jv1E9269i3nRLKBSbG0QQ04fxkRrX+ACrMPjyPA?=
 =?us-ascii?Q?a87jqE7dfIll5sfPFpRKLZu7EpLCzPc3xEsg8SvC4jYdze/N5cSXIHAtKctw?=
 =?us-ascii?Q?SY4TAoILcQOe7Pybwf/AUbv2Z4dVhC+Fm8rGAkF4fLxYs2tzIYvIdODbIpLk?=
 =?us-ascii?Q?gtwDCCdi4gvryXNOVy+kEKR6DN7MmNxKz/x5UEyLuZtuNPGa8Hlm5exsfOfx?=
 =?us-ascii?Q?DCE0D/t3nJHEUHNFniW+MQ1WpJ9s4jGSA+XPcq0K?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1eadfed7-293d-43e0-dbca-08da6ee5e50b
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB6341.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2022 09:04:49.8328
 (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: il/m7bneQiHSFy1YUGdghbjk9u4lp7q31KUAJybdhK79IOQjHv+QoV3uqRr4PrXOOrb33uuJ71tzptssyZjMKA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR04MB4344
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

In dynamic resolution change, streamoff and streamon on the capture
queue may be called,
the V4L2_DEC_CMD_STOP cmd may be called before driver parsed the
jpeg header.
don't clear the stop state in streamoff of handling the
dynamic resolution change,
otherwise the drain may not complete.

Fixes: 4911c5acf9351 ("media: imx-jpeg: Implement drain using v4l2-mem2mem helpers")
Signed-off-by: Ming Qian <ming.qian@nxp.com>
---
 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
index fd5a65e577f4..ec13394bdddd 100644
--- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
+++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
@@ -1278,7 +1278,8 @@ static void mxc_jpeg_stop_streaming(struct vb2_queue *q)
 		v4l2_m2m_buf_done(vbuf, VB2_BUF_STATE_ERROR);
 	}
 
-	v4l2_m2m_update_stop_streaming_state(ctx->fh.m2m_ctx, q);
+	if (V4L2_TYPE_IS_OUTPUT(q->type) || !ctx->source_change)
+		v4l2_m2m_update_stop_streaming_state(ctx->fh.m2m_ctx, q);
 	if (V4L2_TYPE_IS_OUTPUT(q->type) &&
 	    v4l2_m2m_has_stopped(ctx->fh.m2m_ctx)) {
 		notify_eos(ctx);
-- 
2.36.1