On Thu, Aug 02, 2007 at 04:32:13PM -0700, David Brownell wrote: > On Wednesday 01 August 2007, Christoph Hellwig wrote: > > On Wed, Aug 01, 2007 at 09:28:07AM +0200, Domen Puncer wrote: > > > > It doesn't make any assumption on struct clk, it's just a > > > > wrapper around functions from clk.h. > > > > Point of this patch was to abstract exported functions, since > > > > arch/powerpc/ can support multiple platfroms in one binary. > > > > > > So... the thread just ended without any consensus, ACK or whatever. > > > > > > Choices I see: > > > - have EXPORT_SYMBOL for clk.h functions in ie. lib/clock.c and have > > > every implemenation fill some global struct. > > > - leave this patch as it is, abstraction only for arch/powerpc/. > > That seems the best solution for now, I agree. I've not been avoiding replying further to this thread in spite, it's just that I've not had any time what so ever to look at this further. It's very low priority for me at the moment, so it's getting zero time, and will continue to be in that state for at least the next month and a bit. Sorry. -- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: