Chase Metzger
cb2ec2eff2
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 <chasemetzger15@gmail.com >
Acked-by: Viresh Kumar <viresh.kumar@linaro.org >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:25 +02:00
Elise Lennion
cf3ba55dca
staging: greybus: arche-platform: Format block comments.
...
Fix checkpatch warning:
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Elise Lennion <elise.lennion@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:24 +02:00
Elise Lennion
c4fc2ebcbf
staging: greybus: camera: Use kcalloc for array's memory allocation.
...
Fix checkpatch warning:
WARNING: Prefer kcalloc over kzalloc with multiply
kcalloc is designed to allocate memory for arrays, its use is
preferable than kzalloc in these cases.
Signed-off-by: Elise Lennion <elise.lennion@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:23 +02:00
Elise Lennion
9d3318f48c
staging: greybus: camera: Replace blank spaces with tabstops.
...
Fix checkpatch warning:
WARNING: Statements should start on a tabstop
Signed-off-by: Elise Lennion <elise.lennion@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:22 +02:00
Elise Lennion
cd7b701f52
staging: greybus: camera: Add blank lines after variable declarations.
...
Fix checkpatch warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Elise Lennion <elise.lennion@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:21 +02:00
Wayne Porter
b41514b1de
staging: rts5208: rtsx.c: Alignment fix
...
Line goes too long when aligned with parenthesis, so moved to a new line
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:20 +02:00
Wayne Porter
88936fe853
staging: rts5208: rtsx_chip.c: Long lines
...
Fix lines that go over 80 characters to stop warnings from checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:19 +02:00
Wayne Porter
7b468a297d
staging: rts5208: rtxs_chip.h: Long lines
...
Move comments above long definitions so they don't go over 80 characters
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:18 +02:00
Wayne Porter
7314203b38
staging: rts5208: rtsx_scsi.h: Long lines
...
Aligning with parenthesis causes lines to go too long, so the
parenthesis are on a new line
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:17 +02:00
Wayne Porter
d264ceb779
staging: rts5208: CamelCase fix
...
Change label to not use camel case per checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:16 +02:00
Wayne Porter
a3e7c0928b
staging: rts5208: ms.c: Braces fix
...
Add braces to all arms of if statement
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:14 +02:00
Wayne Porter
36e32ee542
staging: rts5208: ms.c: Multiple assignments
...
Fix check found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:13 +02:00
Wayne Porter
25ccf0b0a7
staging: rts5208: ms.c: Long Lines
...
Fix long lines detected by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:12 +02:00
Wayne Porter
9c378f1488
staging: rts5208: ms.c: Parenthesis alignment
...
Fix checkpatch messages: Alignment should match open parenthesis
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:11 +02:00
Wayne Porter
8473e5bb5c
staging: rts5208: ms.h: Parenthesis alignment
...
Fix alignment to match open parenthesis
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:10 +02:00
Wayne Porter
2eb9d8cbb3
staging: rts5208: rtsx.c: Alloc sizeof struct
...
Satisfy checkpatch message: Prefer kzalloc(sizeof(*dev->chip)...) over
kzalloc(sizeof(struct rtsx_chip)...)
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:09 +02:00
Wayne Porter
8149b9ab57
staging: rts5208: rtsx.c: Spacing
...
Remove blank line after opening {
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:08 +02:00
Wayne Porter
cd4b77586b
staging: rts5208: rtsx.c: CamelCase
...
Make camel case labels all lowercase
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:07 +02:00
Wayne Porter
7e0ea476ed
staging: rts5208: rtsx.c: Spacing
...
Add spaces around operator for readability
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:06 +02:00
Wayne Porter
cd5491af0d
staging: rts5208: rtsx.c: Multiple assignments
...
Fix multiple assignments found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:05 +02:00
Wayne Porter
c8b07957c4
staging: rts5208: rtsx.c: Alignment
...
Fix to match open parenthesis in order to satisfy checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:04 +02:00
Wayne Porter
379dfd3269
staging: rts5208: rtsx.c: Unnecessary parentheses
...
Remove parentheses found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:03 +02:00
Wayne Porter
a38e369b31
staging: rts5208: Comparison to NULL
...
Fix to resolve checkpatch message
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:02 +02:00
Wayne Porter
911e84a09a
staging: rts5208: Spacing
...
Adding space around operator for better readability and to stop
checkpatch check messages
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:01 +02:00
Wayne Porter
117b6038b2
staging: rts5208: Unnecessary parentheses
...
Remove parentheses found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:27:00 +02:00
Wayne Porter
e379c79bae
staging: rts5208: Parenthesis alignment
...
Fix alignment to stop checkpath check messages
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:59 +02:00
Wayne Porter
f5becff728
staging: rts5208: Unecessary parantheses
...
Remove parentheses to stop checkpatch check message
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:58 +02:00
Wayne Porter
5d2de0d9b7
staging: rts5208: Parenthesis alignment
...
Matching open parenthesis
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:57 +02:00
Wayne Porter
2c6b3b9347
staging: rts5208: Prefer using BIT macro
...
Replace all instances of (1 << x) with BIT(x) to satisfy checkpatch
check messages
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:56 +02:00
Wayne Porter
15794518d7
staging: rts5208: Long lines fixes
...
Break up long lines in rtsx_chip.h
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:55 +02:00
Wayne Porter
33f28123af
staging: rts5208: Alignment fixes
...
Fix alignment issues that checkpatch found
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:54 +02:00
Wayne Porter
3f19de3681
staging: rts5208: Add space around operator
...
Spaces are preferred around operators for readability
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:53 +02:00
Wayne Porter
44000fe6f1
staging: rts5208: Add braces to if()
...
Braces should be around every part of the if block and not just the else
if
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:52 +02:00
Wayne Porter
f4bfffb959
staging: rts5208: Remove multiple assignment
...
Assign values to variables on their own lines instead of using multiple
assignment
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:51 +02:00
Wayne Porter
904af7d5b5
staging: rts5208: CamelCase fixes
...
Change camel case variables found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:50 +02:00
Wayne Porter
33a1b1cd2e
staging: rts5208: Remove parentheses
...
Unnecessary parentheses found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:49 +02:00
Wayne Porter
3b8cd5aa17
staging: rts5208: Alignment fixes
...
Align to match open parenthesis, found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:48 +02:00
Wayne Porter
7fb7a2fc40
staging: rts5208: Fix typo in function name
...
Searched the entire kernel for any references to this function and it
appears safe to fix the typo
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:47 +02:00
Wayne Porter
ea2030d5ae
staging: rts5208: Alignment fix
...
Fix issue found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:46 +02:00
Wayne Porter
c3cca2d1b4
staging: rts5208: Unnecessary parentheses cleanup
...
Remove parentheses found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:45 +02:00
Wayne Porter
5426c164ff
staging: rts5208: Alignment fixes
...
Cleaning up checkpatch issues
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:44 +02:00
Wayne Porter
fe421a75ec
staging: rts5208: sd.c: Spacing cleanup
...
Add/remove spaces to make things more readable
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:43 +02:00
Wayne Porter
3c3776441f
staging: rts5208: sd.c: Fix logical continuations
...
Checkpatch detected && at the beginning of new lines
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:42 +02:00
Wayne Porter
57a3fcfa6b
staging: rts5208: sd.c: CamelCase fixes
...
Convert camel case lables to all lowercase
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:41 +02:00
Wayne Porter
7fbe04f1ce
staging: rts5208: sd.c: Long line fixes
...
Break up lines over 80 characters
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:40 +02:00
Wayne Porter
20390dc72e
staging: rts5208: sd.c: Alignment fixes
...
Lining up with open parenthesis found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:39 +02:00
Wayne Porter
33f2b12165
staging: rts5208: sd.c: Remove unnecessary parentheses
...
Cleanup of &(chip->sd_card) to remove parentheses where they are not
needed
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:37 +02:00
Wayne Porter
e78bff3266
staging: rts5208: sd.h: Alignment fixes
...
Lining up code with open parenthses found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:36 +02:00
Wayne Porter
8c3c144bff
staging: rts5208: spi.c: Alignment fixes
...
Lining up code with open parantheses found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:35 +02:00
Wayne Porter
4669f6e275
staging: rts5208: spi.c: Remove unnecessary parenthesis
...
Style warnings found by checkpatch
Signed-off-by: Wayne Porter <wporter82@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2016-10-16 10:26:34 +02:00