On Fri, Mar 17, 2017 at 06:28:29PM +0100, Bartlomiej Zolnierkiewicz wrote:
> Hi,
> 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.

Best regards,

Reply via email to