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 8368AC4707B for ; Thu, 18 Jan 2024 15:57:29 +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-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To: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=dnhI356niaPjaCOwpLLPTYtxxgHzUIaUTHHS6lt2POg=; b=H2fSxXQvA2FZrTlUsJCtNM17XW Xw53yjz/Ks/LIF+8rK47jA60QTpsV3IK0llMBEgNIzwhOP+1ARjjzcYVFJSTF0E+TOwOJxFiJcsN6 I1VXauQwWSuVZtbqCwY3CXyNjsKTJse41oE/NvS7FGAptdxwgC6yvC3Im3Sk2q93RS2bDB8rc4w/V 4hC6KNJLttIZiISpgin26VASWqRzg/6+XvsTs3MqPl6YNl7liXOXU9gvQXLbvRN51Euu65AYH172r emAHSz9vDmRRyM2lHBDIEjlevGUd4eC4/GDlUnfIEDcAqxx6CIYYiPzLQOJsKUn60GQm+OoCIxJ09 yYgZVeZw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rQUlF-003Aj6-24; Thu, 18 Jan 2024 15:56:57 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rQUlA-003Ahq-1f for linux-arm-kernel@lists.infradead.org; Thu, 18 Jan 2024 15:56:54 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by ams.source.kernel.org (Postfix) with ESMTP id 01CF0B818E9; Thu, 18 Jan 2024 15:56:51 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C5ED5C43390; Thu, 18 Jan 2024 15:56:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1705593410; bh=5EROv/PliSUdPC4ViG2nwtB6PhsDjdlT/9YdwUIjIdo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=CLMPK8gcRLNqG2wGliNsZRieedsidby49NUPFFHxvHBMPjxS0OqW54NtBHDiY3I+y w/BAcm620hR+yJiYRUl+nRCeUT6yLscWFUKhp3zil+XtrqF2kM7QF7fGDTIqBpq8ki YQmMPEujp0HnbJErCfREk9VO+GpyRVNIhK2VfS/uildXFJ5qu10qTy9M4hav6v/ZAY H46DItUhNKml38G2FFHQUeKwUhRr/32CmlpWeHJEvT8vOmz87vt2orf+52RZGqWmNA H/HjtLJ2BDIL4aZhuImk1XDElAGIcW5aQLtyyFg0AShtf99DshxQj91bViakWGSAna 2WyEi54gzCHRg== Date: Thu, 18 Jan 2024 15:56:44 +0000 From: Conor Dooley To: Mathieu Othacehe Cc: Primoz Fiser , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Li Yang , Stefan Wahren , Christoph Stoidner , Wadim Egorov , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, upstream@lists.phytec.de Subject: Re: [PATCH v2 2/2] arm64: dts: imx93-phycore-segin: Add Phytec i.MX93 Segin Message-ID: <20240118-excavate-bridged-7378c8ebc16a@spud> References: <20240117074911.7425-1-othacehe@gnu.org> <20240117074911.7425-3-othacehe@gnu.org> <20240118-regretful-viewer-8d7dfc7a0802@spud> <87jzo6vjtg.fsf@gnu.org> MIME-Version: 1.0 In-Reply-To: <87jzo6vjtg.fsf@gnu.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240118_075652_842720_862B4235 X-CRM114-Status: GOOD ( 21.53 ) 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: multipart/mixed; boundary="===============2522527046561909627==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2522527046561909627== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="NQNMzSLYhR4PxVsu" Content-Disposition: inline --NQNMzSLYhR4PxVsu Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jan 18, 2024 at 02:43:07PM +0100, Mathieu Othacehe wrote: >=20 > Hey, >=20 > > Please do not order properties alphabetically. Instead, please read > > the new documentation on property ordering that makes explicit what > > has just been convention until now: > > https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/tree/Doc= umentation/devicetree/bindings/dts-coding-style.rst?h=3Dfor-next&id=3D83a36= 8a3fc8ae8538bccb713dc0cae9eacc04790#n112 >=20 > Thanks for the link. >=20 > I have a question though. Regarding that section: >=20 > --8<---------------cut here---------------start------------->8--- > /* SD-Card */ > &usdhc2 { > pinctrl-names =3D "default", "state_100mhz", "state_200mhz"; > pinctrl-0 =3D <&pinctrl_usdhc2_default>, <&pinctrl_usdhc2_cd>; > pinctrl-1 =3D <&pinctrl_usdhc2_100mhz>, <&pinctrl_usdhc2_cd>; > pinctrl-2 =3D <&pinctrl_usdhc2_200mhz>, <&pinctrl_usdhc2_cd>; > bus-width =3D <4>; > cd-gpios =3D <&gpio3 00 GPIO_ACTIVE_LOW>; > no-sdio; > no-mmc; > vmmc-supply =3D <®_usdhc2_vmmc>; > status =3D "okay"; > }; > --8<---------------cut here---------------end--------------->8--- >=20 > The documentation states: >=20 > --8<---------------cut here---------------start------------->8--- > Order of Properties in Device Node > ---------------------------------- >=20 > The following order of properties in device nodes is preferred: >=20 > 1. "compatible" > 2. "reg" > 3. "ranges" > 4. Standard/common properties (defined by common bindings, e.g. without > vendor-prefixes) > 5. Vendor-specific properties > 6. "status" (if applicable) > 7. Child nodes, where each node is preceded with a blank line > --8<---------------cut here---------------end--------------->8--- >=20 > All of the properties in my example are falling into the "4" category I > guess, except for "status" that should come last. Now, how am I supposed > to order those properties? I had a look to other IMX device trees and it > is hard to establish a pattern. Pinctrl first, then alphabetical order? > Anything else? If that is the established order for imx devicetrees, then yeah, I would follow that ordering. From my own quick check of recently added boards, that is the way things seem to be. Cheers, Conor. --NQNMzSLYhR4PxVsu Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZalKPAAKCRB4tDGHoIJi 0izkAP9EwNLAq04HFpfgHUVYT/Na89yK5jZy0i4ZY1N9tIUjfQEA8dSFHsrQeZTB wUL5SOyLaCYmAv6hVCqWanaLFWQjuQI= =AlYo -----END PGP SIGNATURE----- --NQNMzSLYhR4PxVsu-- --===============2522527046561909627== 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 --===============2522527046561909627==--