On Mon, Jun 25, 2018 at 2:27 PM Fabio Estevam <festevam@xxxxxxxxx> wrote: > > Hi Rob, > > On Mon, Jun 25, 2018 at 5:21 PM, Rob Herring <robh@xxxxxxxxxx> 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