From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2E60CC3DA64 for ; Thu, 1 Aug 2024 16:10:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=q7FUd7e7Xsvhbr26ZIjAxaRKy5nBWeoI1ybNhZWY3lY=; b=L9/bIiaG6iWRMwI0IdJND41SQd tdZVC7E0zfpyHkQdnfLNupo3aI22YudOo0An9QmCOa5G/3Q2pGKVDEkI96dmJHD6bxQRnCB/c6fCp DroDZ24O++1L8epXnEvFHokdBhME2a4WLFCgrhxldaNTJZa08gAytkNsJTo6Y7Fx7ggz2IV4Wg24T c/6empyeQgs9X2lF/wNgwiaoWbGZJZh/9qhPygOkqeQCozk7LB/sCBk7cPSkswdOURHccozIUh34b dgp/75TpWOOvMkREIatNMTX5lH+wsIGq1PURKyjYWt8oQEZuTBqWj2T62QRAs9ip4Ms5ji/0RSqD+ UziS+8rQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sZYO8-00000005zqc-0vDY; Thu, 01 Aug 2024 16:10:48 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sZYK7-00000005ySD-2DyV for linux-arm-kernel@lists.infradead.org; Thu, 01 Aug 2024 16:06:40 +0000 Received: from pendragon.ideasonboard.com (81-175-209-231.bb.dnainternet.fi [81.175.209.231]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id C0819842; Thu, 1 Aug 2024 18:05:44 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1722528345; bh=DK9tWdz/0iSJkJT5x7sqs/bLGa54vj+NBpGI/VPp5gk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=m/BAis2FeetJ8Yd9EIMbkHlbRtKVOYWR6wQoO8lx5euCS0oPA+N3XzB9EUdTxwvUt 5ATESrc1Ab6ffMWPUn07zVB0XSsVOsc1A/Ww0CNuDaPEftsvNupdlyM27KXiHMX+Qh 0DOaka4qvdFJNF38MBi3oeeQYy59RvA/uLb0wIyY= Date: Thu, 1 Aug 2024 19:06:12 +0300 From: Laurent Pinchart To: yuji2.ishikawa@toshiba.co.jp Cc: hverkuil@xs4all.nl, mchehab@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, broonie@kernel.org, sakari.ailus@linux.intel.com, nobuhiro1.iwamatsu@toshiba.co.jp, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v11 5/6] documentation: media: add documentation for Toshiba Visconti Video Input Interface driver Message-ID: <20240801160612.GA18732@pendragon.ideasonboard.com> References: <20240709000848.1108788-1-yuji2.ishikawa@toshiba.co.jp> <20240709000848.1108788-6-yuji2.ishikawa@toshiba.co.jp> <20240722180251.GP13497@pendragon.ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240801_090639_757104_FBD8E9D8 X-CRM114-Status: GOOD ( 27.70 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Ishikawa-san, On Thu, Aug 01, 2024 at 09:23:43AM +0000, yuji2.ishikawa@toshiba.co.jp wrote: > On Tuesday, July 23, 2024 3:03 AM, Laurent Pinchart wrote: > > On Tue, Jul 09, 2024 at 09:08:47AM +0900, Yuji Ishikawa wrote: > > > Added description of Video Input Interface driver of Toshiba Visconti > > > architecture. > > > It includes hardware organization, structure of the driver and > > > metadata format for embedded image signal processor. > > > > > > Signed-off-by: Yuji Ishikawa > > > --- > > > Changelog v3: > > > - Newly add documentation to describe SW and HW > > > > > > Changelog v4: > > > - no change > > > > > > Changelog v5: > > > - no change > > > > > > Changelog v6: > > > - add description of CSI2RX subdevice > > > - add ordering of ioctl(S_FMT) and ioctl(S_EXT_CTRLS) > > > > > > Changelog v7: > > > - no change > > > > > > Changelog v8: > > > - add usage of V4L2_CTRL_TYPE_VISCONTI_ISP > > > > > > Changelog v9: > > > - fix warning: set reference target for keyword > > > V4L2_CTRL_TYPE_VISCONTI_ISP > > > > > > Changelog v10: > > > - use parameter buffers instead of compound control > > > - removed description of vendor specific compound control > > > - add description of parameter buffers for ISP control > > > - update directory structure > > > - remove documents under driver-api > > > - add documents to admin-guide, userspace-api > > > > > > Changelog v11: > > > - update usage of the driver > > > > > > .../admin-guide/media/v4l-drivers.rst | 1 + > > > .../admin-guide/media/visconti-viif.dot | 18 ++ > > > .../admin-guide/media/visconti-viif.rst | 255 ++++++++++++++++++ > > > .../userspace-api/media/v4l/meta-formats.rst | 1 + > > > .../media/v4l/metafmt-visconti-viif.rst | 48 ++++ > > > 5 files changed, 323 insertions(+) > > > create mode 100644 Documentation/admin-guide/media/visconti-viif.dot > > > create mode 100644 Documentation/admin-guide/media/visconti-viif.rst > > > create mode 100644 Documentation/userspace-api/media/v4l/metafmt-visconti-viif.rst [snip] > > > diff --git a/Documentation/admin-guide/media/visconti-viif.rst b/Documentation/admin-guide/media/visconti-viif.rst > > > new file mode 100644 > > > index 0000000000..4ef676754c > > > --- /dev/null > > > +++ b/Documentation/admin-guide/media/visconti-viif.rst > > > @@ -0,0 +1,255 @@ [snip] > > > +viif_capture_sub - Raw Image Capture Video Node > > > +----------------------------------------------- > > > + > > > +This video node is used for capturing bayer image from the sensor. > > > +The output picture has exactly the same resolution and format as the sensor input. > > > +The following depth of bayer format is supported: > > > + > > > +- 8bit > > > +- 10bit > > > +- 12bit > > > +- 14bit > > > > Does the hardware support capturing embedded data from the sensor ? > > The hardware supports capturing embdded data, however the software is > not fully tested for that feature. OK. Support for this can be added later. I recommend already checking what it would imply in terms of changes to the media graph. Changing the media graph later in a way that could break userspace won't be allowed, so it's worth it preparing for embedded data support in the media graph design already. Adding new pads to existing entities and adding new entities later are fine, but renumbering existing pads or inserting new entities between two existing entities could break userspace. [snip] -- Regards, Laurent Pinchart