BMC/Intel-BMC/linux.git/drivers/clk/clk.c, branch dev-4.4

clk: Add clk_hw_is_enabled() for use by clk providers

2015-10-26T19:36:55+00:00

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 Signed-off-by: Stephen Boyd

clk: add missing of_node_put

2015-10-21T23:16:32+00:00

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): // @@ 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 ...; ) ... } // 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 Signed-off-by: Stephen Boyd

clk: Make clk input parameter of __clk_get_name() const

2015-10-16T18:56:19+00:00

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 Signed-off-by: Stephen Boyd

clk: Use %u to format unsigned int in of_clk_src_onecell_get()

2015-10-16T18:49:01+00:00

Signed-off-by: Geert Uytterhoeven Signed-off-by: Stephen Boyd

clk: Make of_clk_get_parent_name() robust with #clock-cells = 1

2015-10-15T23:17:03+00:00

If a clock provider has #clock-cells = 1 and we call of_clk_get_parent_name() on it we may end up returning the name of the provider node if the provider doesn't have a clock-output-names property. This doesn't make sense, especially when you consider that calling of_clk_get_parent_name() on such a node with different indices will return the same name each time. Let's try getting the clock from the framework via of_clk_get() instead, and only fallback to the node name if we have a provider with #clock-cells = 0. This way, we can't hand out the same name for different clocks when we don't actually know their names. Cc: Thomas Petazzoni Signed-off-by: Stephen Boyd

Merge branch 'clk-fixes' into clk-next

2015-09-17T19:28:41+00:00

* clk-fixes: drivers: clk: st: Rename st_pll3200c32_407_c0_x into st_pll3200c32_cx_x clk: check for invalid parent index of orphans in __clk_init()

clk: Remove unneeded semicolons

2015-09-17T18:15:14+00:00

There are cleary typo errors so can be removed. Signed-off-by: Javier Martinez Canillas Acked-by: Leo Yan Signed-off-by: Stephen Boyd

clk: check for invalid parent index of orphans in __clk_init()

2015-09-16T22:35:18+00:00

If a mux clock is initialised (by hardware or firmware) with an invalid parent, its ->get_parent() can return an out of range index. For example, the generic mux clock attempts to return -EINVAL, which due to the u8 return type ends up a rather large number. Using this index with the parent_names[] array results in an invalid pointer and (usually) a crash in the following strcmp(). This patch adds a check for the parent index being in range, ignoring clocks reporting invalid values. Signed-off-by: Mans Rullgard Tested-by: Rhyland Klein Signed-off-by: Stephen Boyd

clk: remove duplicated code with __clk_set_parent_after

2015-08-24T23:49:13+00:00

__clk_set_parent_after() actually used the second argument then we could put this duplicate logic in there and call it with a different order of arguments in the success vs. error paths in this function. Cc: Mike Turquette Suggested-by: Stephen Boyd Signed-off-by: Dong Aisheng Signed-off-by: Stephen Boyd

clk: Constify clk_hw argument to provider APIs

2015-08-24T23:49:11+00:00

We don't modify the clk_hw argument in these functions, so it's safe to mark it as const. Signed-off-by: Stephen Boyd