diff for duplicates of <1409046129.71143501@apps.rackspace.com> diff --git a/a/1.txt b/N1/1.txt index 6ddc0b0..f6dbfd2 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ -On Wednesday, August 20, 2014 11:36am, "Andreas Färber" <afaerber@suse.de> said: +On Wednesday, August 20, 2014 11:36am, "Andreas F?rber" <afaerber@suse.de> said: > Hi, > @@ -11,7 +11,7 @@ On Wednesday, August 20, 2014 11:36am, "Andreas Färber" <afaerber@suse.de> said >> >> Tested on IFC6410 board. >> ->> Signed-off-by: Kiran Padwal <kiran.padwal-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org> +>> Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com> >> --- >> Chages since v1: >> - Renamed pinmux phandle "qcom_pinmux" to "tlmm_pinmux". @@ -30,7 +30,7 @@ On Wednesday, August 20, 2014 11:36am, "Andreas Färber" <afaerber@suse.de> said >> compatible = "qcom,apq8064-ifc6410", "qcom,apq8064"; >> >> soc { ->> + pinmux@800000 { +>> + pinmux at 800000 { >> + i2c1_pins: i2c1_pinmux { > > Is there a requirement to have "_pinmux" in the subnode of the "pinmux" @@ -47,7 +47,7 @@ Ok, I will change it to i2c1. >> + }; >> + }; >> + ->> gsbi@16600000 { +>> gsbi at 16600000 { >> status = "ok"; >> qcom,mode = <GSBI_PROT_I2C_UART>; >> @@ -12,5 +21,25 @@ @@ -55,7 +55,7 @@ Ok, I will change it to i2c1. >> }; >> }; >> + ->> + gsbi1: gsbi@12440000 { +>> + gsbi1: gsbi at 12440000 { >> + qcom,mode = <GSBI_PROT_I2C>; >> + status = "ok"; > @@ -70,13 +70,13 @@ Ok, I will change it to i2c1. > below, so you don't need to do it here again. > Some architectures prefer in such a case that in the .dts the node is > referenced as &gsbi1 {...}; below the root node in alphabetical order -> rather than duplicating the full /soc/foo@bar hierarchy. +> rather than duplicating the full /soc/foo at bar hierarchy. Indeed, I'll change on v3. > >> + ->> + i2c1: i2c@12460000 { +>> + i2c1: i2c at 12460000 { > > Suggest to move the i2c1 label to the .dtsi. Then optionally same could > be done here as outlined for gsbi1. @@ -97,7 +97,7 @@ Ok. >> + pinctrl-0 = <&i2c1_pins>; >> + pinctrl-names = "default"; >> + ->> + eeprom: eeprom@52 { +>> + eeprom: eeprom at 52 { >> + compatible = "atmel,24c128"; >> + reg = <0x52>; >> + pagesize = <32>; @@ -115,7 +115,7 @@ Ok. >> ranges; >> compatible = "simple-bus"; >> ->> + tlmm_pinmux: pinmux@800000 { +>> + tlmm_pinmux: pinmux at 800000 { >> + compatible = "qcom,apq8064-pinctrl"; >> + reg = <0x800000 0x4000>; >> + @@ -132,14 +132,14 @@ Indeed, I'll change on v3. > >> + }; >> + ->> intc: interrupt-controller@2000000 { +>> intc: interrupt-controller at 2000000 { >> compatible = "qcom,msm-qgic2"; >> interrupt-controller; >> @@ -133,6 +144,54 @@ >> regulator; >> }; >> ->> + gsbi1: gsbi@12440000 { +>> + gsbi1: gsbi at 12440000 { >> + compatible = "qcom,gsbi-v1.0.0"; >> + reg = <0x12440000 0x100>; >> + clocks = <&gcc GSBI1_H_CLK>; @@ -149,7 +149,7 @@ Indeed, I'll change on v3. >> + ranges; >> + status = "disabled"; >> + ->> + i2c@12460000 { +>> + i2c at 12460000 { >> + compatible = "qcom,i2c-qup-v1.1.1"; >> + reg = <0x12460000 0x1000>; >> + interrupts = <0 194 0>; @@ -175,7 +175,7 @@ Indeed, I'll change on v3. >> + }; >> + }; >> + ->> + gsbi2: gsbi@12480000 { +>> + gsbi2: gsbi at 12480000 { >> + compatible = "qcom,gsbi-v1.0.0"; >> + reg = <0x12480000 0x100>; >> + clocks = <&gcc GSBI2_H_CLK>; @@ -185,7 +185,7 @@ Indeed, I'll change on v3. >> + ranges; >> + status = "disabled"; >> + ->> + i2c@124a0000 { +>> + i2c at 124a0000 { >> + compatible = "qcom,i2c-qup-v1.1.1"; >> + reg = <0x124a0000 0x1000>; >> + interrupts = <0 196 0>; @@ -209,17 +209,11 @@ Thanks. >> + }; >> + }; >> + ->> gsbi7: gsbi@16600000 { +>> gsbi7: gsbi at 16600000 { >> status = "disabled"; >> compatible = "qcom,gsbi-v1.0.0"; > > -- -> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany -> GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg -> - - --- -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 +> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany +> GF: Jeff Hawn, Jennifer Guild, Felix Imend?rffer; HRB 16746 AG N?rnberg +> diff --git a/a/content_digest b/N1/content_digest index 4219dd7..d870893 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,26 +1,14 @@ "ref\01408536133-13824-1-git-send-email-kiran.padwal@smartplayin.com\0" "ref\053F4C085.5060101@suse.de\0" - "ref\053F4C085.5060101-l3A5Bk7waGM@public.gmane.org\0" - "From\0kiran.padwal-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org\0" - "Subject\0Re: [PATCH v2] ARM: apq8064: Add pinmux and i2c pinctrl nodes\0" + "From\0kiran.padwal@smartplayin.com (kiran.padwal at smartplayin.com)\0" + "Subject\0[PATCH v2] ARM: apq8064: Add pinmux and i2c pinctrl nodes\0" "Date\0Tue, 26 Aug 2014 05:42:09 -0400 (EDT)\0" - "To\0Andreas F\303\244rber <afaerber-l3A5Bk7waGM@public.gmane.org>\0" - "Cc\0robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" - pawel.moll-5wv7dgnIgG8@public.gmane.org - mark.rutland-5wv7dgnIgG8@public.gmane.org - ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org - galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - davidb-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org - linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org - linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - " linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "\n" "\n" - "On Wednesday, August 20, 2014 11:36am, \"Andreas F\303\244rber\" <afaerber@suse.de> said:\n" + "On Wednesday, August 20, 2014 11:36am, \"Andreas F?rber\" <afaerber@suse.de> said:\n" "\n" "> Hi,\n" "> \n" @@ -31,7 +19,7 @@ ">>\n" ">> Tested on IFC6410 board.\n" ">>\n" - ">> Signed-off-by: Kiran Padwal <kiran.padwal-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org>\n" + ">> Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>\n" ">> ---\n" ">> Chages since v1:\n" ">> \t- Renamed pinmux phandle \"qcom_pinmux\" to \"tlmm_pinmux\".\n" @@ -50,7 +38,7 @@ ">> \tcompatible = \"qcom,apq8064-ifc6410\", \"qcom,apq8064\";\n" ">>\n" ">> \tsoc {\n" - ">> +\t\tpinmux@800000 {\n" + ">> +\t\tpinmux at 800000 {\n" ">> +\t\t\ti2c1_pins: i2c1_pinmux {\n" "> \n" "> Is there a requirement to have \"_pinmux\" in the subnode of the \"pinmux\"\n" @@ -67,7 +55,7 @@ ">> +\t\t\t};\n" ">> +\t\t};\n" ">> +\n" - ">> \t\tgsbi@16600000 {\n" + ">> \t\tgsbi at 16600000 {\n" ">> \t\t\tstatus = \"ok\";\n" ">> \t\t\tqcom,mode = <GSBI_PROT_I2C_UART>;\n" ">> @@ -12,5 +21,25 @@\n" @@ -75,7 +63,7 @@ ">> \t\t\t};\n" ">> \t\t};\n" ">> +\n" - ">> +\t\tgsbi1: gsbi@12440000 {\n" + ">> +\t\tgsbi1: gsbi at 12440000 {\n" ">> +\t\t\tqcom,mode = <GSBI_PROT_I2C>;\n" ">> +\t\t\tstatus = \"ok\";\n" "> \n" @@ -90,13 +78,13 @@ "> below, so you don't need to do it here again.\n" "> Some architectures prefer in such a case that in the .dts the node is\n" "> referenced as &gsbi1 {...}; below the root node in alphabetical order\n" - "> rather than duplicating the full /soc/foo@bar hierarchy.\n" + "> rather than duplicating the full /soc/foo at bar hierarchy.\n" "\n" "Indeed, I'll change on v3.\n" "\n" "> \n" ">> +\n" - ">> +\t\t\ti2c1: i2c@12460000 {\n" + ">> +\t\t\ti2c1: i2c at 12460000 {\n" "> \n" "> Suggest to move the i2c1 label to the .dtsi. Then optionally same could\n" "> be done here as outlined for gsbi1.\n" @@ -117,7 +105,7 @@ ">> +\t\t\t\tpinctrl-0 = <&i2c1_pins>;\n" ">> +\t\t\t\tpinctrl-names = \"default\";\n" ">> +\n" - ">> +\t\t\t\teeprom: eeprom@52 {\n" + ">> +\t\t\t\teeprom: eeprom at 52 {\n" ">> +\t\t\t\t\tcompatible = \"atmel,24c128\";\n" ">> +\t\t\t\t\treg = <0x52>;\n" ">> +\t\t\t\t\tpagesize = <32>;\n" @@ -135,7 +123,7 @@ ">> \t\tranges;\n" ">> \t\tcompatible = \"simple-bus\";\n" ">>\n" - ">> +\t\ttlmm_pinmux: pinmux@800000 {\n" + ">> +\t\ttlmm_pinmux: pinmux at 800000 {\n" ">> +\t\t\tcompatible = \"qcom,apq8064-pinctrl\";\n" ">> +\t\t\treg = <0x800000 0x4000>;\n" ">> +\n" @@ -152,14 +140,14 @@ "> \n" ">> +\t\t};\n" ">> +\n" - ">> \t\tintc: interrupt-controller@2000000 {\n" + ">> \t\tintc: interrupt-controller at 2000000 {\n" ">> \t\t\tcompatible = \"qcom,msm-qgic2\";\n" ">> \t\t\tinterrupt-controller;\n" ">> @@ -133,6 +144,54 @@\n" ">> \t\t\tregulator;\n" ">> \t\t};\n" ">>\n" - ">> +\t\tgsbi1: gsbi@12440000 {\n" + ">> +\t\tgsbi1: gsbi at 12440000 {\n" ">> +\t\t\tcompatible = \"qcom,gsbi-v1.0.0\";\n" ">> +\t\t\treg = <0x12440000 0x100>;\n" ">> +\t\t\tclocks = <&gcc GSBI1_H_CLK>;\n" @@ -169,7 +157,7 @@ ">> +\t\t\tranges;\n" ">> +\t\t\tstatus = \"disabled\";\n" ">> +\n" - ">> +\t\t\ti2c@12460000 {\n" + ">> +\t\t\ti2c at 12460000 {\n" ">> +\t\t\t\tcompatible = \"qcom,i2c-qup-v1.1.1\";\n" ">> +\t\t\t\treg = <0x12460000 0x1000>;\n" ">> +\t\t\t\tinterrupts = <0 194 0>;\n" @@ -195,7 +183,7 @@ ">> +\t\t\t};\n" ">> +\t\t};\n" ">> +\n" - ">> +\t\tgsbi2: gsbi@12480000 {\n" + ">> +\t\tgsbi2: gsbi at 12480000 {\n" ">> +\t\t\tcompatible = \"qcom,gsbi-v1.0.0\";\n" ">> +\t\t\treg = <0x12480000 0x100>;\n" ">> +\t\t\tclocks = <&gcc GSBI2_H_CLK>;\n" @@ -205,7 +193,7 @@ ">> +\t\t\tranges;\n" ">> +\t\t\tstatus = \"disabled\";\n" ">> +\n" - ">> +\t\t\ti2c@124a0000 {\n" + ">> +\t\t\ti2c at 124a0000 {\n" ">> +\t\t\t\tcompatible = \"qcom,i2c-qup-v1.1.1\";\n" ">> +\t\t\t\treg = <0x124a0000 0x1000>;\n" ">> +\t\t\t\tinterrupts = <0 196 0>;\n" @@ -229,19 +217,13 @@ ">> +\t\t\t};\n" ">> +\t\t};\n" ">> +\n" - ">> \t\tgsbi7: gsbi@16600000 {\n" + ">> \t\tgsbi7: gsbi at 16600000 {\n" ">> \t\t\tstatus = \"disabled\";\n" ">> \t\t\tcompatible = \"qcom,gsbi-v1.0.0\";\n" "> \n" "> --\n" - "> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N\303\274rnberg, Germany\n" - "> GF: Jeff Hawn, Jennifer Guild, Felix Imend\303\266rffer; HRB 16746 AG N\303\274rnberg\n" - "> \n" - "\n" - "\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 + "> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany\n" + "> GF: Jeff Hawn, Jennifer Guild, Felix Imend?rffer; HRB 16746 AG N?rnberg\n" + > -2b2fdc97e119e921333c930dda0e20f26aeebb21715e86167c405221f7526147 +7d18352ba812c31a8cb95fc63def1ee056347d96dfab4f06dc6a8cf9348f5bd2
diff --git a/a/1.txt b/N2/1.txt index 6ddc0b0..c4dc5e8 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -11,7 +11,7 @@ On Wednesday, August 20, 2014 11:36am, "Andreas Färber" <afaerber@suse.de> said >> >> Tested on IFC6410 board. >> ->> Signed-off-by: Kiran Padwal <kiran.padwal-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org> +>> Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com> >> --- >> Chages since v1: >> - Renamed pinmux phandle "qcom_pinmux" to "tlmm_pinmux". @@ -216,10 +216,4 @@ Thanks. > -- > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg -> - - --- -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/N2/content_digest index 4219dd7..6f9c385 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,21 +1,20 @@ "ref\01408536133-13824-1-git-send-email-kiran.padwal@smartplayin.com\0" "ref\053F4C085.5060101@suse.de\0" - "ref\053F4C085.5060101-l3A5Bk7waGM@public.gmane.org\0" - "From\0kiran.padwal-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org\0" + "From\0kiran.padwal@smartplayin.com\0" "Subject\0Re: [PATCH v2] ARM: apq8064: Add pinmux and i2c pinctrl nodes\0" "Date\0Tue, 26 Aug 2014 05:42:09 -0400 (EDT)\0" - "To\0Andreas F\303\244rber <afaerber-l3A5Bk7waGM@public.gmane.org>\0" - "Cc\0robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" - pawel.moll-5wv7dgnIgG8@public.gmane.org - mark.rutland-5wv7dgnIgG8@public.gmane.org - ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org - galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - davidb-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org - linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org - linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - " linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" + "To\0Andreas F\303\244rber <afaerber@suse.de>\0" + "Cc\0robh+dt@kernel.org" + pawel.moll@arm.com + mark.rutland@arm.com + ijc+devicetree@hellion.org.uk + galak@codeaurora.org + davidb@codeaurora.org + devicetree@vger.kernel.org + linux@arm.linux.org.uk + linux-arm-kernel@lists.infradead.org + linux-arm-msm@vger.kernel.org + " linux-kernel@vger.kernel.org\0" "\00:1\0" "b\0" "\n" @@ -31,7 +30,7 @@ ">>\n" ">> Tested on IFC6410 board.\n" ">>\n" - ">> Signed-off-by: Kiran Padwal <kiran.padwal-edOiRQu9Xnj5XLMNweQjbQ@public.gmane.org>\n" + ">> Signed-off-by: Kiran Padwal <kiran.padwal@smartplayin.com>\n" ">> ---\n" ">> Chages since v1:\n" ">> \t- Renamed pinmux phandle \"qcom_pinmux\" to \"tlmm_pinmux\".\n" @@ -236,12 +235,6 @@ "> --\n" "> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N\303\274rnberg, Germany\n" "> GF: Jeff Hawn, Jennifer Guild, Felix Imend\303\266rffer; HRB 16746 AG N\303\274rnberg\n" - "> \n" - "\n" - "\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 + > -2b2fdc97e119e921333c930dda0e20f26aeebb21715e86167c405221f7526147 +fdda52e91b3b61bb4d5e89e64ca01f59e998a8eabdd0f66e4c1fe4e74c41e8f8
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.