[PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl

Fixed several errors and warnings. Lines 29, 103, 1319, 1906 and 2408: removed unnecessary spaces or added appropriate spaces. Lines 1040 and 1591: changed dev_printk(KERN_DEBUG, ...) to dev_dbg(...). Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/devio.c | 15

[PATCH] drivers/usb/core: sysfs.c: Removed a couple warnings

Removed a warnings(from checkpatch.pl) about sizeof not having parenthesis'. added parenthesis' around the strings being used. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/sysfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb

[PATCH] drivers/usb/core: hub.c: Removed all warnings about a space between a function name and parenthesis'

Removed 41 warnings all about a space between a function and the parenthesis' during a function call. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hub.c | 82 +- 1 file changed, 41 insertions(+), 41 deletions(-) diff

[PATCH] drivers/usb/core: hcd.c: Removed all space warnings from checkpatch.pl

Removed all(except false positives, from calculations) space prohibited warnings generated by checkpatch.pl. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hcd.c | 168 - 1 file changed, 84 insertions(+), 84 deletions

[PATCH] drivers/usb/core: hcd.c: Added parenthesis' to sizeof

Removed all warnings generated by checkpatch.pl about sizeof not having parenthesis'. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hcd.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c

[PATCH] drivers/usb/core: devio.c: Reverted previous debug print changes

Changed dev_dbg(...) back to dev_printk(KERN_DEBUG, ...) based on previous patches feedback. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/devio.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core

[PATCH] drivers/usb/core: hcd.c: Removed BitTime space warnings

Removed BitTime macro space warnings generated by checkpatch.pl. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hcd.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index 601476c7..299ff0d

[PATCH 2/2] drivers/usb/core: devio.c: Removed various warnings and errors generated by checkpatch.pl

)). Lines 1319 and 1906: added spaces to fit coding style consistently. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/devio.c | 69 1 file changed, 35 insertions(+), 34 deletions(-) diff --git a/drivers/usb/core/devio.c b

[PATCH] usb: core: hub.c: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hub.c | 40 1 file changed, 20 insertions(+), 20 deletions

[PATCH] usb: core: hub: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hub.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/usb

[PATCH] usb: core: hub: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hub.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/usb

[PATCH] usb: core: hub: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger chasemetzge...@gmail.com --- drivers/usb/core/hub.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git

[PATCH] staging: android: sync.c: Changed the ways nullptrs were being checked

Removed all checkpatch.pl CHECKs that suggested to check NULL by !obj instead of obj == NULL. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/android/sync.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/a

[PATCH] usb: core: devio.c: Removed unnecessary space

Removed an unnecessary space between a function name and arguments. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/usb/core/devio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 38ae877c..0

[PATCH] staging: android: sync.c: Changed the ways nullptrs were being checked

Removed all checkpatch.pl CHECKs that suggested to check NULL by !obj instead of obj == NULL. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/android/sync.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/a

[PATCH] usb: core: devio.c: Removed unnecessary space

Removed an unnecessary space between a function name and arguments. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/usb/core/devio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 38ae877c..0

[PATCH] drivers/staging/greybus: gpio.c - Fixed a checkpatch generated warning

Removed braces for single line if statement. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/gpio.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/greybus/gpio.c b/drivers/staging/greybus/gpio.c index ea8234a..5

[PATCH] drivers: staging: greybus: Fixed CHECKS for brace issues

Added braces to else statements where checkpatch complained. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/audio_codec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/g

[PATCH] drivers: staging: greybus: Fixed CHECKS for brace issues

Added braces to else statements where checkpatch complained. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/audio_topology.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/s

[PATCH] drivers: staging: greybus: audio_codec.c: Fixed CHECKS for brace issues

Added braces to else statement where checkpatch complained. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/audio_codec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/g

[PATCH] staging: greybus: timesync.c: Fixed WARNING for brace issue

Removed a checkpatch warning for braces on single argument if and else statement. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/timesync.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/greybus/timesync.c b/d

[PATCH] drivers: staging: greybus: audio_topology.c: Fixed CHECKS for brace issues

Added braces for else and else if statements where checkpatch complained. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/audio_topology.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_topolog

[PATCH] drivers: staging: greybus: audio_manager_sysfs.c: Fixed CHECK for brace issue

Added braces to else and else if statements where checkpatch complained. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/greybus/audio_manager_sysfs.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/g

[PATCH] staging: pi433: pi433_if.c: fix if/else if and brace coding style issues

Fix if/else if and braces in function DIO0_irq_handler to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/staging

[PATCH 6/6] staging: pi433: pi433_if.c: use tabs instead of spaces

Replace spaces with tabs where possible to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/d

