diff for duplicates of <9407200.bOeiVUkGKT@wuerfel> diff --git a/a/1.txt b/N1/1.txt index 46b0e0b..3d0cfc9 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ On Friday 13 May 2016 14:26:28 Alexander Shiyan wrote: > +&bus { -> + flash: root at 00000000 { +> + flash: root@00000000 { > + compatible = "cfi-flash"; > + reg = <0 0x00000000 0x01000000>; > + bank-width = <2>; @@ -39,7 +39,7 @@ all of the above. All child devices seem to be in the 0x80000000-0x90000000 range, maybe set the ranges property accordingly? -> + clks: clks at 80000000 { +> + clks: clks@80000000 { > + #clock-cells = <1>; > + compatible = "cirrus,clps711x-clk"; > + reg = <0x80000000 0xc000>; @@ -50,7 +50,7 @@ Please fix the compatible strings for all devices to not have 'x' for wildcards. Normally you'd use the name of hte first chip to have this particular IP block. -> + intc: intc at 80000000 { +> + intc: intc@80000000 { > + compatible = "cirrus,clps711x-intc"; > + reg = <0x80000000 0x4000>; > + interrupt-controller; @@ -60,21 +60,21 @@ first chip to have this particular IP block. Better make the register ranges non-overlapping. This appears to start at the same place as the 'clks' node. -> + porta: gpio at 80000000 { +> + porta: gpio@80000000 { > + compatible = "cirrus,clps711x-gpio"; > + reg = <0x80000000 0x1 0x80000040 0x1>; > + gpio-controller; > + #gpio-cells = <2>; > + }; > + -> + portb: gpio at 80000001 { +> + portb: gpio@80000001 { > + compatible = "cirrus,clps711x-gpio"; > + reg = <0x80000001 0x1 0x80000041 0x1>; > + gpio-controller; > + #gpio-cells = <2>; > + }; > + -> + portc: gpio at 80000002 { +> + portc: gpio@80000002 { > + compatible = "cirrus,clps711x-gpio"; > + reg = <0x80000002 0x1 0x80000042 0x1>; > + gpio-controller; @@ -82,7 +82,7 @@ to start at the same place as the 'clks' node. > + status = "disabled"; > + }; > + -> + portd: gpio at 80000003 { +> + portd: gpio@80000003 { > + compatible = "cirrus,clps711x-gpio"; > + reg = <0x80000003 0x1 0x80000043 0x1>; > + gpio-controller; @@ -93,7 +93,7 @@ These are all just single bytes. My guess is that there is actually one IP block that contains all the gpios, not four identical blocks. -> + bus: bus at 80000180 { +> + bus: bus@80000180 { > + #address-cells = <2>; > + #size-cells = <1>; > + compatible = "cirrus,clps711x-bus", "simple-bus"; @@ -104,3 +104,7 @@ If it has registers, it's not a 'simple-bus'. Is this an external bus controller perhaps? Arnd +-- +To unsubscribe from this list: send the line "unsubscribe devicetree" in +the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 53eb493..18122c8 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,14 +1,23 @@ "ref\01463138788-5390-1-git-send-email-shc_work@mail.ru\0" "ref\01463138788-5390-4-git-send-email-shc_work@mail.ru\0" - "From\0arnd@arndb.de (Arnd Bergmann)\0" - "Subject\0[PATCH 3/3] ARM: dts: clps711x: Add basic Cirrus Logic CDB89712 Development board\0" + "ref\01463138788-5390-4-git-send-email-shc_work-JGs/UdohzUI@public.gmane.org\0" + "From\0Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>\0" + "Subject\0Re: [PATCH 3/3] ARM: dts: clps711x: Add basic Cirrus Logic CDB89712 Development board\0" "Date\0Fri, 13 May 2016 14:00:35 +0200\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org\0" + "Cc\0Alexander Shiyan <shc_work-JGs/UdohzUI@public.gmane.org>" + Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org> + Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org> + Russell King <linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org> + Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> + " Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>\0" "\00:1\0" "b\0" "On Friday 13 May 2016 14:26:28 Alexander Shiyan wrote:\n" "> +&bus {\n" - "> +\tflash: root at 00000000 {\n" + "> +\tflash: root@00000000 {\n" "> +\t\tcompatible = \"cfi-flash\";\n" "> +\t\treg = <0 0x00000000 0x01000000>;\n" "> +\t\tbank-width = <2>;\n" @@ -47,7 +56,7 @@ "All child devices seem to be in the 0x80000000-0x90000000\n" "range, maybe set the ranges property accordingly?\n" "\n" - "> +\t\tclks: clks at 80000000 {\n" + "> +\t\tclks: clks@80000000 {\n" "> +\t\t\t#clock-cells = <1>;\n" "> +\t\t\tcompatible = \"cirrus,clps711x-clk\";\n" "> +\t\t\treg = <0x80000000 0xc000>;\n" @@ -58,7 +67,7 @@ "have 'x' for wildcards. Normally you'd use the name of hte\n" "first chip to have this particular IP block.\n" "\n" - "> +\t\tintc: intc at 80000000 {\n" + "> +\t\tintc: intc@80000000 {\n" "> +\t\t\tcompatible = \"cirrus,clps711x-intc\";\n" "> +\t\t\treg = <0x80000000 0x4000>;\n" "> +\t\t\tinterrupt-controller;\n" @@ -68,21 +77,21 @@ "Better make the register ranges non-overlapping. This appears\n" "to start at the same place as the 'clks' node.\n" "\n" - "> +\t\tporta: gpio at 80000000 {\n" + "> +\t\tporta: gpio@80000000 {\n" "> +\t\t\tcompatible = \"cirrus,clps711x-gpio\";\n" "> +\t\t\treg = <0x80000000 0x1 0x80000040 0x1>;\n" "> +\t\t\tgpio-controller;\n" "> +\t\t\t#gpio-cells = <2>;\n" "> +\t\t};\n" "> +\n" - "> +\t\tportb: gpio at 80000001 {\n" + "> +\t\tportb: gpio@80000001 {\n" "> +\t\t\tcompatible = \"cirrus,clps711x-gpio\";\n" "> +\t\t\treg = <0x80000001 0x1 0x80000041 0x1>;\n" "> +\t\t\tgpio-controller;\n" "> +\t\t\t#gpio-cells = <2>;\n" "> +\t\t};\n" "> +\n" - "> +\t\tportc: gpio at 80000002 {\n" + "> +\t\tportc: gpio@80000002 {\n" "> +\t\t\tcompatible = \"cirrus,clps711x-gpio\";\n" "> +\t\t\treg = <0x80000002 0x1 0x80000042 0x1>;\n" "> +\t\t\tgpio-controller;\n" @@ -90,7 +99,7 @@ "> +\t\t\tstatus = \"disabled\";\n" "> +\t\t};\n" "> +\n" - "> +\t\tportd: gpio at 80000003 {\n" + "> +\t\tportd: gpio@80000003 {\n" "> +\t\t\tcompatible = \"cirrus,clps711x-gpio\";\n" "> +\t\t\treg = <0x80000003 0x1 0x80000043 0x1>;\n" "> +\t\t\tgpio-controller;\n" @@ -101,7 +110,7 @@ "actually one IP block that contains all the gpios, not\n" "four identical blocks.\n" "\n" - "> +\t\tbus: bus at 80000180 {\n" + "> +\t\tbus: bus@80000180 {\n" "> +\t\t\t#address-cells = <2>;\n" "> +\t\t\t#size-cells = <1>;\n" "> +\t\t\tcompatible = \"cirrus,clps711x-bus\", \"simple-bus\";\n" @@ -111,6 +120,10 @@ "If it has registers, it's not a 'simple-bus'. Is this an external\n" "bus controller perhaps?\n" "\n" - "\tArnd" + "\tArnd\n" + "--\n" + "To unsubscribe from this list: send the line \"unsubscribe devicetree\" in\n" + "the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -b5053c66d122a4e0b47f7c8eafd726852382df39961eaf1afdc4ecbb2ef224d7 +fd0030bc1b5bf2afdd041e039914a14362dd408f0f56771713f650e8c4dadcbf
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.