[PATCH v2 1/3] staging: pi433: Comply with 80 character column limit

2018-07-11 Thread Sophie Matter
staging tree - the changes to the defines previously made are deleted due to readability Signed-off-by: Sophie Matter Signed-off-by: Rico Schrage --- drivers/staging/pi433/pi433_if.c | 13 ++--- drivers/staging/pi433/pi433_if.h | 6 -- drivers/staging/pi433/rf69.c | 10

[PATCH v2 2/3] staging: pi433: Use preferred commenting style

2018-07-11 Thread Sophie Matter
this line would not make much sense, so it is kept like it is. Signed-off-by: Sophie Matter Signed-off-by: Rico Schrage --- drivers/staging/pi433/pi433_if.c | 30 -- drivers/staging/pi433/rf69.c | 15 ++- 2 files changed, 30 insertions(+), 15 deletions

[PATCH v2 3/3] staging: pi433: Make only one statement per line

2018-07-11 Thread Sophie Matter
Lines containing multiple statements were broken into multiple lines, increasing readability and complying with the coding standard. This also fixes several checkpatch.pl errors complaining about the lines being too long. Signed-off-by: Sophie Matter Signed-off-by: Rico Schrage --- drivers

[PATCH 3/4] staging: pi433: Use preferred commenting style

2018-07-10 Thread Sophie Matter
this line would not make much sense, so it is kept like it is. Signed-off-by: Sophie Matter Signed-off-by: Rico Schrage --- drivers/staging/pi433/pi433_if.c | 33 +--- drivers/staging/pi433/rf69.c | 6 -- 2 files changed, 26 insertions(+), 13 deletions

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

2018-07-10 Thread Sophie Matter
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 Signed-off-by: Rico Schrage

[PATCH 1/4] staging: pi433: Add SPDX-License-Identifier tag

2018-07-10 Thread Sophie Matter
The missing SPDX tag was added to the top of the files of this driver, according to the comments on top of each file (GPL license or higher). Signed-off-by: Sophie Matter Signed-off-by: Rico Schrage --- drivers/staging/pi433/pi433_if.c | 1 + drivers/staging/pi433/pi433_if.h | 1

[PATCH 4/4] staging: pi433: Make only one statement per line

2018-07-10 Thread Sophie Matter
Lines containing multiple statements were broken into multiple lines, increasing readability and complying with the coding standard. This also fixes several checkpatch.pl errors complaining about the lines being too long. Signed-off-by: Sophie Matter Signed-off-by: Rico Schrage --- drivers