Re: [PATCH] staging: line6: Fix checkpatch.pl switch - case issue

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

 



On Thu, Nov 29, 2012 at 12:21:22PM +0100, Laurent Navet [Mali] wrote:
> From: "Laurent Navet [Mali]" <laurent.navet@xxxxxxxxx>
> 
> delete old commentted code to avoid this checkpatch error :
>  line6/pcm.c:423: ERROR: switch and case should be at the same indent
> 
> Signed-off-by: Laurent Navet [Mali] <laurent.navet@xxxxxxxxx>

What is the [Mali] thing for?

> ---
>  drivers/staging/line6/pcm.c |    8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/drivers/staging/line6/pcm.c b/drivers/staging/line6/pcm.c
> index 6c1e313..a0ce781 100644
> --- a/drivers/staging/line6/pcm.c
> +++ b/drivers/staging/line6/pcm.c
> @@ -455,14 +455,6 @@ int line6_init_pcm(struct usb_line6 *line6,
>  		ep_write = 0x01;
>  		break;
>  
> -		/* this is for interface_number == 1:
> -		   case LINE6_DEVID_TONEPORT_UX2:
> -		   case LINE6_DEVID_PODSTUDIO_UX2:
> -		   ep_read  = 0x87;
> -		   ep_write = 0x00;
> -		   break;
> -		 */

Why not properly indent it to keep the comment around instead?

thanks,

greg k-h
_______________________________________________
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