All of lore.kernel.org
 help / color / mirror / Atom feed
diff for duplicates of <149573872145.52617.10663444060792708996@resonance>

diff --git a/a/1.txt b/N1/1.txt
index 35a3898..cc77e8a 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -1,51 +1,40 @@
 Quoting Jerome Brunet (2017-05-23 02:48:48)
 > On Tue, 2017-05-23 at 12:35 +0300, Adriana Reus wrote:
-> > On Mon, May 22, 2017 at 12:59 AM, Jerome Brunet <jbrunet@baylibre.com> =
-wrote:
+> > On Mon, May 22, 2017 at 12:59 AM, Jerome Brunet <jbrunet@baylibre.com> wrote:
 > > > Create a core function for set_phase, as it is done for set_rate and
 > > > set_parent.
-> > > =
-
+> > > 
 > > > This rework is done to ease the integration of "protected" clock
 > > > functionality.
-> > > =
-
+> > > 
 > > > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
 > > > ---
-> > > =C2=A0drivers/clk/clk.c | 31 +++++++++++++++++++------------
-> > > =C2=A01 file changed, 19 insertions(+), 12 deletions(-)
-> > > =
-
+> > > ?drivers/clk/clk.c | 31 +++++++++++++++++++------------
+> > > ?1 file changed, 19 insertions(+), 12 deletions(-)
+> > > 
 > > > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
 > > > index f5c371532509..6031fada37f9 100644
 > > > --- a/drivers/clk/clk.c
 > > > +++ b/drivers/clk/clk.c
 > > > @@ -1873,6 +1873,23 @@ int clk_set_parent(struct clk *clk, struct clk
 > > > *parent)
-> > > =C2=A0}
-> > > =C2=A0EXPORT_SYMBOL_GPL(clk_set_parent);
-> > > =
-
-> > > +static int clk_core_set_phase_nolock(struct clk_core *core, int degr=
-ees)
+> > > ?}
+> > > ?EXPORT_SYMBOL_GPL(clk_set_parent);
+> > > 
+> > > +static int clk_core_set_phase_nolock(struct clk_core *core, int degrees)
 > > > +{
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0int ret =3D -EINVAL;
+> > > +???????int ret = -EINVAL;
 > > > +
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!core)
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
-=C2=A0=C2=A0=C2=A0=C2=A0return 0;
+> > > +???????if (!core)
+> > > +???????????????return 0;
 > > > +
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase(clk->c=
-ore, degrees);
-> > =
-
-> > =C2=A0^ trace_clk_set_phase(core, degrees)
-> =
-
+> > > +???????trace_clk_set_phase(clk->core, degrees);
+> > 
+> > ?^ trace_clk_set_phase(core, degrees)
+> 
 > Shame ... Once again this is a poor use of 'git add --patch'.
 > This particular diff ended up in patch 5.
-> =
-
+> 
 > Thanks a lot for catching it!
 
 Patch looks good to me overall. Can you reply here with V3? I'll apply
@@ -54,70 +43,52 @@ it to clk-next-protect for testing.
 Regards,
 Mike
 
-> =
-
+> 
 > > > +
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (core->ops->set_phase)
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
-=C2=A0=C2=A0=C2=A0=C2=A0ret =3D core->ops->set_phase(core->hw, degrees);
+> > > +???????if (core->ops->set_phase)
+> > > +???????????????ret = core->ops->set_phase(core->hw, degrees);
 > > > +
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase_comple=
-te(core, degrees);
+> > > +???????trace_clk_set_phase_complete(core, degrees);
 > > > +
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0return ret;
+> > > +???????return ret;
 > > > +}
 > > > +