[PATCH 3/6] staging: pi433: pi433_if.c: fix trailing statement coding style issues

Put trailing statement after if/else statements on new line to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c | 44 +++- 1 file changed, 30 insertions(+), 14 deletions(-)

[PATCH 1/6] staging: pi433: pi433_if.c: fix opening curly brace coding style issues

Put opening curly braces for if/else/else if statements and for/while loops on the same line as the statements or loops to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c

[PATCH 4/6] staging: pi433: pi433_if.c: fix space prohibited coding style issues

Remove unnecessary spaces before and after parenthesis' to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff

[PATCH 0/6] staging: pi433: pi433_if.c: coding style fixes

Fixes some of the coding style violations to comply with the kernel coding style. Chase Metzger (6): staging: pi433: pi433_if.c: fix opening curly brace coding style issues staging: pi433: pi433_if.c: fix else if/else statements coding style issues staging: pi433: pi433_if.c: fix

[PATCH 5/6] staging: pi433: pi433_if.c: fix space(s) required coding style issues

Put space(s) in required spots to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/d

[PATCH 2/6] staging: pi433: pi433_if.c: fix else if/else statements coding style issues

Put else if/else statements on same line as if/else if statements closing curly brace to comply with the kernel coding style. Signed-off-by: Chase Metzger <chasemetzge...@gmail.com> --- drivers/staging/pi433/pi433_if.c | 33 +++-- 1 file changed, 11 insertions(

[PATCH] staging: pi433: pi433_if.c: fix if/else if and brace coding style issues

Fix if/else if and braces in function DIO0_irq_handler to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging

[PATCH 0/6] staging: pi433: pi433_if.c: coding style fixes

Fixes some of the coding style violations to comply with the kernel coding style. Chase Metzger (6): staging: pi433: pi433_if.c: fix opening curly brace coding style issues staging: pi433: pi433_if.c: fix else if/else statements coding style issues staging: pi433: pi433_if.c: fix

[PATCH 4/6] staging: pi433: pi433_if.c: fix space prohibited coding style issues

Remove unnecessary spaces before and after parenthesis' to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/staging/pi433

[PATCH 3/6] staging: pi433: pi433_if.c: fix trailing statement coding style issues

Put trailing statement after if/else statements on new line to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 44 +++- 1 file changed, 30 insertions(+), 14 deletions(-) diff --git a/drivers/staging

[PATCH 1/6] staging: pi433: pi433_if.c: fix opening curly brace coding style issues

Put opening curly braces for if/else/else if statements and for/while loops on the same line as the statements or loops to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 110 ++- 1 file changed, 38

[PATCH 5/6] staging: pi433: pi433_if.c: fix space(s) required coding style issues

Put space(s) in required spots to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index

[PATCH 2/6] staging: pi433: pi433_if.c: fix else if/else statements coding style issues

Put else if/else statements on same line as if/else if statements closing curly brace to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 33 +++-- 1 file changed, 11 insertions(+), 22 deletions(-) diff --git

[PATCH 6/6] staging: pi433: pi433_if.c: use tabs instead of spaces

Replace spaces with tabs where possible to comply with the kernel coding style. Signed-off-by: Chase Metzger --- drivers/staging/pi433/pi433_if.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index

[PATCH 2/2] drivers/usb/core: devio.c: Removed various warnings and errors generated by checkpatch.pl

me)). Lines 1319 and 1906: added spaces to fit coding style consistently. Signed-off-by: Chase Metzger --- drivers/usb/core/devio.c | 69 1 file changed, 35 insertions(+), 34 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devi

[PATCH] drivers/usb/core: hub.c: Removed all warnings about a space between a function name and parenthesis'

Removed 41 warnings all about a space between a function and the parenthesis' during a function call. Signed-off-by: Chase Metzger --- drivers/usb/core/hub.c | 82 +- 1 file changed, 41 insertions(+), 41 deletions(-) diff --git a/drivers/usb

[PATCH] drivers/usb/core: devio.c: Removed various errors and warnings generated by checkpatch.pl

Fixed several errors and warnings. Lines 29, 103, 1319, 1906 and 2408: removed unnecessary spaces or added appropriate spaces. Lines 1040 and 1591: changed dev_printk(KERN_DEBUG, ...) to dev_dbg(...). Signed-off-by: Chase Metzger --- drivers/usb/core/devio.c | 15 +++ 1 file

[PATCH] drivers/usb/core: sysfs.c: Removed a couple warnings

Removed a warnings(from checkpatch.pl) about sizeof not having parenthesis'. added parenthesis' around the strings being used. Signed-off-by: Chase Metzger --- drivers/usb/core/sysfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/core/sysfs.c b/drivers

