On Fri, Jun 21, 2013 at 08:54:53AM +0400, Alexander Shiyan wrote: > > On Thu, Jun 20, 2013 at 04:50:14PM +0200, Markus Pargmann wrote: > > > + cpus { > > > + #size-cells = <0>; > > > + #address-cells = <1>; > > > + > > > + cpu@0 { > > > + device_type = "cpu"; > > > + compatible = "fsl,imx27", "arm,arm926ejs"; > > > > From what Documentation/devicetree/bindings/arm/cpus.txt tells, it > > should be "arm,arm926". Also, why do you put "fsl,imx27" there? > > "imx27" is a SoC name not cpu core. > > I think Markus take this ARM property from one of existing DTS. > > shc@shc /home/git/linux-mx27/arch/arm/boot/dts $ grep arm926 *.dtsi > at91sam9260.dtsi: compatible = "arm,arm926ejs"; > at91sam9263.dtsi: compatible = "arm,arm926ejs"; > at91sam9g45.dtsi: compatible = "arm,arm926ejs"; > at91sam9n12.dtsi: compatible = "arm,arm926ejs"; > at91sam9x5.dtsi: compatible = "arm,arm926ejs"; > imx23.dtsi: compatible = "arm,arm926ejs"; > imx28.dtsi: compatible = "arm,arm926ejs"; > lpc32xx.dtsi: compatible = "arm,arm926ejs"; > s3c2416.dtsi: compatible = "arm,arm926ejs"; > spear3xx.dtsi: compatible = "arm,arm926ejs"; > spear600.dtsi: compatible = "arm,arm926ejs"; > wm8505.dtsi: compatible = "arm,arm926ejs"; > > So, documentation need to be updated or these values should be fixed. > Another solution is specify compatible string as: > compatible = "arm,arm926ejs", "arm,arm926"; > What you think about this?
I assume that the compatible string in the binding doc has been reviewed and agreed by people, so we should fix the existing users before kernel starts using it to for matching something. Lorenzo, comment? Shawn _______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss