<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/drivers/clk/clk.c, branch linux-4.5.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-4.5.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-4.5.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2015-12-23T21:08:56Z</updated>
<entry>
<title>Merge branch 'clk-rockchip' into clk-next</title>
<updated>2015-12-23T21:08:56Z</updated>
<author>
<name>Michael Turquette</name>
</author>
<published>2015-12-23T21:08:56Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=a915e30dd26ea5f3cc2e2c044aba38ee5973d3fa'/>
<id>urn:sha1:a915e30dd26ea5f3cc2e2c044aba38ee5973d3fa</id>
<content type='text'>
</content>
</entry>
<entry>
<title>clk: add flag for clocks that need to be enabled on rate changes</title>
<updated>2015-12-23T20:57:31Z</updated>
<author>
<name>Heiko Stuebner</name>
</author>
<published>2015-12-22T21:27:58Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=2eb8c7104c648ad4bfae1f5333f98c09522149b5'/>
<id>urn:sha1:2eb8c7104c648ad4bfae1f5333f98c09522149b5</id>
<content type='text'>
Some clocks need to be enabled to accept rate changes. This patch adds a
new flag CLK_SET_RATE_UNGATE that lets clk_change_rate enable the clock
before trying to change the rate and disable it again afterwards.
This of course doesn't effect clocks that are already running at that
point, as their refcount will only temporarily increase.

Signed-off-by: Heiko Stuebner &lt;heiko@sntech.de&gt;
Tested-by: Sjoerd Simons &lt;sjoerd.simons@collabora.co.uk&gt;
Reviewed-by: Sjoerd Simons &lt;sjoerd.simons@collabora.co.uk&gt;
Signed-off-by: Michael Turquette &lt;mturquette@baylibre.com&gt;
</content>
</entry>
<entry>
<title>clk: let of_clk_get_parent_name() fail for invalid clock-indices</title>
<updated>2015-12-03T06:34:43Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2015-12-03T02:20:35Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=8da411cc1964c5d6a0a0f94c7bbf8693c9673a87'/>
<id>urn:sha1:8da411cc1964c5d6a0a0f94c7bbf8693c9673a87</id>
<content type='text'>
Currently, of_clk_get_parent_name() returns a wrong parent clock name
when "clock-indices" property exists and the target index is not
found in the property.  In this case, NULL should be returned.

For example,

        oscillator {
                compatible = "myclocktype";
                #clock-cells = &lt;1&gt;;
                clock-indices = &lt;1&gt;, &lt;3&gt;;
                clock-output-names = "clka", "clkb";
        };

        consumer {
                compatible = "myclockconsumer";
                clocks = &lt;&amp;oscillator 0&gt;, &lt;&amp;oscillator 1&gt;;
        };

Currently, of_clk_get_parent_name(consumer_np, 0) returns "clka"
(and of_clk_get_parent_name(consumer_np, 1) also returns "clka",
this is correct).   Because the "clock-indices" in the clock parent
does not contain &lt;0&gt;, of_clk_get_parent_name(consumer_np, 0) should
return NULL.

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: fix a typo in comment block of clk_notifier_register()</title>
<updated>2015-11-30T19:36:07Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2015-11-30T07:40:51Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=198bb59493f810ece01e56e9694bfdb39b5aa056'/>
<id>urn:sha1:198bb59493f810ece01e56e9694bfdb39b5aa056</id>
<content type='text'>
The word "cases" is doubled.  Keep decent forms for the following
lines.

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: Spelling s/derefing/dereferencing/</title>
<updated>2015-11-20T19:17:55Z</updated>
<author>
<name>Geert Uytterhoeven</name>
</author>
<published>2015-10-29T19:55:00Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=3fe003f944755f6d959387f9568d271512dcb12d'/>
<id>urn:sha1:3fe003f944755f6d959387f9568d271512dcb12d</id>
<content type='text'>
Signed-off-by: Geert Uytterhoeven &lt;geert+renesas@glider.be&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: use IS_ERR_OR_NULL(hw) instead of !hw || IS_ERR(hw)</title>
<updated>2015-11-20T17:20:54Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2015-11-20T05:38:49Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=c1de13574d7880c7321abe789ef02fa540631eaf'/>
<id>urn:sha1:c1de13574d7880c7321abe789ef02fa540631eaf</id>
<content type='text'>
This minor refactoring does not change the function behavior.

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: remove redundant negative index check in of_clk_get_parent_name()</title>
<updated>2015-11-20T17:18:24Z</updated>
<author>
<name>Masahiro Yamada</name>
</author>
<published>2015-11-20T07:36:19Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=c736c4e11e9def555482f3f626254594d3f11f9c'/>
<id>urn:sha1:c736c4e11e9def555482f3f626254594d3f11f9c</id>
<content type='text'>
This if-block can be dropped because the of_parse_phandle_with_args()
in the following line returns -EINVAL for negative index.

