On Fri, Mar 17, 2017 at 06:28:29PM +0100, Bartlomiej Zolnierkiewicz wrote:
> On Friday, March 17, 2017 04:49:22 PM Krzysztof Kozlowski wrote:
> > Driver uses threaded interrupt handler so there is no real need for
> > using spinlocks for synchronization. Mutexes would do fine and are
> > friendlier for overall system preemptivness and real-time behavior.
> Are you sure that this conversion is safe? This driver also uses
> a tasklet and tasklets run in the interrupt context.
Yes, you're right. This is not safe and patch should be dropped. Thanks
for spotting this.