From mboxrd@z Thu Jan 1 00:00:00 1970 MIME-Version: 1.0 Date: Tue, 26 Jun 2018 08:24:01 -0600 References: <1529693129-29817-1-git-send-email-festevam@gmail.com> <20180625202109.GA22892@rob-hp-laptop> In-Reply-To: Message-ID: Subject: Re: [PATCH v2 1/2] dt-bindings: fsl-imx-sahara: Add i.MX51 as a supported SoC From: Rob Herring Content-Type: text/plain; charset="UTF-8" To: Fabio Estevam Cc: Herbert Xu , "open list:HARDWARE RANDOM NUMBER GENERATOR CORE" , devicetree@vger.kernel.org, =?UTF-8?Q?Horia_Geant=C4=83?= , Fabio Estevam List-ID: On Mon, Jun 25, 2018 at 2:27 PM Fabio Estevam wrote: > > Hi Rob, > > On Mon, Jun 25, 2018 at 5:21 PM, Rob Herring wrote: > > > Looks like imx51 should be a fallback and you can drop the driver > > change. > > I thought about that too. > > If I do like this in imx51.dtsi: > > compatible = "fsl,imx51-sahara", "fsl,imx53-sahara"; > > Then the driver works just fine. > > I was not sure if this was OK or not because imx51 was introduced > prior than imx53 was launched. > > Would it be OK to use: compatible = "fsl,imx51-sahara", "fsl,imx53-sahara"; ? Yes, but the order should be reversed as it should be most specific and newest first. Rob