-> > > =C2=A0/**
-> > > =C2=A0 * clk_set_phase - adjust the phase shift of a clock signal
-> > > =C2=A0 * @clk: clock signal source
+> > > ?/**
+> > > ? * clk_set_phase - adjust the phase shift of a clock signal
+> > > ? * @clk: clock signal source
 > > > @@ -1895,7 +1912,7 @@ EXPORT_SYMBOL_GPL(clk_set_parent);
-> > > =C2=A0 */
-> > > =C2=A0int clk_set_phase(struct clk *clk, int degrees)
-> > > =C2=A0{
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0int ret =3D -EINVAL;
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0int ret;
-> > > =
-
-> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!clk)
-> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
-=A0=C2=A0=C2=A0=C2=A0=C2=A0return 0;
+> > > ? */
+> > > ?int clk_set_phase(struct clk *clk, int degrees)
+> > > ?{
+> > > -???????int ret = -EINVAL;
+> > > +???????int ret;
+> > > 
+> > > ????????if (!clk)
+> > > ????????????????return 0;
 > > > @@ -1906,17 +1923,7 @@ int clk_set_phase(struct clk *clk, int degrees)
-> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
-=A0=C2=A0=C2=A0=C2=A0=C2=A0degrees +=3D 360;
-> > > =
-
-> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0clk_prepare_lock();
+> > > ????????????????degrees += 360;
+> > > 
+> > > ????????clk_prepare_lock();
 > > > -
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase(clk->c=
-ore, degrees);
+> > > -???????trace_clk_set_phase(clk->core, degrees);
 > > > -
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (clk->core->ops->set_ph=
-ase)
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
-=C2=A0=C2=A0=C2=A0=C2=A0ret =3D clk->core->ops->set_phase(clk->core->hw, de=
-grees);
+> > > -???????if (clk->core->ops->set_phase)
+> > > -???????????????ret = clk->core->ops->set_phase(clk->core->hw, degrees);
 > > > -
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase_comple=
-te(clk->core, degrees);
+> > > -???????trace_clk_set_phase_complete(clk->core, degrees);
 > > > -
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!ret)
-> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
-=C2=A0=C2=A0=C2=A0=C2=A0clk->core->phase =3D degrees;
+> > > -???????if (!ret)
+> > > -???????????????clk->core->phase = degrees;
 > > > -
-> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0ret =3D clk_core_set_phase=
-_nolock(clk->core, degrees);
-> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0clk_prepare_unlock();
-> > > =
-
-> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0return ret;
+> > > +???????ret = clk_core_set_phase_nolock(clk->core, degrees);
+> > > ????????clk_prepare_unlock();
+> > > 
+> > > ????????return ret;
 > > > --
 > > > 2.9.4
-> > > =
-
+> > > 
 > > > --
