On Tue, Nov 27, 2012 at 06:32:15AM +0000, Mark Brown wrote: > Avoids surprising users. > > Signed-off-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> > --- > include/linux/input.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/include/linux/input.h b/include/linux/input.h > index 82ce323..0ba0de1 100644 > --- a/include/linux/input.h > +++ b/include/linux/input.h > @@ -475,7 +475,8 @@ extern struct class input_class; > * struct ff_device - force-feedback part of an input device > * @upload: Called to upload an new effect into device > * @erase: Called to erase an effect from device > - * @playback: Called to request device to start playing specified effect > + * @playback: Called to request device to start playing specified effect. > + * Must not sleep. > * @set_gain: Called to set specified gain > * @set_autocenter: Called to auto-center device > * @destroy: called by input core when parent input device is being 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 ;) Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