[PATCH] drivers/usb/core: hcd.c: Removed all space warnings from checkpatch.pl

Removed all(except false positives, from calculations) space prohibited warnings generated by checkpatch.pl. Signed-off-by: Chase Metzger --- drivers/usb/core/hcd.c | 168 - 1 file changed, 84 insertions(+), 84 deletions(-) diff --git a/drivers

[PATCH] drivers/usb/core: hcd.c: Added parenthesis' to sizeof

Removed all warnings generated by checkpatch.pl about sizeof not having parenthesis'. Signed-off-by: Chase Metzger --- drivers/usb/core/hcd.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index 557f2b8..601476c7

[PATCH] drivers/usb/core: devio.c: Reverted previous debug print changes

Changed dev_dbg(...) back to dev_printk(KERN_DEBUG, ...) based on previous patches feedback. Signed-off-by: Chase Metzger --- drivers/usb/core/devio.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 62c0be6

[PATCH] drivers/usb/core: hcd.c: Removed BitTime space warnings

Removed BitTime macro space warnings generated by checkpatch.pl. Signed-off-by: Chase Metzger --- drivers/usb/core/hcd.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index 601476c7..299ff0d 100644 --- a/drivers/usb

[PATCH] usb: core: hub: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger --- drivers/usb/core/hub.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb

[PATCH] usb: core: hub.c: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger --- drivers/usb/core/hub.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/usb

[PATCH] usb: core: hub: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger --- drivers/usb/core/hub.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/usb/core/hub.c b

[PATCH] usb: core: hub: Removed some warnings generated by checkpatch.pl

Removed some checkpatch.pl warnings saying there was an unwanted space between function names and their arguments. Signed-off-by: Chase Metzger --- drivers/usb/core/hub.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb

[PATCH] drivers: staging: greybus: Fixed CHECKS for brace issues

Added braces to else statements where checkpatch complained. Signed-off-by: Chase Metzger --- drivers/staging/greybus/audio_topology.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c

[PATCH] drivers: staging: greybus: Fixed CHECKS for brace issues

Added braces to else statements where checkpatch complained. Signed-off-by: Chase Metzger --- drivers/staging/greybus/audio_codec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index 8a0744b

[PATCH] staging: greybus: timesync.c: Fixed WARNING for brace issue

Removed a checkpatch warning for braces on single argument if and else statement. Signed-off-by: Chase Metzger --- drivers/staging/greybus/timesync.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/greybus/timesync.c b/drivers/staging/greybus

[PATCH] drivers/staging/greybus: gpio.c - Fixed a checkpatch generated warning

Removed braces for single line if statement. Signed-off-by: Chase Metzger --- drivers/staging/greybus/gpio.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/greybus/gpio.c b/drivers/staging/greybus/gpio.c index ea8234a..5e06e42 100644 --- a/drivers/staging

[PATCH] drivers: staging: greybus: audio_codec.c: Fixed CHECKS for brace issues

Added braces to else statement where checkpatch complained. Signed-off-by: Chase Metzger --- drivers/staging/greybus/audio_codec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index 8a0744b

[PATCH] drivers: staging: greybus: audio_manager_sysfs.c: Fixed CHECK for brace issue

Added braces to else and else if statements where checkpatch complained. Signed-off-by: Chase Metzger --- drivers/staging/greybus/audio_manager_sysfs.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_manager_sysfs.c b/drivers/staging

[PATCH] drivers: staging: greybus: audio_topology.c: Fixed CHECKS for brace issues

Added braces for else and else if statements where checkpatch complained. Signed-off-by: Chase Metzger --- drivers/staging/greybus/audio_topology.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus

[PATCH] usb: core: devio.c: Removed unnecessary space

Removed an unnecessary space between a function name and arguments. Signed-off-by: Chase Metzger --- drivers/usb/core/devio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 38ae877c..0bcd45e 100644 --- a/drivers/usb

[PATCH] staging: android: sync.c: Changed the ways nullptrs were being checked

Removed all checkpatch.pl CHECKs that suggested to check NULL by !obj instead of obj == NULL. Signed-off-by: Chase Metzger --- drivers/staging/android/sync.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/android/sync.c b/drivers/staging

[PATCH] usb: core: devio.c: Removed unnecessary space

Removed an unnecessary space between a function name and arguments. Signed-off-by: Chase Metzger --- drivers/usb/core/devio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c index 38ae877c..0bcd45e 100644 --- a/drivers/usb

[PATCH] staging: android: sync.c: Changed the ways nullptrs were being checked

Removed all checkpatch.pl CHECKs that suggested to check NULL by !obj instead of obj == NULL. Signed-off-by: Chase Metzger --- drivers/staging/android/sync.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/android/sync.c b/drivers/staging