diff for duplicates of <20170804001836.GU2146@codeaurora.org> diff --git a/a/1.txt b/N1/1.txt index ae52f8a..bd6e135 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -47,16 +47,16 @@ Sounds fine. We can take this up on another thread. > > [..] > > > + > > > @@ -2952,6 +3134,17 @@ void __clk_put(struct clk *clk) -> > > ? -> > > ? clk_prepare_lock(); -> > > ? +> > > +> > > clk_prepare_lock(); +> > > > > > + /* -> > > + ?* Before calling clk_put, all calls to clk_rate_protect from a +> > > + * Before calling clk_put, all calls to clk_rate_protect from a > > > given -> > > + ?* user must be balanced with calls to clk_rate_unprotect and by +> > > + * user must be balanced with calls to clk_rate_unprotect and by > > > that -> > > + ?* same user -> > > + ?*/ +> > > + * same user +> > > + */ > > > + WARN_ON(clk->protect_count); > > > + > > > + /* We voiced our concern, let's sanitize the situation */ @@ -113,9 +113,9 @@ returned. > > > +++ b/include/linux/clk.h > > > @@ -331,6 +331,30 @@ struct clk *devm_clk_get(struct device *dev, const char > > > *id); -> > > ? */ -> > > ?struct clk *devm_get_clk_from_child(struct device *dev, -> > > ? ????struct device_node *np, const char +> > > */ +> > > struct clk *devm_get_clk_from_child(struct device *dev, +> > > struct device_node *np, const char > > > *con_id); > > > +/** > > > + * clk_rate_protect - inform the system when the clock rate must be @@ -141,7 +141,7 @@ returned. > By the now, you may have forgot but I explained the use-case in first RFC [0] > Here is an example (wip) of usage [1] > -> [0]: http://lkml.kernel.org/r/20170302173835.18313-1-jbrunet at baylibre.com +> [0]: http://lkml.kernel.org/r/20170302173835.18313-1-jbrunet@baylibre.com > [1]: https://github.com/jeromebrunet/linux/commits/amlogic/wip/audio-clk-lock If we're forgetting why something is introduced then it means the diff --git a/a/content_digest b/N1/content_digest index 7394c25..b16adc4 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,10 +2,17 @@ "ref\020170612194438.12298-6-jbrunet@baylibre.com\0" "ref\020170726001217.GC2146@codeaurora.org\0" "ref\01501089516.2401.29.camel@baylibre.com\0" - "From\0sboyd@codeaurora.org (Stephen Boyd)\0" - "Subject\0[PATCH v3 05/10] clk: add support for clock protection\0" + "From\0Stephen Boyd <sboyd@codeaurora.org>\0" + "Subject\0Re: [PATCH v3 05/10] clk: add support for clock protection\0" "Date\0Thu, 3 Aug 2017 17:18:36 -0700\0" - "To\0linus-amlogic@lists.infradead.org\0" + "To\0Jerome Brunet <jbrunet@baylibre.com>\0" + "Cc\0Michael Turquette <mturquette@baylibre.com>" + linux-clk@vger.kernel.org + Kevin Hilman <khilman@baylibre.com> + linux-amlogic@lists.infradead.org + Russell King <linux@armlinux.org.uk> + Linus Walleij <linus.walleij@linaro.org> + " Boris Brezillon <boris.brezillon@free-electrons.com>\0" "\00:1\0" "b\0" "On 07/26, Jerome Brunet wrote:\n" @@ -57,16 +64,16 @@ "> > [..]\n" "> > > +\n" "> > > @@ -2952,6 +3134,17 @@ void __clk_put(struct clk *clk)\n" - "> > > ?\n" - "> > > ?\tclk_prepare_lock();\n" - "> > > ?\n" + "> > > \302\240\n" + "> > > \302\240\tclk_prepare_lock();\n" + "> > > \302\240\n" "> > > +\t/*\n" - "> > > +\t?* Before calling clk_put, all calls to clk_rate_protect from a\n" + "> > > +\t\302\240* Before calling clk_put, all calls to clk_rate_protect from a\n" "> > > given\n" - "> > > +\t?* user must be balanced with calls to clk_rate_unprotect and by\n" + "> > > +\t\302\240* user must be balanced with calls to clk_rate_unprotect and by\n" "> > > that\n" - "> > > +\t?* same user\n" - "> > > +\t?*/\n" + "> > > +\t\302\240* same user\n" + "> > > +\t\302\240*/\n" "> > > +\tWARN_ON(clk->protect_count);\n" "> > > +\n" "> > > +\t/* We voiced our concern, let's sanitize the situation */\n" @@ -123,9 +130,9 @@ "> > > +++ b/include/linux/clk.h\n" "> > > @@ -331,6 +331,30 @@ struct clk *devm_clk_get(struct device *dev, const char\n" "> > > *id);\n" - "> > > ? */\n" - "> > > ?struct clk *devm_get_clk_from_child(struct device *dev,\n" - "> > > ?\t\t\t\t????struct device_node *np, const char\n" + "> > > \302\240 */\n" + "> > > \302\240struct clk *devm_get_clk_from_child(struct device *dev,\n" + "> > > \302\240\t\t\t\t\302\240\302\240\302\240\302\240struct device_node *np, const char\n" "> > > *con_id);\n" "> > > +/**\n" "> > > + * clk_rate_protect - inform the system when the clock rate must be\n" @@ -151,7 +158,7 @@ "> By the now, you may have forgot but I explained the use-case in first RFC [0]\n" "> Here is an example (wip) of usage [1]\n" "> \n" - "> [0]: http://lkml.kernel.org/r/20170302173835.18313-1-jbrunet at baylibre.com\n" + "> [0]: http://lkml.kernel.org/r/20170302173835.18313-1-jbrunet@baylibre.com\n" "> [1]: https://github.com/jeromebrunet/linux/commits/amlogic/wip/audio-clk-lock\n" "\n" "If we're forgetting why something is introduced then it means the\n" @@ -240,4 +247,4 @@ "Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,\n" a Linux Foundation Collaborative Project -5748b721601ba38a846602671c983e0429b92badebe6db95de3122d2b6b26ff0 +42b3b9ba9184b26c72807bbb1068e923de367fdd967502f9a1fa9ecc336af0db
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.