On Mon, Nov 26, 2012 at 10:42:38PM -0800, Dmitry Torokhov wrote:

We already have a few lines below:

quoted

* Note that playback(), set_gain() and set_autocenter() are called with
* dev->event_lock spinlock held and interrupts off and thus may not
* sleep.

So I doubt they will pay more attention to your addition ;)

Ah, it's quite a few lines below - it was actually off the bottom of my
screen which rather buried the lead (though TBH I think I was most
surprised by the fact that the lock isn't actually in the struct).