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 X-Spam-Level: X-Spam-Status: No, score=-5.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 62999C64E7B for ; Tue, 1 Dec 2020 12:13:49 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D035920770 for ; Tue, 1 Dec 2020 12:13:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2390516AbgLAMNd (ORCPT ); Tue, 1 Dec 2020 07:13:33 -0500 Received: from new4-smtp.messagingengine.com ([66.111.4.230]:58343 "EHLO new4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726840AbgLAMNc (ORCPT ); Tue, 1 Dec 2020 07:13:32 -0500 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailnew.nyi.internal (Postfix) with ESMTP id 88F085801D5; Tue, 1 Dec 2020 07:12:45 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Tue, 01 Dec 2020 07:12:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm1; bh=Z+nDVstGc5vXsyJReaUNqDRiJfZ TlfID4JCekxNseFc=; b=FbEw7Izv467DChudl8cMiKyc+ajYm31ye5emZJmiaju n+XrLVKRifgoE7bgSS3LIlgl5A2a2dS0aNGGlpxn6P1flHrDww902HyHqcFHucZ6 bEU7/R1XS6/HSWNYzpPTN61Znx4stCAGoHLbuzZN2oNbQVke3eNvJQqTzSSSatEm BKEt8qjwZOfhYbe4fK2jR0lQ2GYkhktsEQ4mmIVch+eF3y8kjUNsEue5IDj6dTft hFZY3gzD+QqfYXWCUCMAXhND+ZmsCjvfLQ/1tVmK0De9O1XHF8Na2HAHy7TcOg6P rL4hKzSp75UxfiIoU3RxoNhUYbh0LMN76JwoNRuaehw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=Z+nDVs tGc5vXsyJReaUNqDRiJfZTlfID4JCekxNseFc=; b=DE/axZFj8rFjb5yrWCxEKH /y+uE3qSJEr1G3BTtyAb1e2GbObJGW/wP8QNSn+rnY+hQeA2E/iL1c4UGhnW2iGX WB0Z8qIsQwbDSqSv0/+K0ctB0DXvneGGRt0xoN15lMb/T0Dcv0MDV4JVU5rvVC/A q3YoKUEsMc4tHxEzeF57unej6WUM3t/XH3w6d3MHIST1qi/CR9nz9i7NdAzj6+Ge 5OwL6XNO1c0W05FXO+ab4fvzGii9GszsZiJxdj9G7wh9tf9cVkY/mK7tH1cRcPi8 56Rq1QKvwJmy9yA+lIS5LlNb76OLWQYxcB6zLviu+WimbtUO03BCj1TBcOUQBlLQ == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudeivddgfeelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepleekgeehhfdutdeljefgleejffehfffgieejhffgueefhfdtveetgeehieeh gedunecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id E09C33280064; Tue, 1 Dec 2020 07:12:42 -0500 (EST) Date: Tue, 1 Dec 2020 13:12:41 +0100 From: Maxime Ripard To: Paul Kocialkowski Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, devel@driverdev.osuosl.org, linux-sunxi@googlegroups.com, Yong Deng , Mauro Carvalho Chehab , Rob Herring , Chen-Yu Tsai , Jernej Skrabec , Jonathan Corbet , Kishon Vijay Abraham I , Vinod Koul , Helen Koike , Dafna Hirschfeld , Greg Kroah-Hartman , Philipp Zabel , Sakari Ailus , Hans Verkuil , Thomas Petazzoni , kevin.lhopital@hotmail.com Subject: Re: [PATCH v2 07/19] media: sun6i-csi: Add support for MIPI CSI-2 bridge input Message-ID: <20201201121241.cyafjhot45puusfc@gilmour> References: <20201128142839.517949-1-paul.kocialkowski@bootlin.com> <20201128142839.517949-8-paul.kocialkowski@bootlin.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="n3hdtuwi2v3e62ny" Content-Disposition: inline In-Reply-To: <20201128142839.517949-8-paul.kocialkowski@bootlin.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --n3hdtuwi2v3e62ny Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Sat, Nov 28, 2020 at 03:28:27PM +0100, Paul Kocialkowski wrote: > The A31 CSI controller supports a MIPI CSI-2 bridge input, which has > its own dedicated port in the fwnode graph. >=20 > Support for this input is added with this change: > - two pads are defined for the media entity instead of one > and only one needs to be connected at a time; > - the pads currently match the fwnode graph representation; > - links are created between our pads and the subdevs for each > interface and are no longer immutable so that userspace can select > which interface to use in case both are bound to a subdev; > - fwnode endpoints are parsed and stored for each interface; > - the active subdev (and fwnode endpoint) is retrieved when validating > the media link at stream on time and cleared at stream off; > - an error is raised if both links are active at the same time; > - the MIPI interface bit is set if the MIPI CSI-2 bridge endpoint is > active. >=20 > In the future, the media entity representation might evolve to: > - distinguish the internal parallel bridge and data formatter; > - represent each of the 4 internal channels that can exist between > the parallel bridge (for BT656 time-multiplex) and MIPI CSI-2 > (internal channels can be mapped to virtual channels); > - connect the controller's output to the ISP instead of its > DMA engine. >=20 > Finally note that the MIPI CSI-2 bridges should not be linked in > the fwnode graph unless they have a sensor subdev attached. I'll leave most of the review to Laurent and Sakari, but I'm not quite sure what you meant in the last paragraph. Did you mean that the MIPI-CSI controller in the Allwinner SoC should only be linked if it has a sensor attached, or did you mean that any MIPI-CSI2 bridge cannot be attached to the controller? Also, having somewhere (like your cover letter) the media-ctl setup you tested with would be great. Maxime --n3hdtuwi2v3e62ny Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX8YzOQAKCRDj7w1vZxhR xe1pAQDJ4VWYPmX2sjefU+JMNUL5vx5k6ObBfcW4bEsF8eT67wEAzkFZu7gABP+M zDT0Q5EbIbl1JDpuCyD6lxVTgkvvKgM= =bri9 -----END PGP SIGNATURE----- --n3hdtuwi2v3e62ny-- 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 X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C7BF5C64E7A for ; Tue, 1 Dec 2020 12:13:19 +0000 (UTC) Received: from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id ADCD020770 for ; Tue, 1 Dec 2020 12:13:18 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org ADCD020770 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=driverdev-devel-bounces@linuxdriverproject.org Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id 2B10B86C20; Tue, 1 Dec 2020 12:13:18 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from fraxinus.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jgVWTIvPp94J; Tue, 1 Dec 2020 12:13:15 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by fraxinus.osuosl.org (Postfix) with ESMTP id 82B6F86C35; Tue, 1 Dec 2020 12:13:15 +0000 (UTC) Received: from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136]) by ash.osuosl.org (Postfix) with ESMTP id 5C2581BF5D7 for ; Tue, 1 Dec 2020 12:12:51 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id 41CC52E133 for ; Tue, 1 Dec 2020 12:12:51 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jGATEMwGjmJS for ; Tue, 1 Dec 2020 12:12:47 +0000 (UTC) X-Greylist: from auto-whitelisted by SQLgrey-1.7.6 Received: from new4-smtp.messagingengine.com (new4-smtp.messagingengine.com [66.111.4.230]) by silver.osuosl.org (Postfix) with ESMTPS id B8B142041F for ; Tue, 1 Dec 2020 12:12:46 +0000 (UTC) Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailnew.nyi.internal (Postfix) with ESMTP id 88F085801D5; Tue, 1 Dec 2020 07:12:45 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Tue, 01 Dec 2020 07:12:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm1; bh=Z+nDVstGc5vXsyJReaUNqDRiJfZ TlfID4JCekxNseFc=; b=FbEw7Izv467DChudl8cMiKyc+ajYm31ye5emZJmiaju n+XrLVKRifgoE7bgSS3LIlgl5A2a2dS0aNGGlpxn6P1flHrDww902HyHqcFHucZ6 bEU7/R1XS6/HSWNYzpPTN61Znx4stCAGoHLbuzZN2oNbQVke3eNvJQqTzSSSatEm BKEt8qjwZOfhYbe4fK2jR0lQ2GYkhktsEQ4mmIVch+eF3y8kjUNsEue5IDj6dTft hFZY3gzD+QqfYXWCUCMAXhND+ZmsCjvfLQ/1tVmK0De9O1XHF8Na2HAHy7TcOg6P rL4hKzSp75UxfiIoU3RxoNhUYbh0LMN76JwoNRuaehw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=Z+nDVs tGc5vXsyJReaUNqDRiJfZTlfID4JCekxNseFc=; b=DE/axZFj8rFjb5yrWCxEKH /y+uE3qSJEr1G3BTtyAb1e2GbObJGW/wP8QNSn+rnY+hQeA2E/iL1c4UGhnW2iGX WB0Z8qIsQwbDSqSv0/+K0ctB0DXvneGGRt0xoN15lMb/T0Dcv0MDV4JVU5rvVC/A q3YoKUEsMc4tHxEzeF57unej6WUM3t/XH3w6d3MHIST1qi/CR9nz9i7NdAzj6+Ge 5OwL6XNO1c0W05FXO+ab4fvzGii9GszsZiJxdj9G7wh9tf9cVkY/mK7tH1cRcPi8 56Rq1QKvwJmy9yA+lIS5LlNb76OLWQYxcB6zLviu+WimbtUO03BCj1TBcOUQBlLQ == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudeivddgfeelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepleekgeehhfdutdeljefgleejffehfffgieejhffgueefhfdtveetgeehieeh gedunecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id E09C33280064; Tue, 1 Dec 2020 07:12:42 -0500 (EST) Date: Tue, 1 Dec 2020 13:12:41 +0100 From: Maxime Ripard To: Paul Kocialkowski Subject: Re: [PATCH v2 07/19] media: sun6i-csi: Add support for MIPI CSI-2 bridge input Message-ID: <20201201121241.cyafjhot45puusfc@gilmour> References: <20201128142839.517949-1-paul.kocialkowski@bootlin.com> <20201128142839.517949-8-paul.kocialkowski@bootlin.com> MIME-Version: 1.0 In-Reply-To: <20201128142839.517949-8-paul.kocialkowski@bootlin.com> X-BeenThere: driverdev-devel@linuxdriverproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Dafna Hirschfeld , linux-doc@vger.kernel.org, linux-sunxi@googlegroups.com, Thomas Petazzoni , kevin.lhopital@hotmail.com, devel@driverdev.osuosl.org, Jonathan Corbet , Kishon Vijay Abraham I , Chen-Yu Tsai , Hans Verkuil , linux-media@vger.kernel.org, devicetree@vger.kernel.org, Sakari Ailus , Helen Koike , Rob Herring , Yong Deng , Mauro Carvalho Chehab , linux-arm-kernel@lists.infradead.org, Jernej Skrabec , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Vinod Koul , Philipp Zabel Content-Type: multipart/mixed; boundary="===============5575979219080895020==" Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" --===============5575979219080895020== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="n3hdtuwi2v3e62ny" Content-Disposition: inline --n3hdtuwi2v3e62ny Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Sat, Nov 28, 2020 at 03:28:27PM +0100, Paul Kocialkowski wrote: > The A31 CSI controller supports a MIPI CSI-2 bridge input, which has > its own dedicated port in the fwnode graph. >=20 > Support for this input is added with this change: > - two pads are defined for the media entity instead of one > and only one needs to be connected at a time; > - the pads currently match the fwnode graph representation; > - links are created between our pads and the subdevs for each > interface and are no longer immutable so that userspace can select > which interface to use in case both are bound to a subdev; > - fwnode endpoints are parsed and stored for each interface; > - the active subdev (and fwnode endpoint) is retrieved when validating > the media link at stream on time and cleared at stream off; > - an error is raised if both links are active at the same time; > - the MIPI interface bit is set if the MIPI CSI-2 bridge endpoint is > active. >=20 > In the future, the media entity representation might evolve to: > - distinguish the internal parallel bridge and data formatter; > - represent each of the 4 internal channels that can exist between > the parallel bridge (for BT656 time-multiplex) and MIPI CSI-2 > (internal channels can be mapped to virtual channels); > - connect the controller's output to the ISP instead of its > DMA engine. >=20 > Finally note that the MIPI CSI-2 bridges should not be linked in > the fwnode graph unless they have a sensor subdev attached. I'll leave most of the review to Laurent and Sakari, but I'm not quite sure what you meant in the last paragraph. Did you mean that the MIPI-CSI controller in the Allwinner SoC should only be linked if it has a sensor attached, or did you mean that any MIPI-CSI2 bridge cannot be attached to the controller? Also, having somewhere (like your cover letter) the media-ctl setup you tested with would be great. Maxime --n3hdtuwi2v3e62ny Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX8YzOQAKCRDj7w1vZxhR xe1pAQDJ4VWYPmX2sjefU+JMNUL5vx5k6ObBfcW4bEsF8eT67wEAzkFZu7gABP+M zDT0Q5EbIbl1JDpuCyD6lxVTgkvvKgM= =bri9 -----END PGP SIGNATURE----- --n3hdtuwi2v3e62ny-- --===============5575979219080895020== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel --===============5575979219080895020==-- 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 X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 528EFC64E7A for ; Tue, 1 Dec 2020 12:14:19 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6807520770 for ; Tue, 1 Dec 2020 12:14:18 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Ih95Vj37"; dkim=temperror (0-bit key) header.d=cerno.tech header.i=@cerno.tech header.b="FbEw7Izv"; dkim=temperror (0-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="DE/axZFj" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6807520770 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject: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=Dl8xvzemTzpEwReUjO5dMYktJ397QHiz5C5GZKuwDFI=; b=Ih95Vj37Sdk9AwDNGFv7qEl/3 uin3VuNpykNn/9hGPfGyBwxwAcNjjIDoNT7VfY9vOfchB+tGT/PD8gM9GMvMaNtc+Y3WUGeUTjvR+ bH63Qm9k1R5FC1yV/8mObvjqgHKpQRxr2FfzCxABAGGEPPjNP08C3YWLu3vvE7ffJK0ZHABMB2+PT dPqIusjZpJetLh77UPT49j8HWoZNnSX1Fqzda2GhDPaZy6IpWJzAYIsHYNCIkGFwJ1fwk8akR7PFI hhe9mJTOOMpB2QhTReRPC4IPSqlMMpokU4L8W5d4wi9g68VY7YS/1eY141/UIVyZjjkwC6G6Nu0vC kdOPuo3Gg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kk4Wa-0006na-NX; Tue, 01 Dec 2020 12:12:52 +0000 Received: from new4-smtp.messagingengine.com ([66.111.4.230]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kk4WX-0006mg-A6 for linux-arm-kernel@lists.infradead.org; Tue, 01 Dec 2020 12:12:50 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailnew.nyi.internal (Postfix) with ESMTP id 88F085801D5; Tue, 1 Dec 2020 07:12:45 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Tue, 01 Dec 2020 07:12:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm1; bh=Z+nDVstGc5vXsyJReaUNqDRiJfZ TlfID4JCekxNseFc=; b=FbEw7Izv467DChudl8cMiKyc+ajYm31ye5emZJmiaju n+XrLVKRifgoE7bgSS3LIlgl5A2a2dS0aNGGlpxn6P1flHrDww902HyHqcFHucZ6 bEU7/R1XS6/HSWNYzpPTN61Znx4stCAGoHLbuzZN2oNbQVke3eNvJQqTzSSSatEm BKEt8qjwZOfhYbe4fK2jR0lQ2GYkhktsEQ4mmIVch+eF3y8kjUNsEue5IDj6dTft hFZY3gzD+QqfYXWCUCMAXhND+ZmsCjvfLQ/1tVmK0De9O1XHF8Na2HAHy7TcOg6P rL4hKzSp75UxfiIoU3RxoNhUYbh0LMN76JwoNRuaehw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=Z+nDVs tGc5vXsyJReaUNqDRiJfZTlfID4JCekxNseFc=; b=DE/axZFj8rFjb5yrWCxEKH /y+uE3qSJEr1G3BTtyAb1e2GbObJGW/wP8QNSn+rnY+hQeA2E/iL1c4UGhnW2iGX WB0Z8qIsQwbDSqSv0/+K0ctB0DXvneGGRt0xoN15lMb/T0Dcv0MDV4JVU5rvVC/A q3YoKUEsMc4tHxEzeF57unej6WUM3t/XH3w6d3MHIST1qi/CR9nz9i7NdAzj6+Ge 5OwL6XNO1c0W05FXO+ab4fvzGii9GszsZiJxdj9G7wh9tf9cVkY/mK7tH1cRcPi8 56Rq1QKvwJmy9yA+lIS5LlNb76OLWQYxcB6zLviu+WimbtUO03BCj1TBcOUQBlLQ == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudeivddgfeelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepleekgeehhfdutdeljefgleejffehfffgieejhffgueefhfdtveetgeehieeh gedunecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id E09C33280064; Tue, 1 Dec 2020 07:12:42 -0500 (EST) Date: Tue, 1 Dec 2020 13:12:41 +0100 From: Maxime Ripard To: Paul Kocialkowski Subject: Re: [PATCH v2 07/19] media: sun6i-csi: Add support for MIPI CSI-2 bridge input Message-ID: <20201201121241.cyafjhot45puusfc@gilmour> References: <20201128142839.517949-1-paul.kocialkowski@bootlin.com> <20201128142839.517949-8-paul.kocialkowski@bootlin.com> MIME-Version: 1.0 In-Reply-To: <20201128142839.517949-8-paul.kocialkowski@bootlin.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201201_071249_613958_29E8DB95 X-CRM114-Status: GOOD ( 23.64 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Dafna Hirschfeld , linux-doc@vger.kernel.org, linux-sunxi@googlegroups.com, Thomas Petazzoni , kevin.lhopital@hotmail.com, devel@driverdev.osuosl.org, Jonathan Corbet , Kishon Vijay Abraham I , Chen-Yu Tsai , Hans Verkuil , linux-media@vger.kernel.org, devicetree@vger.kernel.org, Sakari Ailus , Helen Koike , Rob Herring , Yong Deng , Mauro Carvalho Chehab , linux-arm-kernel@lists.infradead.org, Jernej Skrabec , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Vinod Koul , Philipp Zabel Content-Type: multipart/mixed; boundary="===============4432859313821836987==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============4432859313821836987== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="n3hdtuwi2v3e62ny" Content-Disposition: inline --n3hdtuwi2v3e62ny Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Sat, Nov 28, 2020 at 03:28:27PM +0100, Paul Kocialkowski wrote: > The A31 CSI controller supports a MIPI CSI-2 bridge input, which has > its own dedicated port in the fwnode graph. >=20 > Support for this input is added with this change: > - two pads are defined for the media entity instead of one > and only one needs to be connected at a time; > - the pads currently match the fwnode graph representation; > - links are created between our pads and the subdevs for each > interface and are no longer immutable so that userspace can select > which interface to use in case both are bound to a subdev; > - fwnode endpoints are parsed and stored for each interface; > - the active subdev (and fwnode endpoint) is retrieved when validating > the media link at stream on time and cleared at stream off; > - an error is raised if both links are active at the same time; > - the MIPI interface bit is set if the MIPI CSI-2 bridge endpoint is > active. >=20 > In the future, the media entity representation might evolve to: > - distinguish the internal parallel bridge and data formatter; > - represent each of the 4 internal channels that can exist between > the parallel bridge (for BT656 time-multiplex) and MIPI CSI-2 > (internal channels can be mapped to virtual channels); > - connect the controller's output to the ISP instead of its > DMA engine. >=20 > Finally note that the MIPI CSI-2 bridges should not be linked in > the fwnode graph unless they have a sensor subdev attached. I'll leave most of the review to Laurent and Sakari, but I'm not quite sure what you meant in the last paragraph. Did you mean that the MIPI-CSI controller in the Allwinner SoC should only be linked if it has a sensor attached, or did you mean that any MIPI-CSI2 bridge cannot be attached to the controller? Also, having somewhere (like your cover letter) the media-ctl setup you tested with would be great. Maxime --n3hdtuwi2v3e62ny Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX8YzOQAKCRDj7w1vZxhR xe1pAQDJ4VWYPmX2sjefU+JMNUL5vx5k6ObBfcW4bEsF8eT67wEAzkFZu7gABP+M zDT0Q5EbIbl1JDpuCyD6lxVTgkvvKgM= =bri9 -----END PGP SIGNATURE----- --n3hdtuwi2v3e62ny-- --===============4432859313821836987== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============4432859313821836987==--