diff for duplicates of <54AAC4B2.8040401@free-electrons.com> diff --git a/a/1.txt b/N1/1.txt index 69a93fa..5bdccbb 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -5,7 +5,7 @@ On 31/12/2014 11:32, Thomas Petazzoni wrote: > > On Sat, 27 Dec 2014 12:00:35 +0100, Gregory CLEMENT wrote: > ->> + spi-flash at 0 { +>> + spi-flash@0 { >> + #address-cells = <1>; >> + #size-cells = <1>; >> + compatible = "st,m25p128"; @@ -19,11 +19,11 @@ is a M25P128-VMF6P. As there was no 'B' in the part number then it was a 130nm device which is limited to 50MHz. > ->> + i2c at 11000 { +>> + i2c@11000 { >> + status = "okay"; >> + clock-frequency = <100000>; >> + ->> + pca9555_0: pca9555 at 20 { +>> + pca9555_0: pca9555@20 { >> + compatible = "nxp,pca9555"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pca0_pins>; @@ -36,7 +36,7 @@ is a M25P128-VMF6P. As there was no 'B' in the part number then it was a >> + reg = <0x20>; >> + }; >> + ->> + pca9555_1: pca9555 at 21 { +>> + pca9555_1: pca9555@21 { >> + compatible = "nxp,pca9555"; >> + pinctrl-names = "default"; >> + interrupt-parent = <&gpio0>; @@ -58,7 +58,7 @@ is a M25P128-VMF6P. As there was no 'B' in the part number then it was a Yes it was a left over of from a previous version, I will remove it. ->> + ethernet at 70000 { +>> + ethernet@70000 { >> + status = "okay"; >> + phy = <&phy1>; >> + phy-mode = "rgmii-id"; @@ -68,12 +68,12 @@ Yes it was a left over of from a previous version, I will remove it. > > One too many empty new line. > ->> + mdio at 72004 { ->> + phy0: ethernet-phy at 0 { +>> + mdio@72004 { +>> + phy0: ethernet-phy@0 { >> + reg = <0>; >> + }; >> + ->> + phy1: ethernet-phy at 1 { +>> + phy1: ethernet-phy@1 { >> + reg = <1>; >> + }; >> + }; @@ -88,20 +88,20 @@ Yes it was a left over of from a previous version, I will remove it. > > So maybe we should have: > -> ethernet at 30000 { +> ethernet@30000 { > phy = <&phy1>; > }; > -> ethernet at 70000 { +> ethernet@70000 { > phy = <&phy0>; > }; > -> mdio at 72004 { -> phy0: ethernet-phy at 1 { +> mdio@72004 { +> phy0: ethernet-phy@1 { > reg = <1>; > }; > -> phy1: ethernet-phy at 0 { +> phy1: ethernet-phy@0 { > reg = <0>; > }; > }; @@ -111,18 +111,18 @@ Yes it was a left over of from a previous version, I will remove it. > ->> + sata at a8000 { +>> + sata@a8000 { >> + nr-ports = <2>; >> + status = "okay"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + ->> + sata0: sata-port at 0 { +>> + sata0: sata-port@0 { >> + reg = <0>; >> + target-supply = <®_5v_sata0>; >> + }; >> + ->> + sata1: sata-port at 1 { +>> + sata1: sata-port@1 { >> + reg = <1>; >> + target-supply = <®_5v_sata1>; >> + }; @@ -152,18 +152,18 @@ but it also should be stable. > >> + }; >> + ->> + sata at e0000 { +>> + sata@e0000 { >> + nr-ports = <2>; >> + status = "okay"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + ->> + sata2: sata-port at 0 { +>> + sata2: sata-port@0 { >> + reg = <0>; >> + target-supply = <®_5v_sata2>; >> + }; >> + ->> + sata3: sata-port at 1 { +>> + sata3: sata-port@1 { >> + reg = <1>; >> + target-supply = <®_5v_sata3>; >> + }; @@ -171,7 +171,7 @@ but it also should be stable. > > Ditto. > ->> + sdhci at d8000 { +>> + sdhci@d8000 { >> + clock-frequency = <200000000>; > > Why? There is already a 'clocks' property in armada-38x.dtsi. However, @@ -207,3 +207,7 @@ Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com +-- +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 2ceffcd..7b641ed 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,10 +1,23 @@ "ref\01419678035-10658-1-git-send-email-gregory.clement@free-electrons.com\0" "ref\01419678035-10658-4-git-send-email-gregory.clement@free-electrons.com\0" "ref\020141231113250.6ae98b19@free-electrons.com\0" - "From\0gregory.clement@free-electrons.com (Gregory CLEMENT)\0" - "Subject\0[PATCH 3/3] ARM: mvebu: Add Armada 385 General Purpose Development Board support\0" + "ref\020141231113250.6ae98b19-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org\0" + "From\0Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>\0" + "Subject\0Re: [PATCH 3/3] ARM: mvebu: Add Armada 385 General Purpose Development Board support\0" "Date\0Mon, 05 Jan 2015 18:06:58 +0100\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Thomas Petazzoni <thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>\0" + "Cc\0Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>" + Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> + Sebastian Hesselbarth <sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> + Ezequiel Garcia <ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org + Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + Boris BREZILLON <boris.brezillon-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + Lior Amsalem <alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org> + Tawfik Bayouk <tawfik-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org> + Nadav Haklai <nadavh-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org> + Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> + " devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" "\00:1\0" "b\0" "Hi Thomas,\n" @@ -14,7 +27,7 @@ "> \n" "> On Sat, 27 Dec 2014 12:00:35 +0100, Gregory CLEMENT wrote:\n" "> \n" - ">> +\t\t\t\tspi-flash at 0 {\n" + ">> +\t\t\t\tspi-flash@0 {\n" ">> +\t\t\t\t\t#address-cells = <1>;\n" ">> +\t\t\t\t\t#size-cells = <1>;\n" ">> +\t\t\t\t\tcompatible = \"st,m25p128\";\n" @@ -28,11 +41,11 @@ "130nm device which is limited to 50MHz.\n" "\n" "> \n" - ">> +\t\t\ti2c at 11000 {\n" + ">> +\t\t\ti2c@11000 {\n" ">> +\t\t\t\tstatus = \"okay\";\n" ">> +\t\t\t\tclock-frequency = <100000>;\n" ">> +\n" - ">> +\t\t\t\tpca9555_0: pca9555 at 20 {\n" + ">> +\t\t\t\tpca9555_0: pca9555@20 {\n" ">> +\t\t\t\t\tcompatible = \"nxp,pca9555\";\n" ">> +\t\t\t\t\tpinctrl-names = \"default\";\n" ">> +\t\t\t\t\tpinctrl-0 = <&pca0_pins>;\n" @@ -45,7 +58,7 @@ ">> +\t\t\t\t\treg = <0x20>;\n" ">> +\t\t\t\t};\n" ">> +\n" - ">> +\t\t\t\tpca9555_1: pca9555 at 21 {\n" + ">> +\t\t\t\tpca9555_1: pca9555@21 {\n" ">> +\t\t\t\t\tcompatible = \"nxp,pca9555\";\n" ">> +\t\t\t\t\tpinctrl-names = \"default\";\n" ">> +\t\t\t\t\tinterrupt-parent = <&gpio0>;\n" @@ -67,7 +80,7 @@ "\n" "Yes it was a left over of from a previous version, I will remove it.\n" "\n" - ">> +\t\t\tethernet at 70000 {\n" + ">> +\t\t\tethernet@70000 {\n" ">> +\t\t\t\tstatus = \"okay\";\n" ">> +\t\t\t\tphy = <&phy1>;\n" ">> +\t\t\t\tphy-mode = \"rgmii-id\";\n" @@ -77,12 +90,12 @@ "> \n" "> One too many empty new line.\n" "> \n" - ">> +\t\t\tmdio at 72004 {\n" - ">> +\t\t\t\tphy0: ethernet-phy at 0 {\n" + ">> +\t\t\tmdio@72004 {\n" + ">> +\t\t\t\tphy0: ethernet-phy@0 {\n" ">> +\t\t\t\t\treg = <0>;\n" ">> +\t\t\t\t};\n" ">> +\n" - ">> +\t\t\t\tphy1: ethernet-phy at 1 {\n" + ">> +\t\t\t\tphy1: ethernet-phy@1 {\n" ">> +\t\t\t\t\treg = <1>;\n" ">> +\t\t\t\t};\n" ">> +\t\t\t};\n" @@ -97,20 +110,20 @@ "> \n" "> So maybe we should have:\n" "> \n" - "> \tethernet at 30000 {\n" + "> \tethernet@30000 {\n" "> \t\tphy = <&phy1>;\n" "> \t};\n" "> \n" - "> \tethernet at 70000 {\n" + "> \tethernet@70000 {\n" "> \t\tphy = <&phy0>;\n" "> \t};\n" "> \n" - "> \tmdio at 72004 {\n" - "> \t\tphy0: ethernet-phy at 1 {\n" + "> \tmdio@72004 {\n" + "> \t\tphy0: ethernet-phy@1 {\n" "> \t\t\treg = <1>;\n" "> \t\t};\n" "> \n" - "> \t\tphy1: ethernet-phy at 0 {\n" + "> \t\tphy1: ethernet-phy@0 {\n" "> \t\t\treg = <0>;\n" "> \t\t};\n" "> \t};\n" @@ -120,18 +133,18 @@ "\n" "\n" "> \n" - ">> +\t\t\tsata at a8000 {\n" + ">> +\t\t\tsata@a8000 {\n" ">> +\t\t\t\tnr-ports = <2>;\n" ">> +\t\t\t\tstatus = \"okay\";\n" ">> +\t\t\t\t#address-cells = <1>;\n" ">> +\t\t\t\t#size-cells = <0>;\n" ">> +\n" - ">> +\t\t\t\tsata0: sata-port at 0 {\n" + ">> +\t\t\t\tsata0: sata-port@0 {\n" ">> +\t\t\t\t\treg = <0>;\n" ">> +\t\t\t\t\ttarget-supply = <®_5v_sata0>;\n" ">> +\t\t\t\t};\n" ">> +\n" - ">> +\t\t\t\tsata1: sata-port at 1 {\n" + ">> +\t\t\t\tsata1: sata-port@1 {\n" ">> +\t\t\t\t\treg = <1>;\n" ">> +\t\t\t\t\ttarget-supply = <®_5v_sata1>;\n" ">> +\t\t\t\t};\n" @@ -161,18 +174,18 @@ "> \n" ">> +\t\t\t};\n" ">> +\n" - ">> +\t\t\tsata at e0000 {\n" + ">> +\t\t\tsata@e0000 {\n" ">> +\t\t\t\tnr-ports = <2>;\n" ">> +\t\t\t\tstatus = \"okay\";\n" ">> +\t\t\t\t#address-cells = <1>;\n" ">> +\t\t\t\t#size-cells = <0>;\n" ">> +\n" - ">> +\t\t\t\tsata2: sata-port at 0 {\n" + ">> +\t\t\t\tsata2: sata-port@0 {\n" ">> +\t\t\t\t\treg = <0>;\n" ">> +\t\t\t\t\ttarget-supply = <®_5v_sata2>;\n" ">> +\t\t\t\t};\n" ">> +\n" - ">> +\t\t\t\tsata3: sata-port at 1 {\n" + ">> +\t\t\t\tsata3: sata-port@1 {\n" ">> +\t\t\t\t\treg = <1>;\n" ">> +\t\t\t\t\ttarget-supply = <®_5v_sata3>;\n" ">> +\t\t\t\t};\n" @@ -180,7 +193,7 @@ "> \n" "> Ditto.\n" "> \n" - ">> +\t\t\tsdhci at d8000 {\n" + ">> +\t\t\tsdhci@d8000 {\n" ">> +\t\t\t\tclock-frequency = <200000000>;\n" "> \n" "> Why? There is already a 'clocks' property in armada-38x.dtsi. However,\n" @@ -215,6 +228,10 @@ "Gregory Clement, Free Electrons\n" "Kernel, drivers, real-time and embedded Linux\n" "development, consulting, training and support.\n" - http://free-electrons.com + "http://free-electrons.com\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 -44d10e5291e96973ee17b736211d85cb9f37d17b4df560c8971f8274a49ae943 +9eec55cf8111a0ac8c277be50fb6ece412eb120968f6f04cbf516fe8eafa7921
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.