Re: [PATCH 17/17] staging: speakup: remove custom locking macro definitions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux