From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.16]) (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 C8E1A4A0137 for ; Mon, 14 Sep 2026 20:54:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.16 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789419247; cv=none; b=FteKmkRr7s/U+2W0WEizenjAwqGGKk/UbAX9xDFLP2+W2R+iVrfWGCY5kEJT1xBsTufcaF9D22MgE3hDiJdJsqDN02r+eNiBz3aDjVldLdHpF/4pEzTk6wLG9VKsWY+Il/OYOQHIJsDMMkgYOqPloFIFuO22OstUAHKUpHbAMcs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789419247; c=relaxed/simple; bh=jCLYR4WzEFyoc2fFViC1b0v7zKeYs86PT/6k7/Vtcw8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=hrvutVOVhoZhCu1luvo7LwRPYUD+6fsdXl32uTrZl5UJndr98gvMZ2trU3zjpkF6+7D211EOp42sUVn+gejsPoz5VaV08cvAsT1fRTo/mQRnJhDX+piEqcBJWOnjPwbFg4nWEvrPyKa6xSHW5BDH6pZbfr6b1HVNOrGUoLY2TCg= 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=NvqTWuV8; arc=none smtp.client-ip=198.175.65.16 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="NvqTWuV8" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1789419245; x=1820955245; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=jCLYR4WzEFyoc2fFViC1b0v7zKeYs86PT/6k7/Vtcw8=; b=NvqTWuV8FHq6igYxiLPU2xeqvIwIK6kE1HTOWyD+YgytU/xc4nnycChW eq7vz1J01R4oWF3tb6EfRT/fMExsEQlD8LbRu+OMnMYZyxYQ0VgeAWqK/ ahbw57EOnpivjItWQU85ZiNzu8Mn23eRuA+9fgEz+Wa9iE695etcarLWd zBttpk2tX9EYP/rhDzgk8kt5KFM6FoDe0ROLG0ax484YSUcqpcIl6xgBo uJ9EoEcYzJ1lwRYzLmeGVf3nn0DwwDfkUjInoBsZwTDHMncZxRQwNtpmS fkZwf2A1hgs0MHIjVMb2zTb2SeI3lKr/MPNqea67M9F6fffFLN0roNMfv A==; X-CSE-ConnectionGUID: 1F/nGO1sRfSqMJ55OIO3NA== X-CSE-MsgGUID: Dw+woIFdSR2/J4CJ5jMucQ== X-IronPort-AV: E=McAfee;i="6800,10657,11905"; a="89984125" X-IronPort-AV: E=Sophos;i="6.27,103,1787036400"; d="scan'208";a="89984125" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by orvoesa108.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 Sep 2026 13:53:59 -0700 X-CSE-ConnectionGUID: MxgBhgdOSue0CC32hIDo0w== X-CSE-MsgGUID: ROEEJ1J2Rt2dIeSoI+L57A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.27,103,1787036400"; d="scan'208";a="297760731" Received: from pgcooper-mobl3.ger.corp.intel.com (HELO kekkonen.fi.intel.com) ([10.245.245.173]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 14 Sep 2026 13:53:56 -0700 Received: from punajuuri.localdomain (unknown [192.168.240.130]) by kekkonen.fi.intel.com (Postfix) with ESMTP id 573CE121BBF; Mon, 14 Sep 2026 23:53:59 +0300 (EEST) Received: from sailus by punajuuri.localdomain with local (Exim 4.98.2) (envelope-from ) id 1x6Dgc-000000035JI-3WCN; Mon, 14 Sep 2026 23:53:58 +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 Subject: [PATCH 09/21] media: ipu6: Start streaming once all streams have started, stop when not Date: Mon, 14 Sep 2026 23:53:46 +0300 Message-ID: <20260914205358.735307-10-sakari.ailus@linux.intel.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260914205358.735307-1-sakari.ailus@linux.intel.com> References: <20260914205358.735307-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 For each virtual channel, start streaming once all buffer queues connected via enabled routes have been set streaming, similarly stop when the first queue is stopped. Signed-off-by: Sakari Ailus --- drivers/media/pci/intel/ipu6/ipu6-isys-csi2.c | 107 +++++++++++++++++- drivers/media/pci/intel/ipu6/ipu6-isys-csi2.h | 1 + .../media/pci/intel/ipu6/ipu6-isys-queue.c | 7 +- 3 files changed, 105 insertions(+), 10 deletions(-) diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.c b/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.c index f92577054d89..946a9fbfd6bf 100644 --- a/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.c +++ b/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include @@ -435,6 +436,89 @@ static int ipu7_isys_csi2_stream_enable(struct ipu6_isys_csi2 *csi2) return 0; } +static bool ipu6_isys_csi2_streaming_change(struct ipu6_isys_subdev *asd, + struct v4l2_subdev_state *state, + u32 pad, u8 *vc, bool enable) +{ + u32 streams_enabled = 0, nodes_streaming = 0; + struct v4l2_subdev_route *route, *this_route = NULL; + struct v4l2_mbus_frame_desc *desc __free(v4l2_subdev_free_frame_desc) = + NULL; + struct v4l2_mbus_frame_desc_entry *this_entry = NULL; + + for_each_active_route(&state->routing, this_route) + if (pad == this_route->source_pad) + break; + if (!this_route) { + dev_dbg(asd->sd.dev, "no route found for pad %u\n", pad); + return false; + } + + struct media_pad *remote_pad = + media_pad_remote_pad_unique(&asd->sd.entity.pads[this_route->sink_pad]); + struct v4l2_subdev *remote_sd = + media_entity_to_v4l2_subdev(remote_pad->entity); + desc = v4l2_subdev_get_frame_desc(remote_sd, remote_pad->index, + V4L2_MBUS_FRAME_DESC_TYPE_CSI2); + if (IS_ERR(desc)) + return PTR_ERR(desc); + + for (unsigned int i = 0; i < desc->num_entries; i++) { + if (desc->entry[i].stream == this_route->sink_stream) { + this_entry = &desc->entry[i]; + break; + } + } + if (!this_entry) { + dev_dbg(asd->sd.dev, + "no frame descriptor entry found for stream %u\n", + this_route->sink_stream); + return false; + } + + for_each_active_route(&state->routing, route) { + struct v4l2_mbus_frame_desc_entry *entry = NULL; + + for (unsigned int i = 0; i < desc->num_entries; i++) { + if (desc->entry[i].stream == this_entry->stream) { + entry = &desc->entry[i]; + break; + } + } + + if (entry->bus.csi2.vc != this_entry->bus.csi2.vc) + continue; + + struct media_pad *video_pad = + media_pad_remote_pad_unique(&asd->sd.entity.pads[route->source_pad]); + struct ipu6_isys_video *av = !video_pad ? NULL : + container_of_const(video_pad, + struct ipu6_isys_video, pad); + + if (!av) { + dev_dbg(asd->sd.dev, + "can't find source pad for \"%s\":%u\n", + asd->sd.entity.name, route->source_pad); + return false; + } + + streams_enabled++; + if (av->streaming || (enable && pad == route->source_pad)) + nodes_streaming++; + } + + *vc = this_entry->bus.csi2.vc; + + if (streams_enabled == nodes_streaming) { + dev_dbg(asd->sd.dev, + "changing streaming state to %s on \"%s\":%u\n", + str_enabled_disabled(enable), asd->sd.entity.name, pad); + return true; + } + + return false; +} + static int ipu6_isys_csi2_enable_streams(struct v4l2_subdev *sd, struct v4l2_subdev_state *state, u32 pad, u64 streams_mask) @@ -450,6 +534,7 @@ static int ipu6_isys_csi2_enable_streams(struct v4l2_subdev *sd, struct ipu6_isys_buffer_list bl; u64 sink_streams; int ret; + u8 vc; ret = ipu6_isys_buffer_list_get(av->stream, &bl); if (ret < 0) { @@ -459,6 +544,9 @@ static int ipu6_isys_csi2_enable_streams(struct v4l2_subdev *sd, list_add(&av->csi2_entry, &csi2->av_head); + if (!ipu6_isys_csi2_streaming_change(asd, state, pad, &vc, true)) + return 0; + ret = ipu6_isys_start_stream_firmware(av, &bl); if (ret) { dev_err(sd->dev, "start stream of firmware failed\n"); @@ -472,16 +560,20 @@ static int ipu6_isys_csi2_enable_streams(struct v4l2_subdev *sd, v4l2_subdev_state_xlate_streams(state, pad, CSI2_PAD_SINK, &streams_mask); - ret = IS_IPU7(isp) ? ipu7_isys_csi2_stream_enable(csi2) : - ipu6_isys_csi2_stream_enable(csi2); - if (ret) - goto err_stop_stream_firmware; + if (!csi2->streaming_vc) { + ret = IS_IPU7(isp) ? ipu7_isys_csi2_stream_enable(csi2) : + ipu6_isys_csi2_stream_enable(csi2); + if (ret) + goto err_stop_stream_firmware; + } ret = v4l2_subdev_enable_streams(remote_sd, remote_pad->index, sink_streams); if (ret) goto err_stop_stream_csi2; + csi2->streaming_vc |= BIT(vc); + return 0; err_stop_stream_csi2: @@ -514,6 +606,12 @@ static int ipu6_isys_csi2_disable_streams(struct v4l2_subdev *sd, struct ipu6_device *isp = asd->isys->adev->isp; struct v4l2_subdev *remote_sd; u64 sink_streams; + u8 vc; + + if (!ipu6_isys_csi2_streaming_change(asd, state, pad, &vc, false)) + goto out_del_csi2_entry; + + csi2->streaming_vc &= ~BIT(vc); sink_streams = v4l2_subdev_state_xlate_streams(state, pad, CSI2_PAD_SINK, @@ -533,6 +631,7 @@ static int ipu6_isys_csi2_disable_streams(struct v4l2_subdev *sd, ipu6_isys_close_streaming_firmware(av); +out_del_csi2_entry: list_del(&av->csi2_entry); return 0; diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.h b/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.h index 7c936154691e..a98a60e443c6 100644 --- a/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.h +++ b/drivers/media/pci/intel/ipu6/ipu6-isys-csi2.h @@ -49,6 +49,7 @@ struct ipu6_isys_csi2 { unsigned int port; u32 legacy_irq_mask; unsigned int phy_mode; + u32 streaming_vc; }; struct ipu6_isys_csi2_timing { diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c b/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c index 4754e95e4007..5eb6f17bf45b 100644 --- a/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c +++ b/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c @@ -533,14 +533,10 @@ static int start_streaming(struct vb2_queue *q, unsigned int count) ipu6_isys_configure_stream_watermark(av, source_pad->entity); ipu6_isys_update_stream_watermark(av, true); - if (stream->nr_streaming != stream->nr_queues) - goto out; - ret = ipu6_isys_stream_start(av); if (ret) goto out_stream_start; -out: mutex_unlock(&stream->mutex); return 0; @@ -574,8 +570,7 @@ static void stop_streaming(struct vb2_queue *q) ipu6_isys_update_stream_watermark(av, false); mutex_lock(&av->isys->stream_mutex); - if (stream->nr_streaming == stream->nr_queues) - ipu6_isys_video_set_streaming(av, 0); + ipu6_isys_video_set_streaming(av, 0); list_del(&aq->node); mutex_unlock(&av->isys->stream_mutex); -- 2.47.3