linux.git/drivers/clk/ti, branch v4.12

clk: ti: divider: try to fix ti_clk_register_divider

2017-04-28T18:07:09+00:00

Commit 6c0afb503937 ("clk: ti: convert to use proper register
definition for all accesses") converted all register accesses in
the TI clk driver to use a proper struct instead of a void
pointer casted struct that fits into a u32. Unfortunately, it
missed a conversion here in the didivder code, leading to a
compiler warning like so:

drivers/clk/ti/divider.c: In function 'ti_clk_register_divider':
drivers/clk/ti/divider.c:460:8: error: 'reg' may be used uninitialized in this function [-Werror=maybe-uninitialized]

Treating a 'u32' variable as a structure leads to a stack
overflow here, and the register address we pass down is never
initialized. Convert this part of the code as well so things
work properly.

Fixes: 6c0afb503937 ("clk: ti: convert to use proper register definition for all accesses")
Signed-off-by: Arnd Bergmann 
[sboyd@codeaurora.org: Fixed fixes tag, rewrote commit message,
s/reg_setup/reg/]
Signed-off-by: Stephen Boyd

clk: ti: fix building without legacy omap3

2017-04-22T02:01:39+00:00

When CONFIG_ATAGS or CONFIG_OMAP3 is disabled, we get a build error:

In file included from include/linux/clk-provider.h:15:0,
                 from drivers/clk/ti/clk.c:19:
drivers/clk/ti/clk.c: In function 'ti_clk_add_aliases':
drivers/clk/ti/clk.c:438:29: error: 'simple_clk_match_table' undeclared (first use in this function); did you mean 'simple_attr_write'?

Moving the match table down fixes it.

Fixes: c17435c56bb1 ("clk: ti: add API for creating aliases automatically for simple clock types")
Signed-off-by: Arnd Bergmann 
Acked-by: Tero Kristo 
Signed-off-by: Stephen Boyd

clk: ti: fix linker error with !SOC_OMAP4

2017-04-22T02:01:18+00:00

When none of the OMAP4-generation SoCs are enabled, we run into a link
error for am43xx/am43xx:

drivers/clk/ti/dpll.o: In function `of_ti_am3_dpll_x2_setup':
dpll.c:(.init.text+0xd8): undefined reference to `clkhwops_omap4_dpllmx'

This is easily fixed by adding another #ifdef.

While looking at the code, I also spotted another problem with the
assignment of hw_ops variable that is not used again later. I'm
changing this to setting clk_hw->ops instead, which I guess is what
was intended here.

Fixes: 473adbf4e028 ("clk: ti: dpll44xx: fix clksel register initialization")
Signed-off-by: Arnd Bergmann 
Acked-by: Tero Kristo 
[sboyd@codeaurora.org: Replaced fixes tag with correct one]
Signed-off-by: Stephen Boyd

clk: ti: convert to use proper register definition for all accesses

2017-03-08T11:06:15+00:00

Currently, TI clock driver uses an encapsulated struct that is cast into
a void pointer to store all register addresses. This can be considered
as rather nasty hackery, and prevents from expanding the register
address field also. Instead, replace all the code to use proper struct
in place for this, which contains all the previously used data.

This patch is rather large as it is touching multiple files, but this
can't be split up as we need to avoid any boot breakage.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren

clk: ti: dpll44xx: fix clksel register initialization

2017-03-08T11:05:42+00:00

clksel register pointer should be used for the DPLL-MX autoidle handling.
Currently this is not setup at all. Fix by adding proper handling for the
register.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren

clk: ti: gate: export gate_clk_ops locally

2017-03-08T11:05:03+00:00

These are going to be used by the clkctrl support that will be introduced
later.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren

clk: ti: divider: add driver internal API for parsing divider data

2017-03-08T11:04:38+00:00

This can be used from the divider itself, and also from the clkctrl
clocks once this is introduced.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren

clk: ti: divider: convert TI divider clock to use its own data representation

2017-03-08T11:04:06+00:00

Instead of using the generic clock driver data struct, use one internal
for the TI clock driver itself. This allows modifying the register access
parts in subsequent patch.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren

clk: ti: mux: convert TI mux clock to use its internal data representation

2017-03-08T11:03:22+00:00

Instead of using the generic clock driver data struct, use one internal
for the TI clock driver itself. This allows modifying the register access
parts in subsequent patch.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren

clk: ti: drop unnecessary MEMMAP_ADDRESSING flag

2017-03-08T11:03:07+00:00

This has been superceded by the usage of ti_clk_ll_ops for now.

Signed-off-by: Tero Kristo 
Acked-by: Tony Lindgren