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 CA2C2D2CE0D for ; Tue, 22 Oct 2024 17:19:10 +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=dFIIm5rhxs9qT/MajtS4EYkcanyHeA4sGE0hYbhvTFc=; b=yh/0DEJto5aAbvx+u8ciBqQAb/ 71TBhOxz5xLDpN/tkaCB8B0ncfVHrPbH5uiFTa/HH4k6ze3QSYP670mDKzRv93Q845ub1H9m4Kgx5 TGGfIMQ2iPnT2nx6qPo+PVf1wV5chF1qjVCqy5Xawnhsf73SasBn3514OfeSOqxfajubs5eliqXVx ydL50Pt4bYYOv7NFoBuV2SWQ9a2L06gTWUMIliu2BMD9J170/mKG+EpCkaspOKnpCW0I/FGlJqkqP XJLlSl5JZsj1gcE4swAQ4Uu57d3dAtxLJIZ8PvzIPD6+70w47MZTGUZajnjEjbyGQKZ/Q046DiRwt bxcyAs5A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t3IX7-0000000Bcfy-3QgQ; Tue, 22 Oct 2024 17:19:01 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t3I9x-0000000BYan-1C4H for linux-arm-kernel@lists.infradead.org; Tue, 22 Oct 2024 16:55:06 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 3E0F45C5BAB; Tue, 22 Oct 2024 16:55:00 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E72B2C4CEC3; Tue, 22 Oct 2024 16:55:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1729616104; bh=dFIIm5rhxs9qT/MajtS4EYkcanyHeA4sGE0hYbhvTFc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EJny844qGoatCql5qUU/Q4fh8DIy6u3lc7JLAMq5mILuxu7Qg4uY1+ZFPCeHdARm5 1jify6tlyKKhXrW1Fy7cpaX3pbshaDsAsHui7wnQMneSf2VvkNq+Nvv4/cWDIWW383 bAeLYW8zFI+RNYM8AoBn+ZB/yVXHH6WjJCUw2gQ4PFCmj/jOkMmQ84133NxzSTA7l+ QJfelq4HrbwvM+TQqngohC8q5DESZysL8jAPvDEZi9hoAqvsC8XG8K7RaQF2pi9IxM nb8ZL48xzl3BibX88siWOQMGJweVkFS6ZA4L1MWRglx7R/5LkyZ/kbQ64myclut72a 1hEdOsDtKdV5Q== Date: Tue, 22 Oct 2024 17:54:59 +0100 From: Conor Dooley To: Jason-Hsu Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, joel@jms.id.au, andrew@codeconstruct.com.au, patrick@stwcx.xyz, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org, yang.chen@quantatw.com, jerry.lin@quantatw.com Subject: Re: [PATCH 2/2] ARM: dts: aspeed: ventura: add Meta Ventura BMC Message-ID: <20241022-prayer-unmanaged-86ea24caf242@spud> References: <20241022021724.2322506-1-jasonhell19@gmail.com> <20241022-clean-rebate-e986d6a82a05@spud> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="fGRR7GdKJMsqfzS2" Content-Disposition: inline In-Reply-To: <20241022-clean-rebate-e986d6a82a05@spud> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241022_095505_391480_8DF06625 X-CRM114-Status: GOOD ( 13.27 ) 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 --fGRR7GdKJMsqfzS2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Oct 22, 2024 at 05:51:45PM +0100, Conor Dooley wrote: > On Tue, Oct 22, 2024 at 10:17:24AM +0800, Jason-Hsu wrote: > > Add Linux device tree related to Meta(Facebook) Ventura specific device= s connected to BMC(AST2600) SoC. > > Add subject prefix for the patch. > >=20 > > Signed-off-by: Jason-Hsu >=20 > The indentation in here is very inconsistent. Please fix that. Also, I don't know what the coding style in aspeed in particular is, but the node ordering seems very random here. Could you sort nodes in the dts alphanumerically as the dts coding style suggests? --fGRR7GdKJMsqfzS2 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZxfY4wAKCRB4tDGHoIJi 0g9lAP43/yhfmMpPGE/UTpJbFPS2SOc0ZEN/nLtKSvc25ObaswD+LyRMqnsq87TE r6zdEguM3482DqDtUmYg4SQEYytUVgE= =GZrq -----END PGP SIGNATURE----- --fGRR7GdKJMsqfzS2--