Thread (46 messages) flat view 46 messages, 11 authors, 2015-03-13

[PATCH v13 3/6] clk: Make clk API return per-user struct clk instances

From: tony@atomide.com (Tony Lindgren)
Date: 2015-02-03 15:25:15
Also in: linux-omap, lkml

* Tero Kristo [off-list ref] [150203 00:49]:
On 02/03/2015 09:03 AM, Tomeu Vizoso wrote:
quoted
I think you got it right, just wanted to mention that we can and
probably should make the clk_get_parent_* calls in the consumer API to
return per-user clk instances but that we need to make sure first that
callers call clk_put afterwards.

This should also allow us to remove the reference to struct clk from
clk_hw, which is at best awkward.
For the DPLL code it should just be fine to be able to get the current
parent index (not parent clock handle), and read a parent clock rate based
on an arbitrary index (not just the current one.) I don't think there is any
other need for having the clk_ref / clk_bypass clock handles around.
I'd like to avoid the situation where the children have know that
certain parent index and parent rate means bypass mode for both
parent and children. 

Maybe we can hide that knowledge into some Linux generic PLL code so
the children can get the PLL output as a mux clock. For a PLL, there
can be three mux clock outputs: refclk*multi/div, bypass clock, or no
output.

Regards,

Tony
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help