-> > > To unsubscribe from this list: send the line "unsubscribe linux-clk" =
-in
-> > > the body of a message to majordomo@vger.kernel.org
-> > > More majordomo info at=C2=A0=C2=A0http://vger.kernel.org/majordomo-in=
-fo.html
->=20
\ No newline at end of file
+> > > To unsubscribe from this list: send the line "unsubscribe linux-clk" in
+> > > the body of a message to majordomo at vger.kernel.org
+> > > More majordomo info at??http://vger.kernel.org/majordomo-info.html
+>
\ No newline at end of file
diff --git a/a/content_digest b/N1/content_digest
index d3d4541..b0e8f01 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -11,25 +11,16 @@
   "ref\0001495532928.2344.14.camel\@baylibre.com\0"
 ]
 [
-  "From\0Michael Turquette <mturquette\@baylibre.com>\0"
+  "From\0mturquette\@baylibre.com (Michael Turquette)\0"
 ]
 [
-  "Subject\0Re: [PATCH v2 02/11] clk: add clk_core_set_phase_nolock function\0"
+  "Subject\0[PATCH v2 02/11] clk: add clk_core_set_phase_nolock function\0"
 ]
 [
   "Date\0Thu, 25 May 2017 11:58:41 -0700\0"
 ]
 [
-  "To\0Jerome Brunet <jbrunet\@baylibre.com>",
-  " Adriana Reus <adi.reus\@gmail.com>\0"
-]
-[
-  "Cc\0Stephen Boyd <sboyd\@codeaurora.org>",
-  " Kevin Hilman <khilman\@baylibre.com>",
-  " linux-clk\@vger.kernel.org",
-  " linux-amlogic\@lists.infradead.org",
-  " Linus Walleij <linus.walleij\@linaro.org>",
-  " Boris Brezillon <boris.brezillon\@free-electrons.com>\0"
+  "To\0linus-amlogic\@lists.infradead.org\0"
 ]
 [
   "\0000:1\0"
@@ -40,52 +31,41 @@
 [
   "Quoting Jerome Brunet (2017-05-23 02:48:48)\n",
   "> On Tue, 2017-05-23 at 12:35 +0300, Adriana Reus wrote:\n",
-  "> > On Mon, May 22, 2017 at 12:59 AM, Jerome Brunet <jbrunet\@baylibre.com> =\n",
-  "wrote:\n",
+  "> > On Mon, May 22, 2017 at 12:59 AM, Jerome Brunet <jbrunet\@baylibre.com> wrote:\n",
   "> > > Create a core function for set_phase, as it is done for set_rate and\n",
   "> > > set_parent.\n",
-  "> > > =\n",
-  "\n",
+  "> > > \n",
   "> > > This rework is done to ease the integration of \"protected\" clock\n",
   "> > > functionality.\n",
-  "> > > =\n",
-  "\n",
+  "> > > \n",
   "> > > Signed-off-by: Jerome Brunet <jbrunet\@baylibre.com>\n",
   "> > > ---\n",
-  "> > > =C2=A0drivers/clk/clk.c | 31 +++++++++++++++++++------------\n",
-  "> > > =C2=A01 file changed, 19 insertions(+), 12 deletions(-)\n",
-  "> > > =\n",
-  "\n",
+  "> > > ?drivers/clk/clk.c | 31 +++++++++++++++++++------------\n",
+  "> > > ?1 file changed, 19 insertions(+), 12 deletions(-)\n",
+  "> > > \n",
   "> > > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c\n",
   "> > > index f5c371532509..6031fada37f9 100644\n",
   "> > > --- a/drivers/clk/clk.c\n",
   "> > > +++ b/drivers/clk/clk.c\n",
   "> > > \@\@ -1873,6 +1873,23 \@\@ int clk_set_parent(struct clk *clk, struct clk\n",
   "> > > *parent)\n",
-  "> > > =C2=A0}\n",
-  "> > > =C2=A0EXPORT_SYMBOL_GPL(clk_set_parent);\n",
-  "> > > =\n",
-  "\n",
-  "> > > +static int clk_core_set_phase_nolock(struct clk_core *core, int degr=\n",
-  "ees)\n",
+  "> > > ?}\n",
+  "> > > ?EXPORT_SYMBOL_GPL(clk_set_parent);\n",
+  "> > > \n",
+  "> > > +static int clk_core_set_phase_nolock(struct clk_core *core, int degrees)\n",
   "> > > +{\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0int ret =3D -EINVAL;\n",
+  "> > > +???????int ret = -EINVAL;\n",
   "> > > +\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!core)\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=\n",
-  "=C2=A0=C2=A0=C2=A0=C2=A0return 0;\n",
+  "> > > +???????if (!core)\n",
+  "> > > +???????????????return 0;\n",
   "> > > +\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase(clk->c=\n",
-  "ore, degrees);\n",
-  "> > =\n",
-  "\n",
-  "> > =C2=A0^ trace_clk_set_phase(core, degrees)\n",
-  "> =\n",
-  "\n",
+  "> > > +???????trace_clk_set_phase(clk->core, degrees);\n",
+  "> > \n",
+  "> > ?^ trace_clk_set_phase(core, degrees)\n",
+  "> \n",
   "> Shame ... Once again this is a poor use of 'git add --patch'.\n",
   "> This particular diff ended up in patch 5.\n",
-  "> =\n",
-  "\n",
+  "> \n",
   "> Thanks a lot for catching it!\n",
   "\n",
   "Patch looks good to me overall. Can you reply here with V3? I'll apply\n",
@@ -94,73 +74,55 @@
   "Regards,\n",
   "Mike\n",
   "\n",
-  "> =\n",
-  "\n",
+  "> \n",
   "> > > +\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (core->ops->set_phase)\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=\n",
-  "=C2=A0=C2=A0=C2=A0=C2=A0ret =3D core->ops->set_phase(core->hw, degrees);\n",
+  "> > > +???????if (core->ops->set_phase)\n",
+  "> > > +???????????????ret = core->ops->set_phase(core->hw, degrees);\n",
   "> > > +\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase_comple=\n",
-  "te(core, degrees);\n",
+  "> > > +???????trace_clk_set_phase_complete(core, degrees);\n",
   "> > > +\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0return ret;\n",
+  "> > > +???????return ret;\n",
   "> > > +}\n",
   "> > > +\n",
-  "> > > =C2=A0/**\n",
-  "> > > =C2=A0 * clk_set_phase - adjust the phase shift of a clock signal\n",
-  "> > > =C2=A0 * \@clk: clock signal source\n",
+  "> > > ?/**\n",
+  "> > > ? * clk_set_phase - adjust the phase shift of a clock signal\n",
+  "> > > ? * \@clk: clock signal source\n",
   "> > > \@\@ -1895,7 +1912,7 \@\@ EXPORT_SYMBOL_GPL(clk_set_parent);\n",
-  "> > > =C2=A0 */\n",
-  "> > > =C2=A0int clk_set_phase(struct clk *clk, int degrees)\n",
-  "> > > =C2=A0{\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0int ret =3D -EINVAL;\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0int ret;\n",
-  "> > > =\n",
-  "\n",
-  "> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!clk)\n",
-  "> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=\n",
-  "=A0=C2=A0=C2=A0=C2=A0=C2=A0return 0;\n",
+  "> > > ? */\n",
+  "> > > ?int clk_set_phase(struct clk *clk, int degrees)\n",
+  "> > > ?{\n",
+  "> > > -???????int ret = -EINVAL;\n",
+  "> > > +???????int ret;\n",
+  "> > > \n",
+  "> > > ????????if (!clk)\n",
+  "> > > ????????????????return 0;\n",
   "> > > \@\@ -1906,17 +1923,7 \@\@ int clk_set_phase(struct clk *clk, int degrees)\n",
-  "> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=\n",
-  "=A0=C2=A0=C2=A0=C2=A0=C2=A0degrees +=3D 360;\n",
-  "> > > =\n",
-  "\n",
-  "> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0clk_prepare_lock();\n",
+  "> > > ????????????????degrees += 360;\n",
+  "> > > \n",
+  "> > > ????????clk_prepare_lock();\n",
   "> > > -\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase(clk->c=\n",
-  "ore, degrees);\n",
+  "> > > -???????trace_clk_set_phase(clk->core, degrees);\n",
   "> > > -\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (clk->core->ops->set_ph=\n",
-  "ase)\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=\n",
-  "=C2=A0=C2=A0=C2=A0=C2=A0ret =3D clk->core->ops->set_phase(clk->core->hw, de=\n",
-  "grees);\n",
+  "> > > -???????if (clk->core->ops->set_phase)\n",
+  "> > > -???????????????ret = clk->core->ops->set_phase(clk->core->hw, degrees);\n",
   "> > > -\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0trace_clk_set_phase_comple=\n",
-  "te(clk->core, degrees);\n",
+  "> > > -???????trace_clk_set_phase_complete(clk->core, degrees);\n",
   "> > > -\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if (!ret)\n",
-  "> > > -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=\n",
-  "=C2=A0=C2=A0=C2=A0=C2=A0clk->core->phase =3D degrees;\n",
+  "> > > -???????if (!ret)\n",
+  "> > > -???????????????clk->core->phase = degrees;\n",
   "> > > -\n",
-  "> > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0ret =3D clk_core_set_phase=\n",
-  "_nolock(clk->core, degrees);\n",
-  "> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0clk_prepare_unlock();\n",
-  "> > > =\n",
-  "\n",
-  "> > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0return ret;\n",
+  "> > > +???????ret = clk_core_set_phase_nolock(clk->core, degrees);\n",
+  "> > > ????????clk_prepare_unlock();\n",
+  "> > > \n",
+  "> > > ????????return ret;\n",
   "> > > --\n",
   "> > > 2.9.4\n",
-  "> > > =\n",
-  "\n",
+  "> > > \n",
   "> > > --\n",
-  "> > > To unsubscribe from this list: send the line \"unsubscribe linux-clk\" =\n",
-  "in\n",
-  "> > > the body of a message to majordomo\@vger.kernel.org\n",
-  "> > > More majordomo info at=C2=A0=C2=A0http://vger.kernel.org/majordomo-in=\n",
-  "fo.html\n",
-  ">=20"
+  "> > > To unsubscribe from this list: send the line \"unsubscribe linux-clk\" in\n",
+  "> > > the body of a message to majordomo at vger.kernel.org\n",
+  "> > > More majordomo info at??http://vger.kernel.org/majordomo-info.html\n",
+  ">"
 ]
 
-601acbdce43eb17449645e59d2cc2556eee1ec6dd1f76e2d2c06484d91e09385
+1be8f2062eb8eb1a3a9a96118b5b83167b97aecf0b697a0137129f05ccc0fb80

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.