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 7BF64C76188 for ; Wed, 5 Apr 2023 13:17:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=CbEKIeLuC3oxNTDqDr3O+j+ESQ/kfk2LpRl3PaqMNVc=; b=PT6ioVTl6lQgEO Ra9spB2QIxrz0vQSn8ttzPguNayXss8QJrSULLuinX1kZTzasn9g1dAnX9/S9Vy/6te8ZeCb/cdJr Uqh581b3wwYqCQ6maSrAtp4M7rAFftoo8UHJGhvql63A56PN4wZCMuWzqmSDteNOztZ5FHWLrIBgp jLz65XSeePZ0hgqXMoknizV+1VqHRJ+L1zn5RS7mSULVGajKr9eMCkFxuBX+0w8MaDqkfMeecReBy Yz6japfKPR4wITMN2FkE21WKhDtdaefbO7mTIQ2QDDHlKsReykiuYiYtVhjsHhjBhNi6hN1+H2Fkz hwGhsKdecqr3kfXqyHfA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pk304-004VVo-22; Wed, 05 Apr 2023 13:16:32 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pk301-004VUx-0E for linux-arm-kernel@lists.infradead.org; Wed, 05 Apr 2023 13:16:31 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1680700589; x=1712236589; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=4RVlb8yI3d39rdesnGfgQTzPDTdk41ZCWB+kZCrJpSY=; b=F6L8Z47hKTc+qj3doaaWXULGllYl8YcqOgNLR8D5Kfq+42f2SbD8wRtH ep08sKOk5fEzhsD0hTU2lZgml5jH9VrR/sGy71oXE3DxxvHYIxjPi9cpY SZa2VIXAg5ChoAwOw1sxb3GLB+3gKkLDvVxABYl5Kv7yGMFVgbn2wJv98 bc8iD6pxbolj7NKn8vrWWrM92QKiCHQRS7ONnBt7eCJK1xSSDSnCwZ1Bm k22x30nonduOIkkhpbFGoQccpRm9KbGo9FO3h06BKdFqQOwfM5/dtbtg9 QEOau561AYDGPel4S+hkcZq80x+8O3ZR/zd4ca720+J6U2IQEBfPbOQhw Q==; X-IronPort-AV: E=Sophos;i="5.98,319,1673910000"; d="scan'208";a="30167092" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 05 Apr 2023 15:16:26 +0200 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Wed, 05 Apr 2023 15:16:26 +0200 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Wed, 05 Apr 2023 15:16:26 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1680700586; x=1712236586; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=4RVlb8yI3d39rdesnGfgQTzPDTdk41ZCWB+kZCrJpSY=; b=pQLNM3A4TDGz629L1RdLce1KJeVYKanF4txXLbA1i4jDaJOR1G8eGlUO xxeGb8jOmSarEak2DY6tEH8QLtXer9SHGk2dgZgK7ld69W6rAmu+TcuXv yMj5jjED8zN7qz9MsFba6zzV3+EVHS++LutKePDnu2BaHTiii9FL22zH4 +4w0IT58BDWcmBeYty58P22VSyBtK/S587i9KCjvEUsmUT1ktOvVXkoNP jVLBZgoXxQas0M+Uz4ISvpWLPo8aoUagHbGUIjT50OK15L2sKq+ZAkh3V 0irp0nVSBpUDb0Ql4KSt4GIAaenztNKbsOcZFcv2MYXYMXu55EEBpWWTf A==; X-IronPort-AV: E=Sophos;i="5.98,319,1673910000"; d="scan'208";a="30167091" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 05 Apr 2023 15:16:26 +0200 Received: from steina-w.localnet (unknown [10.123.53.21]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by vtuxmail01.tq-net.de (Postfix) with ESMTPSA id D20DA280056; Wed, 5 Apr 2023 15:16:25 +0200 (CEST) From: Alexander Stein To: linux-arm-kernel@lists.infradead.org, Marek Vasut Cc: Abel Vesa , Dong Aisheng , Fabio Estevam , Guido =?ISO-8859-1?Q?G=FCnther?= , Krzysztof Kozlowski , Lucas Stach , NXP Linux Team , Pengutronix Kernel Team , Richard Cochran , Rob Herring , Sascha Hauer , Shawn Guo , devicetree@vger.kernel.org Subject: Re: [PATCH] arm64: dts: imx8mm: Add display pipeline components Date: Wed, 05 Apr 2023 15:16:25 +0200 Message-ID: <4588730.LvFx2qVVIh@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <3b6696c1-411c-f382-0da4-67f9272fa306@denx.de> References: <20230404000151.534530-1-marex@denx.de> <2790760.mvXUDI8C0e@steina-w> <3b6696c1-411c-f382-0da4-67f9272fa306@denx.de> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230405_061629_494095_475A41C9 X-CRM114-Status: GOOD ( 20.50 ) 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: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, Am Mittwoch, 5. April 2023, 14:32:35 CEST schrieb Marek Vasut: > On 4/5/23 08:00, Alexander Stein wrote: > > Hi Marek, > = > Hi, > = > > Am Dienstag, 4. April 2023, 02:01:51 CEST schrieb Marek Vasut: > >> Add LCDIF scanout engine and DSIM bridge nodes for i.MX8M Mini. > >> This makes the DSI display pipeline available on this SoC. > >> = > >> Signed-off-by: Marek Vasut > >> --- > >> Cc: Abel Vesa > >> Cc: Dong Aisheng > >> Cc: Fabio Estevam > >> Cc: Guido G=FCnther > >> Cc: Krzysztof Kozlowski > >> Cc: Lucas Stach > >> Cc: NXP Linux Team > >> Cc: Pengutronix Kernel Team > >> Cc: Richard Cochran > >> Cc: Rob Herring > >> Cc: Sascha Hauer > >> Cc: Shawn Guo > >> Cc: devicetree@vger.kernel.org > >> Cc: linux-arm-kernel@lists.infradead.org > >> --- > >> = > >> arch/arm64/boot/dts/freescale/imx8mm.dtsi | 57 +++++++++++++++++++++= ++ > >> 1 file changed, 57 insertions(+) > >> = > >> diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > >> b/arch/arm64/boot/dts/freescale/imx8mm.dtsi index > >> e311da7e87bdc..06495c225c94b 100644 > >> --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > >> +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > >> @@ -1119,6 +1119,63 @@ aips4: bus@32c00000 { > >> = > >> #size-cells =3D <1>; > >> ranges =3D <0x32c00000 0x32c00000 0x400000>; > >> = > >> + lcdif: lcdif@32e00000 { > >> + compatible =3D "fsl,imx8mm-lcdif", > > = > > "fsl,imx6sx-lcdif"; > > = > >> + reg =3D <0x32e00000 0x10000>; > >> + clocks =3D <&clk > > = > > IMX8MM_CLK_LCDIF_PIXEL>, > > = > >> + <&clk > > = > > IMX8MM_CLK_DISP_APB_ROOT>, > > = > >> + <&clk > > = > > IMX8MM_CLK_DISP_AXI_ROOT>; > > = > >> + clock-names =3D "pix", "axi", > > = > > "disp_axi"; > > = > >> + assigned-clocks =3D <&clk > > = > > IMX8MM_CLK_LCDIF_PIXEL>, > > = > >> + <&clk > > = > > IMX8MM_CLK_DISP_AXI>, > > = > >> + <&clk > > = > > IMX8MM_CLK_DISP_APB>; > > = > >> + assigned-clock-parents =3D <&clk > > = > > IMX8MM_VIDEO_PLL1_OUT>, > > = > >> + <&clk > > = > > IMX8MM_SYS_PLL2_1000M>, > > = > >> + <&clk > > = > > IMX8MM_SYS_PLL1_800M>; > > = > >> + assigned-clock-rates =3D <594000000>, > > = > > <500000000>, <200000000>; > > = > >> + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > >> + power-domains =3D <&disp_blk_ctrl > > = > > IMX8MM_DISPBLK_PD_LCDIF>; > > = > >> + status =3D "disabled"; > >> + > >> + port { > >> + lcdif_to_dsim: endpoint { > >> + remote-endpoint =3D > > = > > <&dsim_from_lcdif>; > > = > >> + }; > >> + }; > >> + }; > >> + > >> + mipi_dsi: dsi@32e10000 { > >> + #address-cells =3D <1>; > >> + #size-cells =3D <0>; I missed that. These are not necessary. > >> + compatible =3D "fsl,imx8mm-mipi-dsim"; > >> + reg =3D <0x32e10000 0x400>; > >> + clocks =3D <&clk IMX8MM_CLK_DSI_CORE>, > >> + <&clk > > = > > IMX8MM_CLK_DSI_PHY_REF>; > > = > >> + clock-names =3D "bus_clk", "sclk_mipi"; > >> + assigned-clocks =3D <&clk > > = > > IMX8MM_CLK_DSI_CORE>, > > = > >> + <&clk > > = > > IMX8MM_CLK_DSI_PHY_REF>; > > = > >> + assigned-clock-parents =3D <&clk > > = > > IMX8MM_SYS_PLL1_266M>, > > = > >> + <&clk > > = > > IMX8MM_CLK_24M>; > > = > >> + assigned-clock-rates =3D <266000000>, > > = > > <24000000>; > > = > >> + samsung,pll-clock-frequency =3D > > = > > <24000000>; > > = > > What about samsung,burst-clock-frequency and samsung,esc-clock-frequenc= y? > > Is there is sane default? Do they need to be provided in the board file? > Those two (burst and esc) are display or nearest-bridge specific and go > into board files. Thanks for clarification. With the nit from above fixed: Reviewed-by: Alexander Stein -- = TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/ _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel