Hi Russell,
Unless the locking problems can be resolved, the patches aren't ready.
From what I've seen there's still quite a problem with what kind of
lock to use in the clock - mutex or spinlock.
Yes, the clock driver may either use a spinlock or mutex.
However, this exactly the same as the current clock code, my patches do not
alter what we currently have.
I do agree that we should define some specific semantics for the clock API
with regards to sleeping, and I'll start a new thread about that. But we
should definitely separate that issue from the problem of having multiple
definitions for struct clk, which is what these patches address.
Cheers,
Jeremy