diff for duplicates of <20150513025204.20636.10155@quantum> diff --git a/a/1.txt b/N1/1.txt index 942a5ab..b29c5dc 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -5,23 +5,21 @@ Quoting Paul Burton (2015-04-24 06:17:26) > +{ > + int retval; > + -> + cgu =3D ingenic_cgu_new(jz4740_cgu_clocks, +> + cgu = ingenic_cgu_new(jz4740_cgu_clocks, > + ARRAY_SIZE(jz4740_cgu_clocks), np); > + if (!cgu) { > + pr_err("%s: failed to initialise CGU\n", __func__); > + return; > + } > + -> + retval =3D ingenic_cgu_register_clocks(cgu); +> + retval = ingenic_cgu_register_clocks(cgu); > + if (retval) > + pr_err("%s: failed to register CGU Clocks\n", __func__); > +} > +CLK_OF_DECLARE(jz4740_cgu, "ingenic,jz4740-cgu", jz4740_cgu_init); -> -- = - +> -- > 2.3.5 -> = - +> Patch looks good to me, but I have one question. Is it possible for you to have a proper platform_device for the cgu instead of relying on diff --git a/a/content_digest b/N1/content_digest index 89df241..7e54584 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -25,23 +25,21 @@ "> +{\n" "> + int retval;\n" "> +\n" - "> + cgu =3D ingenic_cgu_new(jz4740_cgu_clocks,\n" + "> + cgu = ingenic_cgu_new(jz4740_cgu_clocks,\n" "> + ARRAY_SIZE(jz4740_cgu_clocks), np);\n" "> + if (!cgu) {\n" "> + pr_err(\"%s: failed to initialise CGU\\n\", __func__);\n" "> + return;\n" "> + }\n" "> +\n" - "> + retval =3D ingenic_cgu_register_clocks(cgu);\n" + "> + retval = ingenic_cgu_register_clocks(cgu);\n" "> + if (retval)\n" "> + pr_err(\"%s: failed to register CGU Clocks\\n\", __func__);\n" "> +}\n" "> +CLK_OF_DECLARE(jz4740_cgu, \"ingenic,jz4740-cgu\", jz4740_cgu_init);\n" - "> -- =\n" - "\n" + "> -- \n" "> 2.3.5\n" - "> =\n" - "\n" + "> \n" "\n" "Patch looks good to me, but I have one question. Is it possible for you\n" "to have a proper platform_device for the cgu instead of relying on\n" @@ -53,4 +51,4 @@ "Regards,\n" Mike -ea444726be62afa23fbab5bf43394b4e84ec8fd384db4f8bfcb89c694f4611e6 +468f4e6c6386c79834c7017acecf37fe18eab9d4c9bd1bf7483bc47c0170bec6
diff --git a/a/1.txt b/N2/1.txt index 942a5ab..b29c5dc 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -5,23 +5,21 @@ Quoting Paul Burton (2015-04-24 06:17:26) > +{ > + int retval; > + -> + cgu =3D ingenic_cgu_new(jz4740_cgu_clocks, +> + cgu = ingenic_cgu_new(jz4740_cgu_clocks, > + ARRAY_SIZE(jz4740_cgu_clocks), np); > + if (!cgu) { > + pr_err("%s: failed to initialise CGU\n", __func__); > + return; > + } > + -> + retval =3D ingenic_cgu_register_clocks(cgu); +> + retval = ingenic_cgu_register_clocks(cgu); > + if (retval) > + pr_err("%s: failed to register CGU Clocks\n", __func__); > +} > +CLK_OF_DECLARE(jz4740_cgu, "ingenic,jz4740-cgu", jz4740_cgu_init); -> -- = - +> -- > 2.3.5 -> = - +> Patch looks good to me, but I have one question. Is it possible for you to have a proper platform_device for the cgu instead of relying on diff --git a/a/content_digest b/N2/content_digest index 89df241..640a8bd 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -5,8 +5,7 @@ "Date\0Tue, 12 May 2015 19:52:04 -0700\0" "To\0Paul Burton <paul.burton@imgtec.com>" " linux-mips@linux-mips.org\0" - "Cc\0Paul Burton <paul.burton@imgtec.com>" - Ian Campbell <ijc+devicetree@hellion.org.uk> + "Cc\0Ian Campbell <ijc+devicetree@hellion.org.uk>" Kumar Gala <galak@codeaurora.org> Lars-Peter Clausen <lars@metafoo.de> Mark Rutland <mark.rutland@arm.com> @@ -25,23 +24,21 @@ "> +{\n" "> + int retval;\n" "> +\n" - "> + cgu =3D ingenic_cgu_new(jz4740_cgu_clocks,\n" + "> + cgu = ingenic_cgu_new(jz4740_cgu_clocks,\n" "> + ARRAY_SIZE(jz4740_cgu_clocks), np);\n" "> + if (!cgu) {\n" "> + pr_err(\"%s: failed to initialise CGU\\n\", __func__);\n" "> + return;\n" "> + }\n" "> +\n" - "> + retval =3D ingenic_cgu_register_clocks(cgu);\n" + "> + retval = ingenic_cgu_register_clocks(cgu);\n" "> + if (retval)\n" "> + pr_err(\"%s: failed to register CGU Clocks\\n\", __func__);\n" "> +}\n" "> +CLK_OF_DECLARE(jz4740_cgu, \"ingenic,jz4740-cgu\", jz4740_cgu_init);\n" - "> -- =\n" - "\n" + "> -- \n" "> 2.3.5\n" - "> =\n" - "\n" + "> \n" "\n" "Patch looks good to me, but I have one question. Is it possible for you\n" "to have a proper platform_device for the cgu instead of relying on\n" @@ -53,4 +50,4 @@ "Regards,\n" Mike -ea444726be62afa23fbab5bf43394b4e84ec8fd384db4f8bfcb89c694f4611e6 +98972c278131a8b8d4103219ea1f88a031e5a6c2ab3eb240b9523adfe8e14530
diff --git a/a/1.txt b/N3/1.txt index 942a5ab..c1189ad 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -5,23 +5,21 @@ Quoting Paul Burton (2015-04-24 06:17:26) > +{ > + int retval; > + -> + cgu =3D ingenic_cgu_new(jz4740_cgu_clocks, +> + cgu = ingenic_cgu_new(jz4740_cgu_clocks, > + ARRAY_SIZE(jz4740_cgu_clocks), np); > + if (!cgu) { > + pr_err("%s: failed to initialise CGU\n", __func__); > + return; > + } > + -> + retval =3D ingenic_cgu_register_clocks(cgu); +> + retval = ingenic_cgu_register_clocks(cgu); > + if (retval) > + pr_err("%s: failed to register CGU Clocks\n", __func__); > +} > +CLK_OF_DECLARE(jz4740_cgu, "ingenic,jz4740-cgu", jz4740_cgu_init); -> -- = - +> -- > 2.3.5 -> = - +> Patch looks good to me, but I have one question. Is it possible for you to have a proper platform_device for the cgu instead of relying on @@ -32,3 +30,7 @@ drivers/clk/qcom/gcc-msm8660.c. Regards, Mike +-- +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/N3/content_digest index 89df241..39bf6f2 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -1,21 +1,21 @@ "ref\01429881457-16016-1-git-send-email-paul.burton@imgtec.com\0" "ref\01429881457-16016-27-git-send-email-paul.burton@imgtec.com\0" - "From\0Michael Turquette <mturquette@linaro.org>\0" + "ref\01429881457-16016-27-git-send-email-paul.burton-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org\0" + "From\0Michael Turquette <mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>\0" "Subject\0Re: [PATCH v4 26/37] MIPS,clk: migrate JZ4740 to common clock framework\0" "Date\0Tue, 12 May 2015 19:52:04 -0700\0" - "To\0Paul Burton <paul.burton@imgtec.com>" - " linux-mips@linux-mips.org\0" - "Cc\0Paul Burton <paul.burton@imgtec.com>" - Ian Campbell <ijc+devicetree@hellion.org.uk> - Kumar Gala <galak@codeaurora.org> - Lars-Peter Clausen <lars@metafoo.de> - Mark Rutland <mark.rutland@arm.com> - Pawel Moll <pawel.moll@arm.com> - Ralf Baechle <ralf@linux-mips.org> - Rob Herring <robh+dt@kernel.org> - Stephen Boyd <sboyd@codeaurora.org> - devicetree@vger.kernel.org - " linux-clk@vger.kernel.org\0" + "To\0linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org\0" + "Cc\0Paul Burton <paul.burton-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>" + Ian Campbell <ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org> + Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> + Lars-Peter Clausen <lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org> + Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> + Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org> + Ralf Baechle <ralf-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org> + Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> + Stephen Boyd <sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org + " linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org\0" "\00:1\0" "b\0" "Hi Paul,\n" @@ -25,23 +25,21 @@ "> +{\n" "> + int retval;\n" "> +\n" - "> + cgu =3D ingenic_cgu_new(jz4740_cgu_clocks,\n" + "> + cgu = ingenic_cgu_new(jz4740_cgu_clocks,\n" "> + ARRAY_SIZE(jz4740_cgu_clocks), np);\n" "> + if (!cgu) {\n" "> + pr_err(\"%s: failed to initialise CGU\\n\", __func__);\n" "> + return;\n" "> + }\n" "> +\n" - "> + retval =3D ingenic_cgu_register_clocks(cgu);\n" + "> + retval = ingenic_cgu_register_clocks(cgu);\n" "> + if (retval)\n" "> + pr_err(\"%s: failed to register CGU Clocks\\n\", __func__);\n" "> +}\n" "> +CLK_OF_DECLARE(jz4740_cgu, \"ingenic,jz4740-cgu\", jz4740_cgu_init);\n" - "> -- =\n" - "\n" + "> -- \n" "> 2.3.5\n" - "> =\n" - "\n" + "> \n" "\n" "Patch looks good to me, but I have one question. Is it possible for you\n" "to have a proper platform_device for the cgu instead of relying on\n" @@ -51,6 +49,10 @@ "drivers/clk/qcom/gcc-msm8660.c.\n" "\n" "Regards,\n" - Mike + "Mike\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 -ea444726be62afa23fbab5bf43394b4e84ec8fd384db4f8bfcb89c694f4611e6 +f7df638fa0b6cf4a6f914df611ee62ad2491cdc1db46fe992290a951e73cc201
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.