From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (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 BF56A402BB7; Thu, 11 Jun 2026 13:11:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781183463; cv=none; b=rAJNzydkP5OVNcqm6k0i7/Lo0yN5H4VHczhKD1CIrCDjuE/06pqTQeME+4Ks9QTDmXMMnzvaKM+zPyRVyTkQH+0YlMdQGrwKIGkKCtaH6puzPOslGTfwwgNrAVHaDR0c5Z97OpYBW/UyAbtrkGbjiQeXmxkbnUZCp4zJQwvykxY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781183463; c=relaxed/simple; bh=dKGInWVhf3/eAG/GM57TlIFJIUDuruotPgvubC4is1E=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=uf93z6UoOlLSIeKqVOoOya2QhpbKZSztrnJ3+UCnVyFC6oAGTEFW6tenGIOlYPrr5PjWqsDZ9ZthrQywgbthzSXfHeN9zG+P+UgshMdt9zT9umfuyGj71MIdZ8eWw/iG72bpwmUoCyskm5FnHMSbRLkXEThzdpj/KqWAzGDsTzI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=mKpuxxEB; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="mKpuxxEB" Received: from killaraus.ideasonboard.com (2001-14ba-70f3-e800--a06.rev.dnainternet.fi [IPv6:2001:14ba:70f3:e800::a06]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 34F29D0; Thu, 11 Jun 2026 15:10:30 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1781183430; bh=dKGInWVhf3/eAG/GM57TlIFJIUDuruotPgvubC4is1E=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mKpuxxEBqykpfu7t8V9UpjizqQB+COxMxqDksetJrn1Bg6O/4impTEiVMVT3C7ZZC qkl6L/naPnepF7G5IJ1JkVMPnF8RxMDHE1TXVQGzgaY7TEEPIs7q5/eFBm7ztiCdqS 6Mrja13/+8J8kek0htlabSrpAOp3FaMngyXQdpY4= Date: Thu, 11 Jun 2026 16:10:58 +0300 From: Laurent Pinchart To: Tomi Valkeinen Cc: Sakari Ailus , Dave Stevenson , Mauro Carvalho Chehab , Jacopo Mondi , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] media: imx219: Report streams using frame descriptors Message-ID: <20260611131058.GA1795343@killaraus.ideasonboard.com> References: <20260611-imx219-frame-desc-v1-1-fe7e975bca6e@ideasonboard.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: On Thu, Jun 11, 2026 at 04:06:38PM +0300, Tomi Valkeinen wrote: > On 11/06/2026 12:24, Sakari Ailus wrote: > > On Thu, Jun 11, 2026 at 12:13:02PM +0300, Tomi Valkeinen wrote: > >> From: Laurent Pinchart > >> > >> Implement the .get_frame_desc() subdev operation to report information > >> about streams to the connected CSI-2 receiver. This is required to let > >> the CSI-2 receiver driver know about virtual channels and data types for > >> each stream. > >> > >> Signed-off-by: Laurent Pinchart > >> Reviewed-by: Jacopo Mondi > >> [tomi.valkeinen: picked from "Generic line based metadata support, internal pads" series] > >> Signed-off-by: Tomi Valkeinen > >> --- > >> This patch that adds .get_frame_desc() support to imx219 driver has been > >> circulating for a few years, and is currently posted in "[PATCH v12 > >> 00/86] Generic line based metadata support, internal pads" series. > >> > >> However, as some bridge drivers require modern drivers that support > >> .get_frame_desc, specifically ds90ub960.c, let's pick the patch and > >> queue it separately from the huge metadata series. > > > > I've been recently working on > > . > > In other words, drivers that have a single stream don't need this. We could > > Thanks, I had missed that. I like the idea of a helper that does the > fallback mechanism. But I wonder about the need for dynamic alloc, which > complicates the series. In any case, we can drop this series and > continue the discussion on your series. Maybe we can merge the fallback implementation separately from the dynamic allocation if the latter requires more work ? I'm also not a fan of the dynamic allocation, at least in the way it's implemented in the proposed series. I'm wondering if we could build the frame descriptors at stream enable time and store it in state structures instead. -- Regards, Laurent Pinchart