On 21.05.14, 20:24, Stephen Boyd wrote: > On 05/21/14 07:57, Georgi Djakov wrote: >> + >> + L2: l2-cache { >> + compatible = "qcom,arch-cache"; >> + cache-level = <2>; >> + interrupts = <0 2 0x4>; > > Let's leave out interrupts until the binding is accepted. > >> + qcom,saw = <&saw_l2>; >> + }; >> + }; >> + >> + cpu-pmu { >> + compatible = "qcom,krait-pmu"; >> + interrupts = <1 7 0xf04>; >> + }; >> + >> + soc: soc { >> + #address-cells = <1>; >> + #size-cells = <1>; >> + ranges; >> + compatible = "simple-bus"; >> + >> + intc: interrupt-controller@f9000000 { >> + compatible = "qcom,msm-qgic2"; >> + interrupt-controller; >> + #interrupt-cells = <3>; >> + reg = <0xf9000000 0x1000>, >> + <0xf9002000 0x1000>; >> + }; >> + >> + timer { >> + compatible = "arm,armv7-timer"; >> + interrupts = <1 2 0xf08>, >> + <1 3 0xf08>, >> + <1 4 0xf08>, >> + <1 1 0xf08>; >> + clock-frequency = <19200000>; >> + }; > > Please move this timer node out of the soc container below the pmu. > Will do. Thanks for the comments! BR, Georgi -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html