[PATCH 4/4] drivers: staging: remove BUG_ON

2016-04-17 Thread tcharding
drivers/staging/android/ion/ion.c calls BUG_ON in places where WARN_ON will suffice. This patch replaces two such occurences. Two other occurences remain. Signed-off-by: tcharding --- Changing the remaining two BUG_ON's causes changes to the programm logic. This is my first patch set t

[PATCH 1/4] drivers: staging: fix parameter alignment

2016-04-17 Thread tcharding
drivers/staging/android/ion/ion.c checkpatch produces alignment checks. This patch is whitespace only and fixes these checks. Signed-off-by: tcharding --- drivers/staging/android/ion/ion.c | 64 +++ 1 file changed, 32 insertions(+), 32 deletions(-) diff

[PATCH 0/4] drivers: staging: checkpatch fixes

2016-04-17 Thread tcharding
drivers/staging/android/ion/ion.c produces 2 warnings and 29 checks. This patch set fixes both warnings and all but 4 of the checks. All except the last patch are trivial. Signed-off-by: tcharding --- Patch 1/4 is whitespace only. This is my first Linux kernel patch. thanks tcharding (4

[PATCH 3/4] drivers: staging: remove comparison to NULL

2016-04-17 Thread tcharding
drivers/staging/android/ion/ion.c checkpatch produces 'Comparison to NULL could be written ...' checks. This patch replaces comparison to NULL with !var as suggested by checkpatch. Signed-off-by: tcharding --- drivers/staging/android/ion/ion.c | 6 +++--- 1 file changed, 3 insert

[PATCH 2/4] drivers: staging: fix line length

2016-04-17 Thread tcharding
drivers/staging/android/ion/ion.c checkpatch produces line over 80 character warnings. This patch is whitespace only and fixes these warnings. Signed-off-by: tcharding --- drivers/staging/android/ion/ion.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers