From: Ken Depro
This patch removes extra blank lines from the virthba header and source
files. This is the first of a set of patches that will clean up the
virthba source file. The only checkpatch issue in the header file was
the blank line warnings.
Signed-off-by: Ken Depro
Signed-off-by
From: Ken Depro
This patch removes unneeded spaces after casts within the virthba.c
file. The checkpatch script was run after these changes, and no
"remove spaces after casts" checks were generated.
Later patches will fix the other checkpatch warnings/checks.
Signed-off-by:
From: Ken Depro
This patch removes unneeded spaces after casts within the virthba.c
file. The checkpatch script was run after these changes, and no
"remove spaces after casts" checks were generated.
Later patches will fix the other checkpatch warnings/checks.
Conflicts:
From: Ken Depro
This patch renames the following SignalInsert_withLock parameters
to fix the CamelCase checks:
pChannel to pchannel
Queue to queue
pSignal to psignal
Signed-off-by: Ken Depro
---
drivers/staging/unisys/channels/chanstub.c |6 +++---
drivers/staging/unisys/channels
Signed-off-by: Ken Depro
---
drivers/staging/unisys/channels/Makefile |2 +-
drivers/staging/unisys/channels/chanstub.c | 75
drivers/staging/unisys/channels/chanstub.h | 23 -
drivers/staging/unisys/uislib/uislib.c |1 -
drivers/staging
channels Makefile.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/channels/Makefile |2 +-
drivers/staging/unisys/channels/chanstub.c | 75
drivers/staging/unisys/channels/chanstub.h | 23 -
drivers/staging/unisys/uislib/uislib.c |1
This patch removes all of the unnecessary spaces after casts within
the virthba.c file. The checkpatch script was run again after these
changes, and no "remove spaces after casts" checks were generated.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthb
This patch fixes the CamelCase checks for methods/variables in
virthba.c found with the checkpatch script.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c | 122 +++---
1 file changed, 61 insertions(+), 61 deletions(-)
diff --git a/drivers/staging
This patch removes unnecessary blank lines either before opening braces or
after closing braces, as reported by the checkpatch script.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c |4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/unisys/virthba
This patch fixes the improper CamelCase usage for variables, reported by the
checkpatch script.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c | 122 +++---
1 file changed, 61 insertions(+), 61 deletions(-)
diff --git a/drivers/staging/unisys
This patch fixes checkpatch checks where the logical operator should be at the
end of the line above, not beginning the next line.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c | 38 +++---
1 file changed, 19 insertions(+), 19 deletions(-)
diff
This patch changes a couple of kzalloc calls to pass the variable name to the
call, rather than the variable struct type. This is a result of checks
generated during the checkpatch script.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c |4 ++--
1 file changed, 2
This series fixes various issues in the virthba.c file that were reported by
the checkpatch script.
Ken Depro (11):
staging: unisys: virthba: Remove unneeded spaces after casts
staging: unisys: virthba: Fix CamelCase variables
staging: unisys: virthba: Fix open parenthesis alignment checks
This patch fixes the "alignment should match open parenthesis" checks from the
checkpatch script.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c | 74 +++---
1 file changed, 37 insertions(+), 37 deletions(-)
diff --git a/drivers/stag
This patch removes all unnecessary spaces after casts, as reported by the
checkpatch script.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c | 130 +++---
1 file changed, 65 insertions(+), 65 deletions(-)
diff --git a/drivers/staging/unisys
This patch fixes a couple checkpatch checks where alignment of the parameters
did not match the open parenthesis of the function.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging
This patch fixes warnings generated by checkpatch script regarding lines over
80 characters long.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c | 40 --
1 file changed, 22 insertions(+), 18 deletions(-)
diff --git a/drivers/staging/unisys
This patch fixes checkpatch warnings that resulted from the else portion of
if-else statements missing braces, so that it conforms with the rest of the
statement.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c |6 --
1 file changed, 4 insertions(+), 2 deletions
This patch fixes a warning generated during the checkpatch script that stated
"else not useful after return". I modified the code to return a designated
status at the end of the function, and replaced the return statement in the
"else if" to set the status accordingly.
Sign
This patch fixes a couple small issues reported by the checkpatch script:
Adds a blank line after a struct definition.
Removes unnecessary parentheses surrounding a struct member.
Signed-off-by: Ken Depro
---
drivers/staging/unisys/virthba/virthba.c |3 ++-
1 file changed, 2 insertions
From: Ken Depro
This patch removes some unneeded white space after casts within the
channel.c file. The checkpatch script was run after these changes,
and no further checks or warnings were seen.
Signed-off-by: Ken Depro
Signed-off-by: Ben Romer
---
drivers/staging/unisys/channels/channel.c
21 matches
Mail list logo