On Mon, Nov 26, 2012 at 10:42:38PM -0800, Dmitry Torokhov wrote: > We already have a few lines below: > > * 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).
Attachment:
signature.asc
Description: Digital signature