On 29-07-21, 07:13, Thara Gopinath wrote:

So the interrupt handler is a threaded handler. I moved it in v4 since one
of the "_opp" api has an underlying mutex and was causing issues. So using a
mutex should be pretty safe in this case.

Ahh I see.

IIUC, the main change you are suggesting below is to include enable_irq() /
mod_delayed_work() under the spin_lock as well. Is that right ? In which
case isn't a mutex better than spinlock?

Yeah, sure.

-- 
viresh