Thread (16 messages) 16 messages, 5 authors, 2012-07-31

[PATCH v3 1/3] ARM: omap: clk: add clk_prepare and clk_unprepare

From: Russell King - ARM Linux <hidden>
Date: 2012-07-31 09:27:14
Also in: linux-arm-msm, linux-omap

On Tue, Jul 31, 2012 at 01:21:24AM -0700, Saravana Kannan wrote:
I have heard this idea about removing the clk_prepare/unprepare API too  
many times and it makes me uncomfortable. I would really prefer we (the  
community) take this discussion to the end and put an end to it. We  
either agree to stick with the clk_prepare APIs or figure out the newer  
APIs. I don't want to keep having to deal with the "we really should be  
removing the clk_prepare() APIs" wrench thrown into the discussion every  
time we discuss anything related to a locking issue.
Well, part of the motivation to remove it comes from the fact that we
have this "clk_prepare_enable()" thing which _everyone_ simply converts
their existing clk_enable() call to without _thinking_ one little bit
about the rest of the picture.

It's that which is making the justification for getting rid of the split
API soo easy.  If people put more thought into it then the separation of
the two calls would be much clearer, and there would be more of a
justification to prevent its removal.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help