Hi Frank, On Fri, 2014-04-25 at 17:28 -0700, Frank Rowand wrote: > On 4/23/2014 6:19 AM, Ivan T. Ivanov wrote: <snip> > > spmi { > > compatible = "qcom,spmi-pmic-arb"; > > reg-names = "core", "intr", "cnfg"; > > reg = <0xfc4cf000 0x1000>, > > <0xfc4cb000 0x1000>, > > <0xfc4ca000 0x1000>; > > > > interrupt-names = "periph_irq"; > > interrupts = <0 190 0>; > > > > qcom,ee = <0>; > > qcom,channel = <0>; > > > > #address-cells = <2>; > > #size-cells = <0>; > > > > interrupt-controller; > > #interrupt-cells = <4>; > > > > pm8941@0 { > > compatible = "qcom,pm8941"; > > reg = <0x0 SPMI_USID>; > > > > #address-cells = <1>; > > #size-cells = <0>; > > > > revid@100 { > > compatible = "qcom,qpnp-revid"; > > reg = <0x100 0x100>; This should be just reg = <0x100>; > > }; > > }; > > > > pm8841@4 { > > ^^^^^^^^ typo nit - that should be pm8941@4. > The nit does not change what you reported though. > > > compatible = "qcom,pm8941"; Actually this one is incorrect, it should be "qcom,pm8841", but as you say it doesn't make difference in the end result. > > reg = <0x4 SPMI_USID>; > > > > #address-cells = <1>; > > #size-cells = <0>; > > > > revid@100 { > > compatible = "qcom,qpnp-revid"; > > reg = <0x100 0x100>; also here. Regards, Ivan -- 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