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 C97BBD43352 for ; Thu, 7 Nov 2024 12:52:03 +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:Content-Type: Content-Transfer-Encoding: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=rVq9b6pcFOHREaNN9fex606hOk1inbkqT9uhtMs6aJA=; b=Nyj4HVgXyw0C4xG3AqSmWV5LjS nujuGrmmDMz/3pywuKuD05DYdLWswRhbmGt7NoLRyZ588+jTyjUxfvixlSh7jjBU0PPPyHHNWsUxI 88HhPNcQycvDhYEqN9ewIVAfxM92qL8Pvax4lpVfd4BV3jH+w5XjiN1PEFkOYl/euFJAkLubhm5lN KEYSmfdJwUwHc+ZcUmBshLJKv/EWc5aGDWtYGKiBi5UJtsjGUC/jsA5LAX9rgdRqKTLPeCRHKOXBM jBKoL893WYDbdNy0TAVv4/Z9tDt5z9tzqY/SKAAPEQJdVP/KxKpr0lXsEzqsh6FJttSli1HmkTRr1 oX2JAiVQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t91zN-00000006zXc-0cNg; Thu, 07 Nov 2024 12:51:53 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t91xb-00000006zJt-0e4V for linux-arm-kernel@lists.infradead.org; Thu, 07 Nov 2024 12:50:05 +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=1730983803; x=1762519803; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=rVq9b6pcFOHREaNN9fex606hOk1inbkqT9uhtMs6aJA=; b=AR0GgWcZg/N6WS2Kc9CiVJVuJMjCQGRm3G3BunjiweYc25ar7OMpbRU5 79diilq2tXeTsZznoRfDXCQuRNNTwGh/wTU9Wk2yZQkwNDqtgChwhZAjx +7zbMac2QyQTHgTewbwfkQQlkp57V84X03J5wtcUp+ZSfAtkst8NaFljq 7fdWzKQwRKgPR4zu4o7qpu7hn+321OcM9ArZJo3VjdFhwU8Z/zCeSZX/l FDQFgebhz68NQgj0l47uZbhfOc5Rw8uWJnQhKbpBBpgxHo7bskljkJqsj jjxg2PwM+5ofF7tC3VB/DyaLpkPong/rgRlb/fEuIGw6E+HjgT+S63ovU g==; X-CSE-ConnectionGUID: fz1iyEChRgivj0WuDUOC6A== X-CSE-MsgGUID: zE11X43+SWGtsSl6cj6ZLg== X-IronPort-AV: E=Sophos;i="6.12,266,1728943200"; d="scan'208";a="39911625" Received: from vmailcow01.tq-net.de ([10.150.86.48]) by mx1.tq-group.com with ESMTP; 07 Nov 2024 13:49:58 +0100 X-CheckPoint: {672CB776-13-90CD5875-E0265C0B} X-MAIL-CPID: 5630CB42D6C5881B1DECA5A00818660C_3 X-Control-Analysis: str=0001.0A682F28.672CB776.00BE,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id EF3F5160A41; Thu, 7 Nov 2024 13:49:50 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ew.tq-group.com; s=dkim; t=1730983793; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=rVq9b6pcFOHREaNN9fex606hOk1inbkqT9uhtMs6aJA=; b=OvymNaxWbHJgVj9vxdqQ3uuum2UfzC+CLSQ4JBH6IvTuTH3US6xD+qIv0XOF+o/LtsLBHk 4xmrj5mW0Patp4egGwKF94z9YzISIS6JSBZvEaepmEaHbScQ6pQA6kYoUDjQHu1Z1tZWSV WGhp4dOkHMPiOsoTSIpeOUW2W1RfAdzLT8ERF8XRAShA1A4J94+TfaSEs8bp6GqvuKsfV4 CFRhysC0O0JzhXJY5oGI38zGYyz/eUsGZkVsHdgNQEzDe0y8kdVWwvTV12HCuWtWh7D3w9 D6PmzK9QdSH9RqRtdLTInBw4/J2dm8xY+x2avdzr8Pe1sBjdm9ENQRULKPTDiA== From: Alexander Stein To: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, Pengfei Li Cc: joao.goncalves@toradex.com, frieder.schrempf@kontron.de, marex@denx.de, hvilleneuve@dimonoff.com, peng.fan@nxp.com, m.othacehe@gmail.com, mwalle@kernel.org, Max.Merchel@ew.tq-group.com, hiago.franco@toradex.com, tharvey@gateworks.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 Subject: Re: [PATCH 2/3] arm64: dts: freescale: Add i.MX91 dtsi support Date: Thu, 07 Nov 2024 13:49:50 +0100 Message-ID: <2350046.ElGaqSPkdT@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20241108022703.1877171-3-pengfei.li_1@nxp.com> References: <20241108022703.1877171-1-pengfei.li_1@nxp.com> <20241108022703.1877171-3-pengfei.li_1@nxp.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241107_045003_569405_CE4EAAE6 X-CRM114-Status: GOOD ( 19.78 ) 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 Hi, thanks for putting me on CC. Am Freitag, 8. November 2024, 03:27:02 CET schrieb Pengfei Li: > 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. >=20 > 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 >=20 > Signed-off-by: Pengfei Li > Reviewed-by: Frank Li > --- > arch/arm64/boot/dts/freescale/imx91-pinfunc.h | 770 ++++++++++++++++++ > arch/arm64/boot/dts/freescale/imx91.dtsi | 66 ++ > 2 files changed, 836 insertions(+) > create mode 100644 arch/arm64/boot/dts/freescale/imx91-pinfunc.h > create mode 100644 arch/arm64/boot/dts/freescale/imx91.dtsi >=20 > diff --git a/arch/arm64/boot/dts/freescale/imx91-pinfunc.h b/arch/arm64/b= oot/dts/freescale/imx91-pinfunc.h > new file mode 100644 > index 000000000000..bc58ce2102b2 > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx91-pinfunc.h =46WIW this is a 1:1 copy from downstream kernel > diff --git a/arch/arm64/boot/dts/freescale/imx91.dtsi b/arch/arm64/boot/d= ts/freescale/imx91.dtsi > new file mode 100644 > index 000000000000..a9f4c1fe61cc > --- /dev/null > +++ b/arch/arm64/boot/dts/freescale/imx91.dtsi > @@ -0,0 +1,66 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright 2024 NXP > + */ > + > +#include "imx91-pinfunc.h" > +#include "imx93.dtsi" > + > +&{/thermal-zones/cpu-thermal/cooling-maps/map0} { > + cooling-device =3D > + <&A55_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>; > +}; > + > +&clk { > + compatible =3D "fsl,imx91-ccm"; > +}; > + > +&eqos { > + clocks =3D <&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 =3D <&clk IMX91_CLK_ENET_TIMER>, > + <&clk IMX91_CLK_ENET1_QOS_TSN>; > + assigned-clock-parents =3D <&clk IMX93_CLK_SYS_PLL_PFD1_DIV2>, > + <&clk IMX93_CLK_SYS_PLL_PFD0_DIV2>; Is it just me or is the alignment of new lines not matching? > +}; > + > +&fec { > + clocks =3D <&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 =3D <&clk IMX91_CLK_ENET_TIMER>, > + <&clk IMX91_CLK_ENET2_REGULAR>; > + assigned-clock-parents =3D <&clk IMX93_CLK_SYS_PLL_PFD1_DIV2>, > + <&clk IMX93_CLK_SYS_PLL_PFD0_DIV2>; Here as well: Is it just me or is the alignment of new lines not matching? > + assigned-clock-rates =3D <100000000>, <250000000>; > +}; > + > +&i3c1 { > + clocks =3D <&clk IMX93_CLK_BUS_AON>, > + <&clk IMX93_CLK_I3C1_GATE>, > + <&clk IMX93_CLK_DUMMY>; > +}; > + > +&i3c2 { > + clocks =3D <&clk IMX93_CLK_BUS_WAKEUP>, > + <&clk IMX93_CLK_I3C2_GATE>, > + <&clk IMX93_CLK_DUMMY>; > +}; > + > +&tmu { > + status =3D "disabled"; Why does the TMU needs to be disabled instead of deleted? > +}; > + > +/* i.MX91 only has one A core */ > +/delete-node/ &A55_1; > + > +/* i.MX91 not has cm33 */ > +/delete-node/ &cm33; > + > +/* i.MX91 not has power-domain@44461800 */ > +/delete-node/ &mlmix; >=20 Shouldn't the following node also be removed? * mipi_csi * dsi * lvds_bridge * lcdif_to_dsi * lcdif_to_ldb Also in downstream kernel IMX91_CLK_MEDIA_AXI, which is IMX93_CLK_MEDIA_AXI upstream, is set to 200 MHz. Is this applicable here as well? Best regards, Alexander =2D-=20 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/