Re: [PATCH 2/2] se401: Fix coding style

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

 



Alan Cox wrote:
> From: Alan Cox <alan@xxxxxxxxxxxxxxx>
> 
> Having fixed the sprintfs I decided a quick clean wouldn't do any harm so
> it was actually easy to read in future.
> 
> Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
> ---
> 
>  drivers/media/video/se401.c |  876 ++++++++++++++++++++++---------------------
>  drivers/media/video/se401.h |    7 
>  2 files changed, 452 insertions(+), 431 deletions(-)
> 
> 
> diff --git a/drivers/media/video/se401.c b/drivers/media/video/se401.c
> index 08129a8..c8f0529 100644
> --- a/drivers/media/video/se401.c
> +++ b/drivers/media/video/se401.c

[...]

> @@ -78,8 +79,8 @@ static void *rvmalloc(unsigned long size)
>  	adr = (unsigned long) mem;
>  	while (size > 0) {
>  		SetPageReserved(vmalloc_to_page((void *)adr));
> -		adr += PAGE_SIZE;
> -		size -= PAGE_SIZE;
> +		adr +=  PAGE_SIZE;
> +		size -=  PAGE_SIZE;

Why 2 spaces are better here?

>  	}
>  
>  	return mem;
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux