From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawnguo@kernel.org (Shawn Guo) Date: Thu, 28 Jun 2018 11:21:59 +0800 Subject: [PATCH 1/3] ARM: dts: imx53: Add 'ARM platform' node In-Reply-To: References: <1529329939-14117-1-git-send-email-festevam@gmail.com> <20180628013530.GE22683@dragon> Message-ID: <20180628032158.GG22683@dragon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jun 28, 2018 at 12:02:10AM -0300, Fabio Estevam wrote: > On Wed, Jun 27, 2018 at 10:35 PM, Shawn Guo wrote: > > >> diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi > >> index 1a7a7bb..a70fbf4 100644 > >> --- a/arch/arm/boot/dts/imx53.dtsi > >> +++ b/arch/arm/boot/dts/imx53.dtsi > >> @@ -672,6 +672,11 @@ > >> status = "disabled"; > >> }; > >> > >> + arm_platform: arm-platform at 63fa0000 { > >> + compatible = "fsl,imx53-arm-plat", "fsl,imx51-arm-plat"; > > > > Don't we need a bindings for that? > > I thought this is the same case as "fsl,imx51-src" where we only use > it inside arch/arm/mach-imx/ and there is also no bindings doc for it. Isn't 'ARM platform' something ARM core specific rather than i.MX? > > That's why I did not add an entry for the bindings doc. > > If you think that a binding doc is needed, in which directory should > it be placed? We can ask suggestion from DT folks. Shawn