[PATCH 2/4] staging: pi433: Comply with 80 character column limit

Greg KH gregkh at linuxfoundation.org
Tue Jul 10 10:39:19 UTC 2018


On Tue, Jul 10, 2018 at 12:03:17PM +0200, Sophie Matter wrote:
> Lines have been split where it makes sense to shorten them in order
> to comply with the coding standards and fix checkpatch.pl warnings.
> There are still lines left that are too long, however breaking
> those would impair readability.
> 
> Signed-off-by: Sophie Matter <sophie.matter at web.de>
> Signed-off-by: Rico Schrage <rico.schrage at gmail.com>
> ---
>  drivers/staging/pi433/pi433_if.c | 18 ++++++++++++------
>  drivers/staging/pi433/pi433_if.h |  6 ++++--
>  drivers/staging/pi433/rf69.c     | 10 +++++++---
>  3 files changed, 23 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
> index e2b8a518dafe..a58c535c6638 100644
> --- a/drivers/staging/pi433/pi433_if.c
> +++ b/drivers/staging/pi433/pi433_if.c
> @@ -55,10 +55,10 @@
>  #include "pi433_if.h"
>  #include "rf69.h"
>  
> -#define N_PI433_MINORS			BIT(MINORBITS) /*32*/	/* ... up to 256 */
> -#define MAX_MSG_SIZE			900	/* min: FIFO_SIZE! */
> -#define MSG_FIFO_SIZE			65536   /* 65536 = 2^16  */
> -#define NUM_DIO				2
> +#define N_PI433_MINORS BIT(MINORBITS) /*32*/	/* ... up to 256 */
> +#define MAX_MSG_SIZE 900 /* min: FIFO_SIZE! */
> +#define MSG_FIFO_SIZE 65536 /* 65536 = 2^16  */
> +#define NUM_DIO	2

Ick, is that now easier or harder to read?  I think harder :(

Make code easier for humans to read, we are the ones that have to modify
it.

thanks,

greg k-h


More information about the devel mailing list