On Mon, May 13, 2013 at 12:03:09AM -0500, William Hubbs wrote:
> Signed-off-by: William Hubbs <[email protected]>
> ---
>  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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to