linux.git/include/linux/clk-provider.h, branch v4.6-rc2

clk: Make of_clk_get_parent_count() return unsigned ints

2016-02-27T00:01:32+00:00

Russell King recently pointed out a bug in the clk-gpio code
where it fails to register the clk if of_clk_get_parent_count()
returns an error because the "clocks" property isn't present in
the DT node. If we're trying to count parents from DT we'd like
to know the count, not if there is a "clocks" property or not.
Furthermore, some drivers are assigning the return value to their
clk_init_data::num_parents member which is unsigned, leading to
potentially large numbers of parents when the property isn't
present.

Let's change the API to return an unsigned int instead of an int.
All the callers just want to know the count anyway, and this
avoids the bug that was in the clk-gpio driver.

Signed-off-by: Stephen Boyd

clk: provider: Remove of_gpio_{gate,mux}_clk_setup() prototypes

2016-02-07T07:40:27+00:00

These functions either never existed or were only used in
OF_CLK_DECLARE() macros. Remove the dead prototypes.

Cc: Jyri Sarha 
Signed-off-by: Stephen Boyd

clk: Deprecate CLK_IS_ROOT

2016-02-07T07:38:26+00:00

We don't use CLK_IS_ROOT but in a few places in the common clk
framework core. Let's replace those checks with a check for the
number of parents a clk has instead of the flag, freeing up one
flag for something else. We don't remove the flag yet so that
things keep building, but we'll remove it once all drivers have
removed their flag usage.

Signed-off-by: Stephen Boyd

clk: add clk_unregister_fixed_rate()

2016-01-30T00:37:24+00:00

Allow to unregister fixed rate clock.

Signed-off-by: Masahiro Yamada 
Signed-off-by: Stephen Boyd

clk: add clk_unregister_fixed_factor()

2016-01-30T00:35:49+00:00

Allow to unregister fixed factor clock.

Signed-off-by: Masahiro Yamada 
Signed-off-by: Stephen Boyd

clk: move the common clock's to_clk_*(_hw) macros to clk-provider.h

2016-01-29T20:59:50+00:00

to_clk_*(_hw) macros have been repeatedly defined in many places.
This patch moves all the to_clk_*(_hw) definitions in the common
clock framework to public header clk-provider.h, and drop the local
definitions.

Signed-off-by: Geliang Tang 
Signed-off-by: Stephen Boyd

clk-divider: make sure read-only dividers do not write to their register

2016-01-29T20:44:37+00:00

Commit e6d5e7d90be9 ("clk-divider: Fix READ_ONLY when divider > 1") removed
the special ops struct for read-only clocks and instead opted to handle
them inside the regular ops.

On the rk3368 this results in breakage as aclkm now gets set a value.
While it is the same divider value, the A53 core still doesn't like it,
which can result in the cpu ending up in a hang.
The reason being that "ACLKENMasserts one clock cycle before the rising
edge of ACLKM" and the clock should only be touched when STANDBYWFIL2
is asserted.

To fix this, reintroduce the read-only ops but do include the round_rate
callback. That way no writes that may be unsafe are done to the divider
register in any case.

The Rockchip use of the clk_divider_ops is adapted to this split again,
as is the nxp, lpc18xx-ccu driver that was included since the original
commit. On lpc18xx-ccu the divider seems to always be read-only
so only uses the new ops now.

Fixes: e6d5e7d90be9 ("clk-divider: Fix READ_ONLY when divider > 1")
Reported-by: Zhang Qing 
Signed-off-by: Heiko Stuebner 
Signed-off-by: Stephen Boyd

Merge branch 'clk-rockchip' into clk-next

2015-12-23T21:08:56+00:00

clk: add flag for clocks that need to be enabled on rate changes

2015-12-23T20:57:31+00:00

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 
Tested-by: Sjoerd Simons 
Reviewed-by: Sjoerd Simons 
Signed-off-by: Michael Turquette

clk: Use static inline functions instead of macros for dummies

2015-11-20T19:16:31+00:00

if CONFIG_OF=n:

    drivers/clk/clk-cs2000-cp.c: In function ‘cs2000_remove’:
    drivers/clk/clk-cs2000-cp.c:453:22: warning: unused variable ‘np’ [-Wunused-variable]
      struct device_node *np = dev->of_node;
			  ^

Convert dummies of_clk_del_provider() and of_clk_init() from macros to
static inline functions to kill such compiler warnings.

Reported-by: kbuild test robot 
Signed-off-by: Geert Uytterhoeven 
Signed-off-by: Stephen Boyd