On Mon, May 13, 2013 at 12:03:09AM -0500, William Hubbs wrote: > Signed-off-by: William Hubbs <w.d.hubbs@xxxxxxxxx> > --- > drivers/staging/speakup/spk_priv.h | 12 ------------ > 1 file changed, 12 deletions(-) > > diff --git a/drivers/staging/speakup/spk_priv.h b/drivers/staging/speakup/spk_priv.h > index 141125c..637ba67 100644 > --- a/drivers/staging/speakup/spk_priv.h > +++ b/drivers/staging/speakup/spk_priv.h > @@ -77,16 +77,4 @@ extern struct speakup_info_t speakup_info; > > extern struct var_t synth_time_vars[]; > > -/* Protect the whole speakup machinery, must be taken at each kernel->speakup > - * transition and released at all corresponding speakup->kernel transitions > - * (flags must be the same variable between lock/trylock and unlock). > - * > - * The progression thread only interferes with the speakup machinery through > - * the synth buffer, and so only needs to take the lock while tinkering with > - * it. > - */ > -/* Speakup needs to disable the keyboard IRQ, hence _irqsave/restore */ These comments are nice. The should go next to the speakup_info definition in drivers/staging/speakup/synth.c. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel