Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp1083152pxb; Wed, 27 Oct 2021 19:05:07 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz0iNtviGplaJlORS9rVO67mMeASdPfJgsHyNjPGMbislLieECNOhYCNJzJgqc2cwRh1XCp X-Received: by 2002:a17:90b:4a92:: with SMTP id lp18mr1406636pjb.47.1635386706886; Wed, 27 Oct 2021 19:05:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1635386706; cv=none; d=google.com; s=arc-20160816; b=W/yF32cWGFVaYxmw208MLbRoB/7Fuy+6545tysdxBSWikSLFgK7ErDngMhLclwUbR2 3qZYjmxLkdIcgR5lNsjfeHEaFCasi+vAqd0kfYkULqecKswIUo1+2tSUJsrJt5bNdrdO pFljkDeQrXBzOCbvCnO1alr+D7uozxTVf3P6XG4BZS1+B1ZzEZQ0sp0UK5nr2CsNMWhG 226lNQbEwqScxKoHJi+h+52EWPtqxcNLkv5DKYNbhkAKcmrsjLFUHFTzb5YN2BLW27dH enPAIebTMBAAnDA6VDJKvQ1NEsm2nKHrKh251q88PEK5RQjPiQtov67cNwQzpPK59ygt ZJeQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=13o0T6M+SZyADcFsuIjdVRVDzVZ9SlHMzmanJvHCPuQ=; b=rhvIVMXytapQdOWI4MLjA/pl1TqoMInrQ7gVNdq6ECY/iePWJbyu2yiobj/hPyfFeW yd1kFiR9DZRSgkZQvRpRrNWqipqA8JpbYcCyrp0KSQdOjonTAzuNdxMebScKGFTNOygK Z0rOiqdIwzF+/0NOGfSdPIeUScfQLe8RsGYHeQ4qRguTDbzcWirx2IDyMc+JndLkMH5y aiMrgG2B0YXODie+wvuyXdpDVME8nKz8OzNMsg1dMSiChRIv81hq0vz181kuVnOrRJ3I T//Q08Kufw/KQZJJNMsGGdlSaklFYKiZDJXmn+PJD6fdurvs5OtZERmhwzYVdm1Wv9+0 qpnA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=EdhDdkNd; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id r12si2442677pgv.21.2021.10.27.19.04.54; Wed, 27 Oct 2021 19:05:06 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=EdhDdkNd; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229703AbhJ1CGS (ORCPT + 99 others); Wed, 27 Oct 2021 22:06:18 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:32976 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229534AbhJ1CGR (ORCPT ); Wed, 27 Oct 2021 22:06:17 -0400 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 581B8C061570; Wed, 27 Oct 2021 19:03:51 -0700 (PDT) Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 9738C276; Thu, 28 Oct 2021 04:03:46 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1635386626; bh=5UFjGoxIACVBiNY+4qv4rZSdiMAoqqZWEfDV5pYwRNs=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EdhDdkNdVAA1mlU3SiJC3b5wCrJzER3M7sXwdWK+uDkxiMKzq9Hb0UHl8VB4mCkON cM5Ff1HWD4yNv/hCulSTzuO3xZ5dRxcz7eFSMcsik55NbJDQ0PrZpOA8KziwtcCaa3 yAXEFfoG+/beZqkU7zy0Mah2nJd2L7ezk1ZLy/4I= Date: Thu, 28 Oct 2021 05:03:22 +0300 From: Laurent Pinchart To: Adam Ford Cc: linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, cstevens@beaconembedded.com, aford@beaconembedded.com, Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Catalin Marinas , Will Deacon , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFC V2 1/5] arm64: dts: imx8mm: Add CSI nodes Message-ID: References: <20211023203457.1217821-1-aford173@gmail.com> <20211023203457.1217821-2-aford173@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20211023203457.1217821-2-aford173@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Adam, Thank you for the patch. On Sat, Oct 23, 2021 at 03:34:52PM -0500, Adam Ford wrote: > There is a csi bridge and csis interface that tie together > to allow csi2 capture. > > Signed-off-by: Adam Ford > --- > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 55 +++++++++++++++++++++++ > 1 file changed, 55 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > index c2f3f118f82e..920f9041ef50 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > @@ -1068,6 +1068,22 @@ aips4: bus@32c00000 { > #size-cells = <1>; > ranges = <0x32c00000 0x32c00000 0x400000>; > > + csi: csi@32e20000 { > + compatible = "fsl,imx8mm-csi", "fsl,imx7-csi"; > + reg = <0x32e20000 0x1000>; > + interrupts = ; > + clocks = <&clk IMX8MM_CLK_CSI1_ROOT>; > + clock-names = "mclk"; > + power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_CSI_BRIDGE>; > + status = "disabled"; > + > + port { > + csi_in: endpoint { > + remote-endpoint = <&imx8mm_mipi_csi_out>; > + }; > + }; > + }; > + > disp_blk_ctrl: blk-ctrl@32e28000 { > compatible = "fsl,imx8mm-disp-blk-ctrl", "syscon"; > reg = <0x32e28000 0x100>; > @@ -1095,6 +1111,45 @@ disp_blk_ctrl: blk-ctrl@32e28000 { > #power-domain-cells = <1>; > }; > > + mipi_csi2: mipi-csi@32e30000 { I'd rename the label to mipi_csi to match the name in the reference manual. > + compatible = "fsl,imx8mm-mipi-csi2"; > + reg = <0x32e30000 0x1000>; > + interrupts = ; > + assigned-clocks = <&clk IMX8MM_CLK_CSI1_CORE>, > + <&clk IMX8MM_CLK_CSI1_PHY_REF>; > + assigned-clock-parents = <&clk IMX8MM_SYS_PLL2_1000M>, > + <&clk IMX8MM_SYS_PLL2_1000M>; > + clock-frequency = <333000000>; > + clocks = <&clk IMX8MM_CLK_DISP_APB_ROOT>, > + <&clk IMX8MM_CLK_CSI1_ROOT>, > + <&clk IMX8MM_CLK_CSI1_PHY_REF>, > + <&clk IMX8MM_CLK_DISP_AXI_ROOT>; > + clock-names = "pclk", "wrap", "phy", "axi"; > + power-domains = <&disp_blk_ctrl IMX8MM_DISPBLK_PD_MIPI_CSI>; > + status = "disabled"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + imx8mm_mipi_csi_in: endpoint { > + }; Empty ports are fine as they model the hardware, even when not connected, but an endpoint models a connection, so it needs a remote endpoint. You can drop the endpoint here, board DT files will create one with &mipi_csi { ports { port@0 { board_endpoint: endpoint { ... }; }; }; }; > + }; > + > + port@1 { > + reg = <1>; > + > + imx8mm_mipi_csi_out: endpoint { > + remote-endpoint = <&csi_in>; > + }; > + }; > + }; > + }; > + > + A single blank line is enough. Reviewed-by: Laurent Pinchart > usbotg1: usb@32e40000 { > compatible = "fsl,imx8mm-usb", "fsl,imx7d-usb"; > reg = <0x32e40000 0x200>; -- Regards, Laurent Pinchart