From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 411AC472067 for ; Thu, 8 Oct 2026 08:55:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.17 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791449733; cv=none; b=SDTyRWXstRBj+BmQRudWq0XcPHFMNjJDAPeFak6DTtmzVwg+w0Ot4JE651MfjjJFyUAkXoiH3jNOg1NlCIfhdPNBGCnVmun71Yb7tpATcEzpzG7OFgSwy+opnW9opLbsJWVIEEjTTghRvhU1FgUHTUF4JsNJopE3fJCbYu8PRYc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791449733; c=relaxed/simple; bh=jv63r40vqTHsUfRse4WP6HrvB53IyipxqfunHG3jePY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=D3mT/bVPmleS7KVEfJEQgHLdqeo1wfPkBxQA3S16D3gp2X1OGLF/+jRzUGypAkmOz0HI+Rq5Bs3YHsmMhYRTl+g38G7QH6WtvF6AwSU3A94kGaXMNWvdK/UupO1GOwAazawDdLAzwM7SqSgungWH6HMQNW12jJIy6QorzPZHpXI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=jld3Xmxx; arc=none smtp.client-ip=198.175.65.17 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="jld3Xmxx" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1791449731; x=1822985731; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=jv63r40vqTHsUfRse4WP6HrvB53IyipxqfunHG3jePY=; b=jld3XmxxnCRWNGjvJPPjgh7ojSyzMbroaa4RjlvSlIB1CXcGspe9RG8b fQ9w4ojrEAp95DG6ZBlS1kT0yKWEEjIB6Q31fAoL41q4m2lgYkDiOZazJ fcj4JREkgS0q/Py05T3lEocrRZFqXcC0M1KbIp8NnkLEb1K3PZuIZ/YRE mZTTRvSlRQDj5RzUN7xCF86HmlRx94Ql6ML440ixGzVQz4PlVz0+Q8J13 siMiJ2g5njGGFfvy8CUL2BEaWas8f2pqqzRX/I/ITM04W6MZNjEPAYfqW g5qsJeLuZmzx6L/TVsI/XADL7ZO4EzVaSjgyGBmOfoM90Wxg0Ti4OKbAT Q==; X-CSE-ConnectionGUID: 7o4M019gSIWIWpI+Ziylmg== X-CSE-MsgGUID: xzuJPdYFTeqpITEyC23ayA== X-IronPort-AV: E=McAfee;i="6800,10657,11928"; a="233687" X-IronPort-AV: E=Sophos;i="6.27,146,1787036400"; d="scan'208";a="233687" Received: from fmviesa002.fm.intel.com ([10.60.135.142]) by orvoesa109.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Oct 2026 01:55:30 -0700 X-CSE-ConnectionGUID: cfYP6VyVS4aBeaEIgE/MLg== X-CSE-MsgGUID: 5OEoPCLNSLCaXzEiy4n07A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.27,146,1787036400"; d="scan'208";a="86385" Received: from smoticic-mobl1.ger.corp.intel.com (HELO kekkonen.fi.intel.com) ([10.245.245.95]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Oct 2026 01:55:28 -0700 Received: from punajuuri.localdomain (unknown [192.168.240.130]) by kekkonen.fi.intel.com (Postfix) with ESMTP id 5F9F8121BD4; Thu, 08 Oct 2026 11:55:25 +0300 (EEST) Received: from sailus by punajuuri.localdomain with local (Exim 4.98.2) (envelope-from ) id 1xEjrr-00000003Le8-3P56; Thu, 08 Oct 2026 11:52:47 +0300 Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo From: Sakari Ailus To: linux-media@vger.kernel.org Cc: "Yan, Dongcheng" , Mehdi Djait , "Yu, Ong Hock" , "Ng, Khai Wen" , Antti Laakso , "Bajpai, Manik" , Divyamani Tripathi , "Sapre, Sarang" , "Yao, Hao" Subject: [PATCH 5/5] media: ipu6: Return errors from stream stop and close Date: Thu, 8 Oct 2026 11:52:47 +0300 Message-ID: <20261008085247.798156-6-sakari.ailus@linux.intel.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20261008085247.798156-1-sakari.ailus@linux.intel.com> References: <20261008085247.798156-1-sakari.ailus@linux.intel.com> Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit If the firmware fails to start, stop or close a stream, either by telling about it with an error or due to a timeout, it probably requires a reset before being operational again. Mark isys needing a reset in such cases. Signed-off-by: Sakari Ailus --- .../media/pci/intel/ipu6/ipu6-isys-video.c | 55 ++++++++++++++----- .../media/pci/intel/ipu6/ipu6-isys-video.h | 4 +- 2 files changed, 43 insertions(+), 16 deletions(-) diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys-video.c b/drivers/media/pci/intel/ipu6/ipu6-isys-video.c index 44541aff79b3..799a4be3160b 100644 --- a/drivers/media/pci/intel/ipu6/ipu6-isys-video.c +++ b/drivers/media/pci/intel/ipu6/ipu6-isys-video.c @@ -543,25 +543,31 @@ int ipu6_isys_start_stream_firmware(struct ipu6_isys_stream *stream, out_stream_close: reinit_completion(&stream->stream_close_completion); + scoped_guard(mutex, &stream->isys->mutex) + stream->isys->need_reset = true; + retout = fw_ops->stream_close(stream->isys, stream->stream_handle); if (retout < 0) { dev_dbg(dev, "can't close stream (%d)\n", retout); - return retout; + return ret; } tout = wait_for_completion_timeout(&stream->stream_close_completion, IPU6_FW_CALL_TIMEOUT_JIFFIES); - if (!tout) + if (!tout) { dev_err(dev, "stream close time out\n"); - else if (stream->error) + ret = -ETIMEDOUT; + } else if (stream->error) { dev_err(dev, "stream close error: %d\n", stream->error); - else + ret = -EINVAL; + } else { dev_dbg(dev, "stream close complete\n"); + } return ret; } -void ipu6_isys_stop_stream_firmware(struct ipu6_isys_stream *stream) +int ipu6_isys_stop_stream_firmware(struct ipu6_isys_stream *stream) { struct ipu6_bus_device *adev = stream->asd->isys->adev; const struct ipu6_fw_isys_ops *fw_ops = adev->auxdrv_data->fw_ops; @@ -573,20 +579,30 @@ void ipu6_isys_stop_stream_firmware(struct ipu6_isys_stream *stream) ret = fw_ops->stream_flush(stream->isys, stream->stream_handle); if (ret < 0) { dev_err(dev, "can't stop stream (%d)\n", ret); - return; + return -EBUSY; } tout = wait_for_completion_timeout(&stream->stream_stop_completion, IPU6_FW_CALL_TIMEOUT_JIFFIES); - if (!tout) + if (!tout) { dev_warn(dev, "stream stop time out\n"); - else if (stream->error) + ret = -ETIMEDOUT; + } else if (stream->error) { dev_warn(dev, "stream stop error: %d\n", stream->error); - else + ret = -EINVAL; + } else{ dev_dbg(dev, "stop stream: complete\n"); + } + + if (ret) { + scoped_guard(mutex, &stream->isys->mutex) + stream->isys->need_reset = true; + } + + return ret; } -void ipu6_isys_close_stream_firmware(struct ipu6_isys_stream *stream) +int ipu6_isys_close_stream_firmware(struct ipu6_isys_stream *stream) { struct ipu6_bus_device *adev = stream->asd->isys->adev; const struct ipu6_fw_isys_ops *fw_ops = adev->auxdrv_data->fw_ops; @@ -599,22 +615,33 @@ void ipu6_isys_close_stream_firmware(struct ipu6_isys_stream *stream) ret = fw_ops->stream_close(stream->isys, stream->stream_handle); if (ret < 0) { dev_err(dev, "can't close stream (%d)\n", ret); - return; + goto out_cleanup; } tout = wait_for_completion_timeout(&stream->stream_close_completion, IPU6_FW_CALL_TIMEOUT_JIFFIES); - if (!tout) + if (!tout) { dev_warn(dev, "stream close time out\n"); - else if (stream->error) + ret = -ETIMEDOUT; + } else if (stream->error) { dev_warn(dev, "stream close error: %d\n", stream->error); - else + ret = -EINVAL; + } else { dev_dbg(dev, "close stream: complete\n"); + } +out_cleanup: scoped_guard(spinlock_irqsave, &stream->isys->streams_lock) { stream->isys->streams_by_handle[stream->stream_handle] = NULL; csi2->streams_by_vc[stream->vc] = NULL; } + + if (ret) { + scoped_guard(mutex, &stream->isys->mutex) + stream->isys->need_reset = true; + } + + return ret; } struct ipu6_isys_stream * diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys-video.h b/drivers/media/pci/intel/ipu6/ipu6-isys-video.h index 2821b9b1b943..2ab6f32716ad 100644 --- a/drivers/media/pci/intel/ipu6/ipu6-isys-video.h +++ b/drivers/media/pci/intel/ipu6/ipu6-isys-video.h @@ -99,8 +99,8 @@ int ipu6_isys_fw_pins_prepare(struct ipu6_isys_stream *stream, int ipu6_isys_start_stream_firmware(struct ipu6_isys_stream *stream, struct ipu6_isys_buffer_list *bl, struct v4l2_mbus_frame_desc *desc); -void ipu6_isys_stop_stream_firmware(struct ipu6_isys_stream *stream); -void ipu6_isys_close_stream_firmware(struct ipu6_isys_stream *stream); +int ipu6_isys_stop_stream_firmware(struct ipu6_isys_stream *stream); +int ipu6_isys_close_stream_firmware(struct ipu6_isys_stream *stream); struct ipu6_isys_stream * ipu6_isys_find_stream_firmware(struct ipu6_isys_csi2 *csi2, u8 vc); void ipu6_isys_free_stream_firmware(struct ipu6_isys_stream *stream); -- 2.47.3