[PATCH 5/6] ARM: dts: imx27 cpufreq-cpu0 frequencies

On Fri, Jun 21, 2013 at 08:54:53AM +0400, Alexander Shiyan wrote:

quoted

On Thu, Jun 20, 2013 at 04:50:14PM +0200, Markus Pargmann wrote:

quoted

+	cpus {
+		#size-cells = <0>;
+		#address-cells = <1>;
+
+		cpu at 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 at 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