Signed-off-by: Masahiro Yamada &lt;yamada.masahiro@socionext.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: Add clk_hw_is_enabled() for use by clk providers</title>
<updated>2015-10-26T19:36:55Z</updated>
<author>
<name>Joachim Eastwood</name>
</author>
<published>2015-10-24T16:55:22Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=be68bf883170b3e4123fc4ff3745e38fb45a573e'/>
<id>urn:sha1:be68bf883170b3e4123fc4ff3745e38fb45a573e</id>
<content type='text'>
Add clk_hw_is_enabled() to the provider APIs so clk providers can
use a struct clk_hw instead of a struct clk to check if a clk is
enabled or not.

Signed-off-by: Joachim Eastwood &lt;manabian@gmail.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: add missing of_node_put</title>
<updated>2015-10-21T23:16:32Z</updated>
<author>
<name>Julia Lawall</name>
</author>
<published>2015-10-21T20:41:36Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=6bc9d9d62cbc885414c7d4bb1926c43950498479'/>
<id>urn:sha1:6bc9d9d62cbc885414c7d4bb1926c43950498479</id>
<content type='text'>
for_each_matching_node_and_match performs an of_node_get on each iteration,
so a break out of the loop requires an of_node_put.

A simplified version of the semantic patch that fixes this problem is as
follows (http://coccinelle.lip6.fr):

// &lt;smpl&gt;
@@
expression e1,e2,e;
local idexpression np;
@@

 for_each_matching_node_and_match(np, e1, e2) {
   ... when != of_node_put(np)
       when != e = np
(
   return np;
|
+  of_node_put(np);
?  return ...;
)
   ...
 }
// &lt;/smpl&gt;

Besides the problem identified by the semantic patch, this patch adds an
of_node_get in front of saving np in a field of parent, to account for the
fact that this value will be put on going on to the next element in the
iteration, and then adds of_node_puts in the two loops where the parent
pointer can be freed.

Signed-off-by: Julia Lawall &lt;Julia.Lawall@lip6.fr&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>clk: Make clk input parameter of __clk_get_name() const</title>
<updated>2015-10-16T18:56:19Z</updated>
<author>
<name>Geert Uytterhoeven</name>
</author>
<published>2015-10-16T12:35:21Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=b76281cb97761002277730432812b1687de96062'/>
<id>urn:sha1:b76281cb97761002277730432812b1687de96062</id>
<content type='text'>
When calling __clk_get_name() on a const clock:

    warning: passing argument 1 of '__clk_get_name' discards 'const' qualifier from pointer target type
    include/linux/clk-provider.h:613:13: note: expected 'struct clk *' but argument is of type 'const struct clk *'

__clk_get_name() does not modify the passed clock, hence make it const.

Signed-off-by: Geert Uytterhoeven &lt;geert+renesas@glider.be&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
</content>
</entry>
</feed>
