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 A8C3CC64E7C for ; Wed, 2 Dec 2020 15:42:26 +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 2F26F20B1F for ; Wed, 2 Dec 2020 15:42:23 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2F26F20B1F 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=wjEQF8eotCGgpX44Shagrc/cUARUZjGtvSb6Y+TYp+c=; b=HEJZoRiGHJH2winCbKPWQuw7n 1riVQQRfr6C4j3jqKQ3I+dwB8oH2nOrrH0YB8p0iCfXnlgTLox9jOqTNDRmAsNxUii1qI7FtO0L/B 0Yrw4RPCjUMD6KwSxPMUq7vOnf1lCKPmLbIbUJcOAOvnv0D2DUWbSKGbA/VRefceMRAhO33AJCeiL soeEBT2qAoxmMSMZi3Z8vYYsKGaznz7OFJPQ47TItdJtE3w7P6+UvJTS8Mg3nYjQADsLHMnKWPFM4 0rEo1rhPJoyvjJ7j5C/SGKw6OosFpYrxefT6oWBqklgAao8WifCVdyOQHlrDSTl+B97fULpPru/32 U7QfjA3Eg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kkUFa-0003dp-OR; Wed, 02 Dec 2020 15:41:02 +0000 Received: from new2-smtp.messagingengine.com ([66.111.4.224]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kkUFX-0003d2-UC for linux-arm-kernel@lists.infradead.org; Wed, 02 Dec 2020 15:41:01 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailnew.nyi.internal (Postfix) with ESMTP id 458EC580387; Wed, 2 Dec 2020 10:40:58 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Wed, 02 Dec 2020 10:40:58 -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=9Gmw2R9k0ECc5QddBoAVCGV1sXx zmzO4YbJmiIaU4YY=; b=XTaU5h2f3WAixKnXCS97j2KVWpPWFVh0YhT+m4Nj3CM d8fTiUZiwPxy0O5RDJhyFigFAz8sZz+tbotDWcC+lmTl53Ikj9hzxkoHOvZx4Hdg FHl+5ybpwHd/CInVL1bT07psyo33xntHw9gOTFFJX3XlcgTC7hIttoxNqcv+uUeo LZQPJSK/VNUNsS9C/ou1MotSZMMX0hGIUtSeitIqp/teQgRTE1jozqWsg2tNM+Vn USR1udlNMb86zaufmPt12VFdaaCvyfDp4yC4U8VFzxXM7WpMW7JKENwMOH/bGpX3 WtGeRMbpE+ekTaVlvtl49Bm8G884l6R2RuIACX1pO4w== 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=9Gmw2R 9k0ECc5QddBoAVCGV1sXxzmzO4YbJmiIaU4YY=; b=GObBsEcZzgssNLZ33qx0AJ GhlgBV+VE8PDXxHuIwxRZtWVbtBsMqOVYwL3HUiCeUs7vY8qYe7eEgG24NXTJrw5 eJgCZirNGC/0NxuBWDfYbSsxr1R9Xq00iLJEo3mFb51sjeL9PGDCc09ibc4NbLEd 0KooxPvuwHzHE0LxBpGNgx7SCw9X4xddpdRZbhog0DNe1o2WVVFV9AoU8bzzO1NL UW3xSz6z9XM5YMvkD0pfQlzsIhp4ctFCnj2JwxMvQpMw4wAqOGvL5a3BMOQT1teW JwGpgNnv8qyoHrp5w/jg0fLjmzY+6Be9L1HWOP9B78yeu9+bujjcrxCKZT4ofyvQ == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudeigedgkeduucetufdoteggodetrfdotf 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 D6D14108005B; Wed, 2 Dec 2020 10:40:54 -0500 (EST) Date: Wed, 2 Dec 2020 16:40:53 +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: <20201202154053.3fcxiift2uyqnjvp@gilmour> References: <20201128142839.517949-1-paul.kocialkowski@bootlin.com> <20201128142839.517949-8-paul.kocialkowski@bootlin.com> <20201201121241.cyafjhot45puusfc@gilmour> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201202_104100_157857_9F4D7E4C X-CRM114-Status: GOOD ( 33.10 ) 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="===============6211978757351862844==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6211978757351862844== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="45hcbhhgcfnzxspi" Content-Disposition: inline --45hcbhhgcfnzxspi Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Dec 02, 2020 at 03:19:11PM +0100, Paul Kocialkowski wrote: > Hi, >=20 > On Tue 01 Dec 20, 13:12, Maxime Ripard wrote: > > Hi, > >=20 > > 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. > >=20 > > 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? >=20 > So the use of plural was a mistake and your first understanding is the co= rrect > one: if the bridge is linked to the CSI controller in the OF graph but the > bridge doesn't have a sensor attached, the CSI controller driver will fail > to probe, as far as I could see. I'm not sure it's reasonable to not link it in the DTSI then, we'll want to reduce as much the boilerplate from the board DTS as possible, and the MIPI-CSI controller is always there anyway. However, we should definitely have it disabled if there's no sensor, which should solve your probe issue Maxime --45hcbhhgcfnzxspi Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX8e1hQAKCRDj7w1vZxhR xUmfAQDrnEGoBdbQg8pcpjrHuleZtMVcXIe+BVlxHlRQPOZFXwEAjruXhWd67Q5L aw25nh4z4WBzD8T8dUj7t5A/R28hows= =crGk -----END PGP SIGNATURE----- --45hcbhhgcfnzxspi-- --===============6211978757351862844== 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 --===============6211978757351862844==--