diff for duplicates of <20170809172213.GB2146@codeaurora.org> diff --git a/a/1.txt b/N1/1.txt index 4e7d402..0281675 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,5 +1,5 @@ On 08/09, Eugeniy Paltsev wrote: -> On Thu, 2017-08-03 at 18:53 -0700, Stephen Boyd wrote: +> On Thu, 2017-08-03@18:53 -0700, Stephen Boyd wrote: > > On 07/14, Eugeniy Paltsev wrote: > > > + /* input divider = reg.idiv + 1 */ > > > + idiv = 1 + ((val & CGU_PLL_CTRL_IDIV_MASK) >> @@ -13,7 +13,7 @@ On 08/09, Eugeniy Paltsev wrote: > > > > Maybe just drop these comments. They're just repeating the code. > -> Actually I would prefer to keep them, as "2^(reg.odiv)" is more +> Actually I would prefer to keep them, as "2^(reg.odiv)" is more? > human-readable then > "1 << ((val & CGU_PLL_CTRL_ODIV_MASK) >> CGU_PLL_CTRL_ODIV_SHIFT)" @@ -38,7 +38,7 @@ Alright. > > > + > > > + if (pll_cfg[0].rate == 0) > > > + return -EINVAL; -> > +> >? > > This happens? > > Only if we add bad hsdk_pll_cfg table. But it is quite cold code - we diff --git a/a/content_digest b/N1/content_digest index 7edf622..df9224a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,20 +1,14 @@ "ref\020170714153128.8945-1-Eugeniy.Paltsev@synopsys.com\0" "ref\020170804015351.GW2146@codeaurora.org\0" "ref\01502297028.2586.4.camel@synopsys.com\0" - "From\0sboyd@codeaurora.org <sboyd@codeaurora.org>\0" - "Subject\0Re: [PATCH] ARC: clk: introduce HSDKv1 pll driver\0" + "From\0sboyd@codeaurora.org (sboyd@codeaurora.org)\0" + "Subject\0[PATCH] ARC: clk: introduce HSDKv1 pll driver\0" "Date\0Wed, 9 Aug 2017 10:22:13 -0700\0" - "To\0Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\0" - "Cc\0linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>" - mark.rutland@arm.com <mark.rutland@arm.com> - mturquette@baylibre.com <mturquette@baylibre.com> - robh+dt@kernel.org <robh+dt@kernel.org> - linux-clk@vger.kernel.org <linux-clk@vger.kernel.org> - " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" + "To\0linux-snps-arc@lists.infradead.org\0" "\00:1\0" "b\0" "On 08/09, Eugeniy Paltsev wrote:\n" - "> On Thu, 2017-08-03 at 18:53 -0700, Stephen Boyd wrote:\n" + "> On Thu, 2017-08-03@18:53 -0700, Stephen Boyd wrote:\n" "> > On 07/14, Eugeniy Paltsev wrote:\n" "> > > +\t/* input divider = reg.idiv + 1 */\n" "> > > +\tidiv = 1 + ((val & CGU_PLL_CTRL_IDIV_MASK) >>\n" @@ -28,7 +22,7 @@ "> >\n" "> > Maybe just drop these comments. They're just repeating the code.\n" "> \n" - "> Actually I would prefer to keep them, as \"2^(reg.odiv)\" is more\302\240\n" + "> Actually I would prefer to keep them, as \"2^(reg.odiv)\" is more?\n" "> human-readable then\n" "> \"1 << ((val & CGU_PLL_CTRL_ODIV_MASK) >> CGU_PLL_CTRL_ODIV_SHIFT)\"\n" "\n" @@ -53,7 +47,7 @@ "> > > +\n" "> > > +\tif (pll_cfg[0].rate == 0)\n" "> > > +\t\treturn -EINVAL;\n" - "> >\302\240\n" + "> >?\n" "> > This happens?\n" "> \n" "> Only if we add bad hsdk_pll_cfg table. But it is quite cold code - we\n" @@ -66,4 +60,4 @@ "Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,\n" a Linux Foundation Collaborative Project -74c9e0ad7eea1a534988d8b0e19103c9190305518f0ea2619b59b17dbaa740f6 +01907ead0d2585ac2e4a8903d80836685552e5e35f331cf5e8b385ea99b3ae76
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.