Re: [PATCH] Staging: android: Fix 80 character length

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, Sep 18, 2015 at 02:24:16PM +0530, Punit Vara wrote:
> 	This patch is to the Kconfig file which fixes up lines which
> 	exceeded the standard  80 character limitation.
> 	This file also fixes up 3 warnings regarding paragraph.

Why is this indented?

> 
> Signed-off-by: Punit Vara <punitvara@gmail.com>
> ---
>  drivers/staging/android/Kconfig | 28 ++++++++++++++++------------
>  1 file changed, 16 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/staging/android/Kconfig b/drivers/staging/android/Kconfig
> index 42b1512..f4a2371 100644
> --- a/drivers/staging/android/Kconfig
> +++ b/drivers/staging/android/Kconfig
> @@ -12,7 +12,8 @@ config ASHMEM
>  	  file-based API.
>  
>  	  It is, in theory, a good memory allocator for low-memory devices,
> -	  because it can discard shared memory units when under memory pressure.

This is 80 characters wide, no need to change it.

> +	  because it can discard shared memory units when under memory
> +	  pressure.
>  
>  config ANDROID_TIMED_OUTPUT
>  	bool "Timed output class driver"
> @@ -24,19 +25,20 @@ config ANDROID_TIMED_GPIO
>  	depends on ANDROID_TIMED_OUTPUT
>  	default n
>          ---help---
> -	  Unlike generic gpio is to allow programs to access and manipulate gpio
> -	  registers from user space, timed output/gpio is a system to allow changing
> -	  a gpio pin and restore it automatically after a specified timeout.
> +	  Unlike generic gpio is to allow programs to access and manipulate
> +	  gpio registers from user space, timed output/gpio is a system to
> +	  allow changing a gpio pin and restore it automatically after a
> +	  specified timeout.
>  
>  config ANDROID_LOW_MEMORY_KILLER
>  	bool "Android Low Memory Killer"
>  	---help---
> -	  Registers processes to be killed when low memory conditions, this is useful
> -	  as there is no particular swap space on android.
> +	  Registers processes to be killed when low memory conditions, this
> +	  is useful as there is no particular swap space on android.
>  
> -	  The registered process will kills according to the priorities in android init
> -	  scripts (/init.rc), and it defines priority values with minimum free memory size
> -	  for each priority.
> +	  The registered process will kills according to the priorities in
> +	  android init scripts (/init.rc), and it defines priority values with
> +	  minimum free memory size for each priority.
>  
>  config SYNC
>  	bool "Synchronization framework"
> @@ -44,8 +46,9 @@ config SYNC
>  	select ANON_INODES
>  	select DMA_SHARED_BUFFER
>  	---help---
> -	  This option enables the framework for synchronization between multiple

This is also ok.


> -	  drivers.  Sync implementations can take advantage of hardware
> +	  This option enables the framework for synchronization between
> +	  multiple drivers.
> +	  Sync implementations can take advantage of hardware
>  	  synchronization built into devices like GPUs.
>  
>  config SW_SYNC
> @@ -54,7 +57,8 @@ config SW_SYNC
>  	depends on SYNC
>  	---help---
>  	  A sync object driver that uses a 32bit counter to coordinate
> -	  synchronization.  Useful when there is no hardware primitive backing

And this is just fine, why change it?

thanks,

greg k-h