On 23/04/13 03:27, Lorenzo Pieralisi wrote:
This patch updates the in-kernel dts files according to the latest cpus
and cpu bindings updates for ARM.

Signed-off-by: Lorenzo Pieralisi <[email protected]>
---
  arch/arm/boot/dts/wm8505.dtsi | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/wm8505.dtsi b/arch/arm/boot/dts/wm8505.dtsi
index e74a1c0..a470808 100644
--- a/arch/arm/boot/dts/wm8505.dtsi
+++ b/arch/arm/boot/dts/wm8505.dtsi
@@ -13,7 +13,7 @@
cpus {
                cpu@0 {
-                       compatible = "arm,arm926ejs";
+                       compatible = "arm,arm926";
                };
        };
The more I look at this, the more wrong it is :/

From the new binding documentation,

+       A cpus node must define the following properties:
+
+       - #address-cells
+               Usage: required
+               Value type: <u32>
+               Definition: must be set to 1 for 32-bit systems and 2 for
+                           64-bit systems
+       - #size-cells
+               Usage: required
+               Value type: <u32>
+               Definition: must be set to 0

...

+- cpu node
+
+       Description: Describes a CPU in an ARM based system
+
+       PROPERTIES
+
+       - device_type
+               Usage: required
+               Value type: <string>
+               Definition: must be "cpu"

Three required properties that aren't present in the patch.

cpus {
    #size-cells = <0>;
    #address-cells = <1>;

    cpu {
        device_type = "cpu"
        compatible = "arm,arm926";
    };
};

Regards
Tony P
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to