diff for duplicates of <53CFD8B5.2030205@gmail.com> diff --git a/a/1.txt b/N1/1.txt index 5e64aa8..817fb13 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -27,7 +27,7 @@ It is spelled "Lenovo" and the full name is "Lenovo Iomega ix4-300d". As it is the initial commit, we should be more sensitive on the case here. ->> + * Copyright (C) 2014, Benoit Masson <yahoo@perenite.com> +>> + * Copyright (C) 2014, Benoit Masson <yahoo-+V3Jd3LB6RBWk0Htik3J/w@public.gmane.org> >> + * >> + * This program is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU General Public License @@ -53,7 +53,7 @@ Same comment about "Lenovo Iomega ix4-300d" Consider to add -stdout-path = "/soc/internal-regs/serial at 12000"; +stdout-path = "/soc/internal-regs/serial@12000"; That way you would not have to fix it up for e.g. Barebox. Unfortunately, neither Armada 370 nor XP dtsi define node @@ -69,11 +69,11 @@ labels yet, so you have to put the full node path. >> + pcie-controller { >> + status = "okay"; >> + ->> + pcie at 1,0 { +>> + pcie@1,0 { >> + /* Port 0, Lane 0 */ >> + status = "okay"; >> + }; ->> + pcie at 5,0 { +>> + pcie@5,0 { >> + /* Port 1, Lane 0 */ >> + status = "okay"; >> + }; @@ -84,7 +84,7 @@ If so, put a comment above the node itself naming the attached device, e.g. /* USB 3.0 xHCI controller */ -pcie at 1,0 {...} +pcie@1,0 {...} Also, I saw you have a good impression about the GPIOs used on the ix4-300d. If you also know the PERST# GPIOs, add them with @@ -131,7 +131,7 @@ like the ones below? [...] >> + }; >> + ->> + serial at 12000 { +>> + serial@12000 { >> + clocks = <&coreclk 0>; As Andrew already said, no need to duplicate the clocks property. @@ -150,7 +150,7 @@ As Andrew already said, no need to duplicate the clocks property. >> + #address-cells = <1>; >> + #size-cells = <0>; >> + ->> + gpio2: gpio2 at 0 { +>> + gpio2: gpio2@0 { >> + compatible = "fairchild,74hc595"; Freaky we have a driver for a 74x595 8-bit shift register ;) @@ -187,3 +187,7 @@ I don't get the comment. Yup, Thanks! Sebastian +-- +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 e741e0a..3acbb12 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,9 +1,23 @@ "ref\01406117232-5962-1-git-send-email-yahoo@perenite.com\0" "ref\020140723134534.GF23220@titan.lakedaemon.net\0" - "From\0sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)\0" - "Subject\0[PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas\0" + "ref\020140723134534.GF23220-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org\0" + "From\0Sebastian Hesselbarth <sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>\0" + "Subject\0Re: [PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas\0" "Date\0Wed, 23 Jul 2014 17:45:57 +0200\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>" + " benoitm974 <yahoo-+V3Jd3LB6RBWk0Htik3J/w@public.gmane.org>\0" + "Cc\0benoitm-+V3Jd3LB6RBWk0Htik3J/w@public.gmane.org" + Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> + Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org> + Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org> + Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> + Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org> + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org + linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> + Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> + " Sebastian Hesselbarth <sebastian.hesselbarth-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>\0" "\00:1\0" "b\0" "On 07/23/2014 03:45 PM, Jason Cooper wrote:\n" @@ -35,7 +49,7 @@ "As it is the initial commit, we should be more sensitive on the case\n" "here.\n" "\n" - ">> + * Copyright (C) 2014, Benoit Masson <yahoo@perenite.com>\n" + ">> + * Copyright (C) 2014, Benoit Masson <yahoo-+V3Jd3LB6RBWk0Htik3J/w@public.gmane.org>\n" ">> + *\n" ">> + * This program is free software; you can redistribute it and/or\n" ">> + * modify it under the terms of the GNU General Public License\n" @@ -61,7 +75,7 @@ "\n" "Consider to add\n" "\n" - "stdout-path = \"/soc/internal-regs/serial at 12000\";\n" + "stdout-path = \"/soc/internal-regs/serial@12000\";\n" "\n" "That way you would not have to fix it up for e.g. Barebox.\n" "Unfortunately, neither Armada 370 nor XP dtsi define node\n" @@ -77,11 +91,11 @@ ">> +\t\tpcie-controller {\n" ">> +\t\t\tstatus = \"okay\";\n" ">> +\n" - ">> +\t\t\tpcie at 1,0 {\n" + ">> +\t\t\tpcie@1,0 {\n" ">> +\t\t\t\t/* Port 0, Lane 0 */\n" ">> +\t\t\t\tstatus = \"okay\";\n" ">> +\t\t\t};\n" - ">> +\t\t\tpcie at 5,0 {\n" + ">> +\t\t\tpcie@5,0 {\n" ">> + /* Port 1, Lane 0 */\n" ">> + status = \"okay\";\n" ">> + };\n" @@ -92,7 +106,7 @@ "device, e.g.\n" "\n" "/* USB 3.0 xHCI controller */\n" - "pcie at 1,0 {...}\n" + "pcie@1,0 {...}\n" "\n" "Also, I saw you have a good impression about the GPIOs used on\n" "the ix4-300d. If you also know the PERST# GPIOs, add them with\n" @@ -139,7 +153,7 @@ "[...]\n" ">> +\t\t\t};\n" ">> +\n" - ">> +\t\t\tserial at 12000 {\n" + ">> +\t\t\tserial@12000 {\n" ">> +\t\t\t\tclocks = <&coreclk 0>;\n" "\n" "As Andrew already said, no need to duplicate the clocks property.\n" @@ -158,7 +172,7 @@ ">> + #address-cells = <1>;\n" ">> + #size-cells = <0>;\n" ">> +\n" - ">> + gpio2: gpio2 at 0 {\n" + ">> + gpio2: gpio2@0 {\n" ">> + compatible = \"fairchild,74hc595\";\n" "\n" "Freaky we have a driver for a 74x595 8-bit shift register ;)\n" @@ -194,6 +208,10 @@ "\n" "Yup, Thanks!\n" "\n" - Sebastian + "Sebastian\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 -bff1ce2f55cd78282e5bb588d563795a73a8fb81d74fd0054dcc6a351ef60359 +fee19cb0d9e13f3de5ccdaeb1e9ceac75827fe79c4374add2b94a28fc2add10b
diff --git a/a/1.txt b/N2/1.txt index 5e64aa8..0100e00 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -53,7 +53,7 @@ Same comment about "Lenovo Iomega ix4-300d" Consider to add -stdout-path = "/soc/internal-regs/serial at 12000"; +stdout-path = "/soc/internal-regs/serial@12000"; That way you would not have to fix it up for e.g. Barebox. Unfortunately, neither Armada 370 nor XP dtsi define node @@ -69,11 +69,11 @@ labels yet, so you have to put the full node path. >> + pcie-controller { >> + status = "okay"; >> + ->> + pcie at 1,0 { +>> + pcie@1,0 { >> + /* Port 0, Lane 0 */ >> + status = "okay"; >> + }; ->> + pcie at 5,0 { +>> + pcie@5,0 { >> + /* Port 1, Lane 0 */ >> + status = "okay"; >> + }; @@ -84,7 +84,7 @@ If so, put a comment above the node itself naming the attached device, e.g. /* USB 3.0 xHCI controller */ -pcie at 1,0 {...} +pcie@1,0 {...} Also, I saw you have a good impression about the GPIOs used on the ix4-300d. If you also know the PERST# GPIOs, add them with @@ -131,7 +131,7 @@ like the ones below? [...] >> + }; >> + ->> + serial at 12000 { +>> + serial@12000 { >> + clocks = <&coreclk 0>; As Andrew already said, no need to duplicate the clocks property. @@ -150,7 +150,7 @@ As Andrew already said, no need to duplicate the clocks property. >> + #address-cells = <1>; >> + #size-cells = <0>; >> + ->> + gpio2: gpio2 at 0 { +>> + gpio2: gpio2@0 { >> + compatible = "fairchild,74hc595"; Freaky we have a driver for a 74x595 8-bit shift register ;) diff --git a/a/content_digest b/N2/content_digest index e741e0a..7eaa7a5 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,9 +1,22 @@ "ref\01406117232-5962-1-git-send-email-yahoo@perenite.com\0" "ref\020140723134534.GF23220@titan.lakedaemon.net\0" - "From\0sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)\0" - "Subject\0[PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas\0" + "From\0Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>\0" + "Subject\0Re: [PATCH 1/2] Added dts defintion for Lenovo ix4-300d nas\0" "Date\0Wed, 23 Jul 2014 17:45:57 +0200\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Jason Cooper <jason@lakedaemon.net>" + " benoitm974 <yahoo@perenite.com>\0" + "Cc\0benoitm@perenite.com" + Rob Herring <robh+dt@kernel.org> + Pawel Moll <pawel.moll@arm.com> + Ian Campbell <ijc+devicetree@hellion.org.uk> + Kumar Gala <galak@codeaurora.org> + Russell King <linux@arm.linux.org.uk> + devicetree@vger.kernel.org + linux-arm-kernel@lists.infradead.org + linux-kernel@vger.kernel.org + Andrew Lunn <andrew@lunn.ch> + Gregory CLEMENT <gregory.clement@free-electrons.com> + " Sebastian Hesselbarth <sebastian.hesselbarth@googlemail.com>\0" "\00:1\0" "b\0" "On 07/23/2014 03:45 PM, Jason Cooper wrote:\n" @@ -61,7 +74,7 @@ "\n" "Consider to add\n" "\n" - "stdout-path = \"/soc/internal-regs/serial at 12000\";\n" + "stdout-path = \"/soc/internal-regs/serial@12000\";\n" "\n" "That way you would not have to fix it up for e.g. Barebox.\n" "Unfortunately, neither Armada 370 nor XP dtsi define node\n" @@ -77,11 +90,11 @@ ">> +\t\tpcie-controller {\n" ">> +\t\t\tstatus = \"okay\";\n" ">> +\n" - ">> +\t\t\tpcie at 1,0 {\n" + ">> +\t\t\tpcie@1,0 {\n" ">> +\t\t\t\t/* Port 0, Lane 0 */\n" ">> +\t\t\t\tstatus = \"okay\";\n" ">> +\t\t\t};\n" - ">> +\t\t\tpcie at 5,0 {\n" + ">> +\t\t\tpcie@5,0 {\n" ">> + /* Port 1, Lane 0 */\n" ">> + status = \"okay\";\n" ">> + };\n" @@ -92,7 +105,7 @@ "device, e.g.\n" "\n" "/* USB 3.0 xHCI controller */\n" - "pcie at 1,0 {...}\n" + "pcie@1,0 {...}\n" "\n" "Also, I saw you have a good impression about the GPIOs used on\n" "the ix4-300d. If you also know the PERST# GPIOs, add them with\n" @@ -139,7 +152,7 @@ "[...]\n" ">> +\t\t\t};\n" ">> +\n" - ">> +\t\t\tserial at 12000 {\n" + ">> +\t\t\tserial@12000 {\n" ">> +\t\t\t\tclocks = <&coreclk 0>;\n" "\n" "As Andrew already said, no need to duplicate the clocks property.\n" @@ -158,7 +171,7 @@ ">> + #address-cells = <1>;\n" ">> + #size-cells = <0>;\n" ">> +\n" - ">> + gpio2: gpio2 at 0 {\n" + ">> + gpio2: gpio2@0 {\n" ">> + compatible = \"fairchild,74hc595\";\n" "\n" "Freaky we have a driver for a 74x595 8-bit shift register ;)\n" @@ -196,4 +209,4 @@ "\n" Sebastian -bff1ce2f55cd78282e5bb588d563795a73a8fb81d74fd0054dcc6a351ef60359 +d43bd9e8180c08d7a495d8fab70dec1337e41495c2eb21aced4941e6a1113b79
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.