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 5A0E4E7718F for ; Mon, 30 Dec 2024 03:19:08 +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=RbKh81WgtbyngPn+SsJDB5TvtHitA1PETtGCx7cSW5M=; b=rw9tVqumydxEHbQUyYb8LV9STv 8UZi8FQNReYnicYABOcSalJf+Fb/4DhAr7Ty/RiybV4gb3j0rRr8xf1DodKgZzecyXE+OScOdAbSH EyyPW7jaLNYU3g3dLURpxsJ0Efke2ADLYucp7hVD0TTnMB/ktWFW50xZoaMUAxhCMxIlUEtfVCR6C FhN+SLWH8g/JmC51l/ESNVkenR1vjynSsfg2FLeqpqVwNBQVEimh4NZqShsiVsZbyCXkh3sHqU5No Td7JR31CRoK6NDHkOFL5psgJeJa4eg9AusZvoru6ZtLtmn4WU1o99B/J2RwOIqEoTTK8eqinZtDfy etT+IxFg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tS6Iv-00000004QNr-1ssb; Mon, 30 Dec 2024 03:18:53 +0000 Received: from mail-m16.yeah.net ([1.95.21.14]) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tS6Hj-00000004QIq-1sJp for linux-arm-kernel@lists.infradead.org; Mon, 30 Dec 2024 03:17:41 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yeah.net; s=s110527; h=Date:From:Subject:Message-ID:MIME-Version: Content-Type; bh=RbKh81WgtbyngPn+SsJDB5TvtHitA1PETtGCx7cSW5M=; b=eULHdMTyiAHyXRKyuZtEFCOaDPu+KsJ8oRxJ9irgKnhh0kFJ4qPzmulCLfcQj/ 1i6w3OIoqM5Ewn745smuCbDZow6NkiLlXABGesulR4lFwT2XUQZFbN5fc3sjyQrb 0f1L9BeDf14rAhjjYTQzFMgQqSIjwdrOScGqcve9LV+xc= Received: from dragon (unknown []) by gzsmtp2 (Coremail) with SMTP id Ms8vCgDXXKWPEHJnMwscBg--.61926S3; Mon, 30 Dec 2024 11:16:34 +0800 (CST) Date: Mon, 30 Dec 2024 11:16:31 +0800 From: Shawn Guo To: Pengfei Li Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, catalin.marinas@arm.com, will@kernel.org, kernel@pengutronix.de, festevam@gmail.com, joao.goncalves@toradex.com, marex@denx.de, hvilleneuve@dimonoff.com, hiago.franco@toradex.com, peng.fan@nxp.com, frieder.schrempf@kontron.de, alexander.stein@ew.tq-group.com, m.othacehe@gmail.com, mwalle@kernel.org, Max.Merchel@ew.tq-group.com, quic_bjorande@quicinc.com, geert+renesas@glider.be, dmitry.baryshkov@linaro.org, neil.armstrong@linaro.org, arnd@arndb.de, nfraprado@collabora.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, ping.bai@nxp.com, ye.li@nxp.com, aisheng.dong@nxp.com, frank.li@nxp.com, carlos.song@nxp.com Subject: Re: [PATCH v3 2/4] arm64: dts: freescale: Add i.MX91 dtsi support Message-ID: References: <20241120094945.3032663-1-pengfei.li_1@nxp.com> <20241120094945.3032663-3-pengfei.li_1@nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241120094945.3032663-3-pengfei.li_1@nxp.com> X-CM-TRANSID: Ms8vCgDXXKWPEHJnMwscBg--.61926S3 X-Coremail-Antispam: 1Uf129KBjvJXoWxJFWxKr1DWry8CFyDXF1UZFb_yoW5uw1rpF ykCFy3Wrs7Ka48u3sagFsYgrykJr9rXFW09F42vry0yws3Xr1xta45KF47XF48Xr4rWw4S gr1aqr4ftr9xuaDanT9S1TB71UUUUU7qnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07U9Z23UUUUU= X-Originating-IP: [114.216.146.204] X-CM-SenderInfo: pvkd40hjxrjqh1hdxhhqhw/1tbiCRjBZWdtNPG7vgABs+ X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241229_191740_037340_97A1441F X-CRM114-Status: GOOD ( 18.89 ) 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 On Wed, Nov 20, 2024 at 05:49:43PM +0800, Pengfei Li wrote: > The i.MX 91 family features an Arm Cortex-A55 running at up to > 1.4GHz, support for modern LPDDR4 memory to enable platform longevity, > along with a rich set of peripherals targeting medical, industrial > and consumer IoT market segments. > > The design of the i.MX91 platform is very similar to i.MX93. > The mainly difference between i.MX91 and i.MX93 is as follows: > - i.MX91 removed some clocks and modified the names of some clocks. > - i.MX91 only has one A core > - i.MX91 has different pinmux > > Signed-off-by: Pengfei Li > --- > arch/arm64/boot/dts/freescale/imx91-pinfunc.h | 770 ++++++++++++++++++ > arch/arm64/boot/dts/freescale/imx91.dtsi | 70 ++ > 2 files changed, 840 insertions(+) > create mode 100644 arch/arm64/boot/dts/freescale/imx91-pinfunc.h > create mode 100644 arch/arm64/boot/dts/freescale/imx91.dtsi ... > diff --git a/arch/arm64/boot/dts/freescale/imx91.dtsi b/arch/arm64/boot/dts/freescale/imx91.dtsi > new file mode 100644 > index 000000000000..be923e5076a4 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx91.dtsi > @@ -0,0 +1,70 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright 2024 NXP > + */ > + > +#include "imx91-pinfunc.h" > +#include "imx93.dtsi" > + > +/delete-node/ &A55_1; > +/delete-node/ &cm33; > +/delete-node/ &mlmix; > +/delete-node/ &mu1; > +/delete-node/ &mu2; > + > +&clk { > + compatible = "fsl,imx91-ccm"; > +}; > + > +&eqos { > + clocks = <&clk IMX91_CLK_ENET1_QOS_TSN_GATE>, > + <&clk IMX91_CLK_ENET1_QOS_TSN_GATE>, > + <&clk IMX91_CLK_ENET_TIMER>, > + <&clk IMX91_CLK_ENET1_QOS_TSN>, > + <&clk IMX91_CLK_ENET1_QOS_TSN_GATE>; Can we align the lines at '<'? clocks = <&clk IMX91_CLK_ENET1_QOS_TSN_GATE>, <&clk IMX91_CLK_ENET1_QOS_TSN_GATE>, <&clk IMX91_CLK_ENET_TIMER>, <&clk IMX91_CLK_ENET1_QOS_TSN>, <&clk IMX91_CLK_ENET1_QOS_TSN_GATE>; > + assigned-clocks = <&clk IMX91_CLK_ENET_TIMER>, > + <&clk IMX91_CLK_ENET1_QOS_TSN>; > + assigned-clock-parents = <&clk IMX93_CLK_SYS_PLL_PFD1_DIV2>, > + <&clk IMX93_CLK_SYS_PLL_PFD0_DIV2>; > +}; > + > +&fec { > + clocks = <&clk IMX91_CLK_ENET2_REGULAR_GATE>, > + <&clk IMX91_CLK_ENET2_REGULAR_GATE>, > + <&clk IMX91_CLK_ENET_TIMER>, > + <&clk IMX91_CLK_ENET2_REGULAR>, > + <&clk IMX93_CLK_DUMMY>; > + assigned-clocks = <&clk IMX91_CLK_ENET_TIMER>, > + <&clk IMX91_CLK_ENET2_REGULAR>; > + assigned-clock-parents = <&clk IMX93_CLK_SYS_PLL_PFD1_DIV2>, > + <&clk IMX93_CLK_SYS_PLL_PFD0_DIV2>; > + assigned-clock-rates = <100000000>, <250000000>; > +}; > + > +&i3c1 { > + clocks = <&clk IMX93_CLK_BUS_AON>, > + <&clk IMX93_CLK_I3C1_GATE>, > + <&clk IMX93_CLK_DUMMY>; > +}; > + > +&i3c2 { > + clocks = <&clk IMX93_CLK_BUS_WAKEUP>, > + <&clk IMX93_CLK_I3C2_GATE>, > + <&clk IMX93_CLK_DUMMY>; > +}; > + > +&iomuxc { > + compatible = "fsl,imx91-iomuxc"; > +}; > + > +&tmu { > + status = "disabled"; > +}; > + > +&{/soc@0/ddr-pmu@4e300dc0} { > + compatible = "fsl,imx91-ddr-pmu", "fsl,imx93-ddr-pmu"; > +}; > + > +&{/thermal-zones/cpu-thermal/cooling-maps/map0} { > + cooling-device = <&A55_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; > +}; Could you add labels in imx93.dtsi to make the references a bit easier? Shawn > -- > 2.34.1 >