On Mon, Aug 29, 2016 at 3:16 PM, Arnd Bergmann [off-list ref] wrote:

On Wednesday 24 August 2016, Linus Walleij wrote:

quoted

+
+ebi2 at 1a100000 {
+     compatible = "qcom,apq8060-ebi2";
+     #address-cells = <2>;
+     #size-cells = <1>;
+     ranges = <0 0x0 0x1a800000 0x00800000>,
+              <1 0x0 0x1b000000 0x00800000>,
+              <2 0x0 0x1b800000 0x00800000>,
+              <3 0x0 0x1d000000 0x00800000>,
+              <4 0x0 0x1c800000 0x00800000>,
+              <5 0x0 0x1c000000 0x00800000>;
+     reg = <0x1a100000 0x1000>, <0x1a110000 0x1000>;
+     reg-names = "ebi2", "xmem";

...

quoted

+     foo-ebi2 at 1b800000 {
+             compatible = "foo";
+             reg = <2 0x1b800000 0x100>;
+             (...)
+     };
+};

In this example, the addresses don't match: I assume the child
node should be listed as

        foo-ebi at 2,0 {
                reg = <2 0 0x100>;
        };

instead.

Yeah it's a plain mistake, sorry..

Noticed it too when following up in the other thread.

Yours,
Linus Walleij