]> git.karo-electronics.de Git - linux-beck.git/log
linux-beck.git
7 years agostaging: speakup: speakup_soft.c Align parenthesis
Walt Feasel [Mon, 21 Nov 2016 15:19:25 +0000 (10:19 -0500)]
staging: speakup: speakup_soft.c Align parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_soft.c comment modifications
Walt Feasel [Mon, 21 Nov 2016 15:19:24 +0000 (10:19 -0500)]
staging: speakup: speakup_soft.c comment modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_soft.c Blank line after {
Walt Feasel [Mon, 21 Nov 2016 15:19:23 +0000 (10:19 -0500)]
staging: speakup: speakup_soft.c Blank line after {

Make suggested checkpatch modification for
CHECK: Blank lines aren't necessary after an
open brace '{'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_soft.c Multiple blank lines
Walt Feasel [Mon, 21 Nov 2016 15:19:22 +0000 (10:19 -0500)]
staging: speakup: speakup_soft.c Multiple blank lines

Make suggested checkpatch modification for
CHECK: Please don't use multiple blank lines

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_soft.c Logical continuation
Walt Feasel [Mon, 21 Nov 2016 15:19:21 +0000 (10:19 -0500)]
staging: speakup: speakup_soft.c Logical continuation

Make suggested checkpatch modification for
CHECK: Logical continuations should be on the
previous line

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_soft.c Space around operator
Walt Feasel [Mon, 21 Nov 2016 15:19:20 +0000 (10:19 -0500)]
staging: speakup: speakup_soft.c Space around operator

Make suggested checkpatch modification for
CHECK: spaces preferred around that '|'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_spkout.c Space around operator
Walt Feasel [Mon, 21 Nov 2016 15:17:53 +0000 (10:17 -0500)]
staging: speakup: speakup_spkout.c Space around operator

Make suggested checkpatch modification for
CHECK: spaces preferred around that '|'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_spkout.c comment modifications
Walt Feasel [Mon, 21 Nov 2016 15:17:52 +0000 (10:17 -0500)]
staging: speakup: speakup_spkout.c comment modifications

Make modifications to comments

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_txprt.c Spaces preferred around operator
Walt Feasel [Mon, 21 Nov 2016 15:14:47 +0000 (10:14 -0500)]
staging: speakup: speakup_txprt.c Spaces preferred around operator

Make suggested checkpatch modification for
CHECK: spaces preferred around that '|'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: speakup_txprt.c Comment modifications
Walt Feasel [Mon, 21 Nov 2016 15:14:46 +0000 (10:14 -0500)]
staging: speakup: speakup_txprt.c Comment modifications

Make modifications to comments

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: spk_priv_keyinfo.h Space around operator
Walt Feasel [Mon, 21 Nov 2016 15:13:16 +0000 (10:13 -0500)]
staging: speakup: spk_priv_keyinfo.h Space around operator

Make suggested checkpatch modification for
CHECK: spaces preferred around that '+'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: spk_priv_keyinfo.h align columns
Walt Feasel [Mon, 21 Nov 2016 15:13:15 +0000 (10:13 -0500)]
staging: speakup: spk_priv_keyinfo.h align columns

Make modifications to align columns

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: spk_types.h Align parenthesis
Walt Feasel [Mon, 21 Nov 2016 15:11:25 +0000 (10:11 -0500)]
staging: speakup: spk_types.h Align parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: spk_types.h Comment modifications
Walt Feasel [Mon, 21 Nov 2016 15:11:24 +0000 (10:11 -0500)]
staging: speakup: spk_types.h Comment modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: emxx_udc: emxx_udc.c Align parenthesis
Walt Feasel [Mon, 21 Nov 2016 15:05:51 +0000 (10:05 -0500)]
staging: emxx_udc: emxx_udc.c Align parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: emxx_udc: emxx_udc.c Spaces preferred around operators
Walt Feasel [Mon, 21 Nov 2016 15:05:50 +0000 (10:05 -0500)]
staging: emxx_udc: emxx_udc.c Spaces preferred around operators

Make suggested checkpatch modification for
CHECK: spaces preferred around that '|'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: emxx_udc: emxx_udc.c {} Single statement blocks
Walt Feasel [Mon, 21 Nov 2016 15:05:49 +0000 (10:05 -0500)]
staging: emxx_udc: emxx_udc.c {} Single statement blocks

Make suggested checkpatch modification for
WARNING: braces {} are not necessary for single
statement blocks

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: emxx_udc: emxx_udc.c Logical continuation
Walt Feasel [Mon, 21 Nov 2016 15:05:48 +0000 (10:05 -0500)]
staging: emxx_udc: emxx_udc.c Logical continuation

Make suggested checkpatch modification for
CHECK: Logical continuations should be on the
previous line

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: emxx_udc: emxx_udc.c Blank lines after {
Walt Feasel [Mon, 21 Nov 2016 15:05:47 +0000 (10:05 -0500)]
staging: emxx_udc: emxx_udc.c Blank lines after {

Make suggested checkpatch modification for
CHECK: Blank lines aren't necessary after an
open brace '{'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: bus_configure add error handling
David Kershner [Mon, 21 Nov 2016 17:15:53 +0000 (12:15 -0500)]
staging: unisys: visorbus: bus_configure add error handling

Add proper error handling to bus_configure and have it propagate errors
back up the stack. This helped to streamline the function as well.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: return EAGAIN when empty
David Kershner [Mon, 21 Nov 2016 17:15:52 +0000 (12:15 -0500)]
staging: unisys: visorbus: return EAGAIN when empty

It really isn't an error when you call visorchannel_signalremove and
there isn't anything in the queue. Just means that the IOSP didn't
process anything since the last time you checked. Just inform the
caller that it is empty by returning -EAGAIN.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: device_responder add error handling
David Kershner [Mon, 21 Nov 2016 17:15:51 +0000 (12:15 -0500)]
staging: unisys: visorbus: device_responder add error handling

The function controlvm_respond returns proper error handling, so now we
can propagate the error up and handle it appropriately.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: device_changestate_responder add error handling
David Kershner [Mon, 21 Nov 2016 17:15:50 +0000 (12:15 -0500)]
staging: unisys: visorbus: device_changestate_responder add error handling

Pass the error returned from visorchannel_signalinsert up the stack
instead of just ignoring it.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: bus_responder add error handling
David Kershner [Mon, 21 Nov 2016 17:15:49 +0000 (12:15 -0500)]
staging: unisys: visorbus: bus_responder add error handling

Controlvm_respond now returns an error. Add error handling to the
bus_responder function.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: save_crash_message add error handling
David Kershner [Mon, 21 Nov 2016 17:15:48 +0000 (12:15 -0500)]
staging: unisys: visorbus: save_crash_message add error handling

Add proper error handling to the function save_crash_message.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: controlvm_respond_physdev_changestate add error handling
David Kershner [Mon, 21 Nov 2016 17:15:47 +0000 (12:15 -0500)]
staging: unisys: visorbus: controlvm_respond_physdev_changestate add error handling

Propagate the error up the stack instead of ignoring it.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: controlvm_respond add error handling
David Kershner [Mon, 21 Nov 2016 17:15:46 +0000 (12:15 -0500)]
staging: unisys: visorbus: controlvm_respond add error handling

The function visorchanel_signalinsert now returns an error, propagate
the error up the stack to be handled.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: chipset_init add error handling
David Kershner [Mon, 21 Nov 2016 17:15:45 +0000 (12:15 -0500)]
staging: unisys: visorbus: chipset_init add error handling

Controlvm_respond_chipset_init now errors out correctly. Pass the errors
back up to the stack to be processed.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: controlvm_respond_chipset_init add error handling
David Kershner [Mon, 21 Nov 2016 17:15:44 +0000 (12:15 -0500)]
staging: unisys: visorbus: controlvm_respond_chipset_init add error handling

Now that visorchannel_signalinsert returns a Linux error code, we need
to convert controlvm_respond_chipset_init to return an error instead of
a void.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: visorbus_private rename 'local' var to 'dest'
Erik Arfvidson [Mon, 21 Nov 2016 17:15:43 +0000 (12:15 -0500)]
staging: unisys: visorbus: visorbus_private rename 'local' var to 'dest'

This patch renames the poorly named 'local' var to 'dest' in
the visorbus_private.h file

Signed-off-by: Erik Arfvidson <erik.arfvidson@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: visorbus: visorchannel.c rename 'local' variable to 'dest'
Erik Arfvidson [Mon, 21 Nov 2016 17:15:42 +0000 (12:15 -0500)]
staging: unisys: visorbus: visorchannel.c rename 'local' variable to 'dest'

This patch renames poorly named 'local' variable to 'dest'.

Signed-off-by: Erik Arfvidson <erik.arfvidson@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: include: fix pound defines
Erik Arfvidson [Mon, 21 Nov 2016 17:15:41 +0000 (12:15 -0500)]
staging: unisys: include: fix pound defines

The purpose of this patch is to add style consistency and
beautify the entire file. This patch removes unused pound defines
and cleans up spacing and tabbing of pound defines.

Signed-off-by: Erik Arfvidson <erik.arfvidson@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: unisys: include: fixed iochannel.h comments
Erik Arfvidson [Mon, 21 Nov 2016 17:15:40 +0000 (12:15 -0500)]
staging: unisys: include: fixed iochannel.h comments

The purpose of this patch is to add style consistency and beautify the
entire file.

Grammar:
Uppercased beginning of new sentences/paragraphs
Missing ' and .
Fixed several comments using prefix phrase "as is in" to "as in"
Standard Grammar

Comments:
Fixed comments to follow the multiline styling
All comments are now either same line or above variable/definitions
Removed nested slash-star comments
Removed a random comment left by accident
Aligned star comments

Signed-off-by: Erik Arfvidson <erik.arfvidson@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: change some macros into inline functions
Sergio Paracuellos [Mon, 21 Nov 2016 16:48:24 +0000 (17:48 +0100)]
staging: slicoss: change some macros into inline functions

This patch changes macros into inline functions to avoid this checkpatch
check:
Macro argument 'x' may be better as '(x)' to avoid precedence issues

It also removes two macros because they aren't being used.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agoStaging: rtl8712: rtl871x_*: Removed unnecessary else statements after a break or...
Shiva Kerdel [Mon, 21 Nov 2016 09:12:41 +0000 (10:12 +0100)]
Staging: rtl8712: rtl871x_*: Removed unnecessary else statements after a break or return

The indent code blocks of the else statements were unnecessary
and are better written without them.

Signed-off-by: Shiva Kerdel <shiva@exdev.nl>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: rtl8712: Fix coding style warnings on Block comments
Vijai Kumar K [Mon, 21 Nov 2016 01:01:52 +0000 (17:01 -0800)]
staging: rtl8712: Fix coding style warnings on Block comments

Fixed checkpatch.pl warnings related to Block comments in
staging/rtl8712/*.c files.

Signed-off-by: Vijai Kumar K <vijaikumar.kanagarajan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging/lustre: Use proper number of bytes in copy_from_user
Jian Yu [Mon, 21 Nov 2016 05:46:48 +0000 (00:46 -0500)]
staging/lustre: Use proper number of bytes in copy_from_user

This patch removes the usage of MAX_STRING_SIZE from
copy_from_user() and just copies enough bytes to cover
count passed in.

Signed-off-by: Jian Yu <jian.yu@intel.com>
Reviewed-on: http://review.whamcloud.com/23462
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8774
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: vt6656: Fix coding style warnings on Block comments
Vijai Kumar K [Sun, 20 Nov 2016 20:51:40 +0000 (12:51 -0800)]
staging: vt6656: Fix coding style warnings on Block comments

Fix checkpatch.pl warnings related to Block comments in
staging/vt6656/rf.c file.

Signed-off-by: Vijai Kumar K <vijaikumar.kanagarajan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: vt6655: Fixed coding style warnings on Block comments
Vijai Kumar K [Sun, 20 Nov 2016 18:57:09 +0000 (10:57 -0800)]
staging: vt6655: Fixed coding style warnings on Block comments

Fixed checkpatch.pl warnings related to Block comments in
staging/vt6655/*.c files.

Signed-off-by: Vijai Kumar K <vijaikumar.kanagarajan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_tty: Remove blank lines
Vijai Kumar K [Sun, 20 Nov 2016 18:39:23 +0000 (10:39 -0800)]
staging: dgnc: dgnc_tty: Remove blank lines

Fixes checkpatch warning: waitqueue_active without comment

Signed-off-by: Vijai Kumar K <vijaikumar.kanagarajan@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: skein: threefish_block.c Remove blank lines
Walt Feasel [Sun, 20 Nov 2016 21:10:47 +0000 (16:10 -0500)]
staging: skein: threefish_block.c Remove blank lines

Make suggested checkpatch modification for

CHECK: Please don't use multiple blank lines

Acked-by: Jason Cooper <jason@lakedaemon.net
Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: digi.h Spelling correction
Walt Feasel [Sat, 19 Nov 2016 16:47:41 +0000 (11:47 -0500)]
staging: dgnc: digi.h Spelling correction

Make spelling correction for 'regular'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: digi.h Comment style modifications
Walt Feasel [Sat, 19 Nov 2016 16:47:40 +0000 (11:47 -0500)]
staging: dgnc: digi.h Comment style modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_tty.c Spelling corrections
Walt Feasel [Sat, 19 Nov 2016 16:45:42 +0000 (11:45 -0500)]
staging: dgnc: dgnc_tty.c Spelling corrections

Make spelling corrections for 'transitions' and 'satisfy'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_tty.c comment style modifications
Walt Feasel [Sat, 19 Nov 2016 16:45:41 +0000 (11:45 -0500)]
staging: dgnc: dgnc_tty.c comment style modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_tty.c Align on parenthesis
Walt Feasel [Sat, 19 Nov 2016 16:45:40 +0000 (11:45 -0500)]
staging: dgnc: dgnc_tty.c Align on parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_tty.c Space preferred around
Walt Feasel [Sat, 19 Nov 2016 16:45:39 +0000 (11:45 -0500)]
staging: dgnc: dgnc_tty.c Space preferred around

Make suggested checkpatch modification for
CHECK: spaces preferred around that '|'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: fix different address space warnings
Sergio Paracuellos [Fri, 18 Nov 2016 17:57:18 +0000 (18:57 +0100)]
staging: slicoss: fix different address space warnings

Remove incorrect __iomem annotation.

This patch fix the following sparse warnings in slicoss driver:
warning: incorrect type in assignment (different address spaces)

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: synth.c Align parenthesis
Walt Feasel [Sat, 19 Nov 2016 17:35:47 +0000 (12:35 -0500)]
staging: speakup: synth.c Align parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: synth.c Comment modifications
Walt Feasel [Sat, 19 Nov 2016 17:35:46 +0000 (12:35 -0500)]
staging: speakup: synth.c Comment modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: synth.c Logical continuation
Walt Feasel [Sat, 19 Nov 2016 17:35:45 +0000 (12:35 -0500)]
staging: speakup: synth.c Logical continuation

Make suggested checkpatch modification for
CHECK: Logical continuations should be on the
previous line

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: synth.c Blank line before }
Walt Feasel [Sat, 19 Nov 2016 17:35:44 +0000 (12:35 -0500)]
staging: speakup: synth.c Blank line before }

Make suggested checkpatch modification for
CHECK: Blank lines aren't necessary before a close brace '}'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: synth.c Spaces around operators
Walt Feasel [Sat, 19 Nov 2016 19:34:35 +0000 (14:34 -0500)]
staging: speakup: synth.c Spaces around operators

Make suggested checkpatch modification for
CHECK: spaces preferred around that '|,+,-,/'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: thread.c align on parenthesis
Walt Feasel [Sat, 19 Nov 2016 17:12:12 +0000 (12:12 -0500)]
staging: speakup: thread.c align on parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: thread.c Comment modifications
Walt Feasel [Sat, 19 Nov 2016 17:12:11 +0000 (12:12 -0500)]
staging: speakup: thread.c Comment modifications

Make comment style modifications

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: TODO Correct email
Walt Feasel [Sat, 19 Nov 2016 16:50:54 +0000 (11:50 -0500)]
staging: speakup: TODO Correct email

Make email correction for kirk@reisers.ca

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Acked-by: Chris Brannon <chris@the-brannons.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: varhandlers.c Align parenthesis
Walt Feasel [Sat, 19 Nov 2016 16:50:53 +0000 (11:50 -0500)]
staging: speakup: varhandlers.c Align parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: speakup: varhandlers.c Comment style modifications
Walt Feasel [Sat, 19 Nov 2016 16:50:52 +0000 (11:50 -0500)]
staging: speakup: varhandlers.c Comment style modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_sysfs.c Delete blank line
Walt Feasel [Fri, 18 Nov 2016 23:34:06 +0000 (18:34 -0500)]
staging: dgnc: dgnc_sysfs.c Delete blank line

Make modification to remove extra blank line

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_sysfs.c Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 23:34:05 +0000 (18:34 -0500)]
staging: dgnc: dgnc_sysfs.c Comment style modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_neo.h Spelling correction
Walt Feasel [Fri, 18 Nov 2016 23:21:09 +0000 (18:21 -0500)]
staging: dgnc: dgnc_neo.h Spelling correction

Make spelling correction for 'control'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_neo.h Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 23:21:08 +0000 (18:21 -0500)]
staging: dgnc: dgnc_neo.h Comment style modifications

Make modifications for comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_neo.c Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 23:08:03 +0000 (18:08 -0500)]
staging: dgnc: dgnc_neo.c Comment style modifications

Make modifications for comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_mgmt.c Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 22:49:24 +0000 (17:49 -0500)]
staging: dgnc: dgnc_mgmt.c Comment style modifications

Make modifications for comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: avoid CamelCases slicoss.c
Sergio Paracuellos [Fri, 18 Nov 2016 17:58:10 +0000 (18:58 +0100)]
staging: slicoss: avoid CamelCases slicoss.c

Replace CamelCases to comply with the standard kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: avoid CamelCases in slichw.h
Sergio Paracuellos [Fri, 18 Nov 2016 17:58:09 +0000 (18:58 +0100)]
staging: slicoss: avoid CamelCases in slichw.h

Replace CamelCases to comply with the standard kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: braces should be used on all arms of this statement
Sergio Paracuellos [Fri, 18 Nov 2016 17:58:08 +0000 (18:58 +0100)]
staging: slicoss: braces should be used on all arms of this statement

Add braces in if statement to comply with the standard
kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: logical continuations should be on the previous line
Sergio Paracuellos [Fri, 18 Nov 2016 17:58:07 +0000 (18:58 +0100)]
staging: slicoss: logical continuations should be on the previous line

Move logical or operator to previous line to comply with
the standard kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: fix parenthesis alignment in slicoss.c
Sergio Paracuellos [Fri, 18 Nov 2016 17:58:06 +0000 (18:58 +0100)]
staging: slicoss: fix parenthesis alignment in slicoss.c

This patch fix open parenthesis alignment matching in slicoss.c
file to comply with the standard kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: slicoss: avoid CamelCases in slic.h
Sergio Paracuellos [Fri, 18 Nov 2016 17:58:05 +0000 (18:58 +0100)]
staging: slicoss: avoid CamelCases in slic.h

Replace CamelCases to comply with the standard kernel coding style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: Greybus: Remove unnecessary braces for single statement block
Rahul Krishnan [Fri, 18 Nov 2016 15:15:25 +0000 (20:45 +0530)]
staging: Greybus: Remove unnecessary braces for single statement block

This patch fixes the following checkpath.pl warning
WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Rahul Krishnan <mrahul.krishnan@gmail.com>
Reviewed-by: Rui Miguel Silva <rmfrfs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_driver.h Spelling correction
Walt Feasel [Fri, 18 Nov 2016 22:39:13 +0000 (17:39 -0500)]
staging: dgnc: dgnc_driver.h Spelling correction

Make spelling correction for 'statements'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_driver.h Align columns
Walt Feasel [Fri, 18 Nov 2016 22:39:12 +0000 (17:39 -0500)]
staging: dgnc: dgnc_driver.h Align columns

Make modifications to align columns

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_driver.h Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 22:39:11 +0000 (17:39 -0500)]
staging: dgnc: dgnc_driver.h Comment style modifications

Make modifications to comment style

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_driver.c Blank line before }
Walt Feasel [Fri, 18 Nov 2016 20:18:10 +0000 (15:18 -0500)]
staging: dgnc: dgnc_driver.c Blank line before }

Make suggested checkpatch modification for
CHECK: Blank lines aren't necessary before
a close brace '}'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_driver.c Remove blank lines
Walt Feasel [Fri, 18 Nov 2016 20:18:09 +0000 (15:18 -0500)]
staging: dgnc: dgnc_driver.c Remove blank lines

Make suggested checkpatch modification for
CHECK: Please don't use multiple blank lines

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_driver.c Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 20:18:08 +0000 (15:18 -0500)]
staging: dgnc: dgnc_driver.c Comment style modifications

Make modifications to comment styles

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_cls.c Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 17:45:19 +0000 (12:45 -0500)]
staging: dgnc: dgnc_cls.c Comment style modifications

Make modifications to comment style format

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: dgnc: dgnc_cls.h Spelling correction
Walt Feasel [Fri, 18 Nov 2016 17:36:50 +0000 (12:36 -0500)]
staging: dgnc: dgnc_cls.h Spelling correction

Make spelling correction for 'control'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: vme: vme_user.c Spelling corrections
Walt Feasel [Fri, 18 Nov 2016 17:14:07 +0000 (12:14 -0500)]
staging: vme: vme_user.c Spelling corrections

Make spelling corrections for 'correctly' and
'unregister'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Acked-by: Martyn Welch <martyn at welchs.e.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agoStaging: wlan-ng: prism2mgmt: Fixed operators spacing style issues
Shiva Kerdel [Fri, 18 Nov 2016 13:12:42 +0000 (14:12 +0100)]
Staging: wlan-ng: prism2mgmt: Fixed operators spacing style issues

Fixed spaces around operators to fix their coding style issues.

Signed-off-by: Shiva Kerdel <shiva@exdev.nl>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: XGI_main.h Align data columns
Walt Feasel [Fri, 18 Nov 2016 01:56:01 +0000 (20:56 -0500)]
staging: xgifb: XGI_main.h Align data columns

Make modifications to data column alignment

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_util.h Space after cast
Walt Feasel [Fri, 18 Nov 2016 01:29:08 +0000 (20:29 -0500)]
staging: xgifb: vb_util.h Space after cast

Make suggested checkpatch modification for
CHECK: No space is necessary after a cast

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Align match parenthesis
Walt Feasel [Fri, 18 Nov 2016 00:12:54 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Align match parenthesis

Make suggested checkpatch modification for
CHECK: Alignment should match open parenthesis

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Logical continuation
Walt Feasel [Fri, 18 Nov 2016 00:12:53 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Logical continuation

Make suggested checkpatch modification for
CHECK: Logical continuations should be on the
previous line

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Space after cast
Walt Feasel [Fri, 18 Nov 2016 00:12:52 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Space after cast

Make suggested checkpatch modification for
CHECK: No space is necessary after a cast

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Braces single statement blocks
Walt Feasel [Fri, 18 Nov 2016 00:12:51 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Braces single statement blocks

Make suggested checkpatch modificationfor
WARNING: braces {} are not necessary for single
statement blocks

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Space around operator
Walt Feasel [Fri, 18 Nov 2016 00:12:50 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Space around operator

Make suggested checkpatch modificationfor
CHECK: spaces preferred around that '+'

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Comment style modifications
Walt Feasel [Fri, 18 Nov 2016 00:12:49 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Comment style modifications

Make comment style modifications.

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: xgifb: vb_setmode.c Comment Spelling correction
Walt Feasel [Fri, 18 Nov 2016 00:12:48 +0000 (19:12 -0500)]
staging: xgifb: vb_setmode.c Comment Spelling correction

Make spelling correction for 'vertical'.

Signed-off-by: Walt Feasel <waltfeasel@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: use uXX instead of __uXX types in headers
James Simmons [Fri, 18 Nov 2016 16:48:44 +0000 (11:48 -0500)]
staging: lustre: libcfs: use uXX instead of __uXX types in headers

The types __[u|s]XX is only used for UAPI headers and userspace.
Only keep these types for the libcfs headers that are UAPI
headers. The rest convert to the standard uXX types.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: remove zero comparisons in headers
James Simmons [Fri, 18 Nov 2016 16:48:43 +0000 (11:48 -0500)]
staging: lustre: libcfs: remove zero comparisons in headers

Remove the zero comparisions in the libcfs headers.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: remove NULL comparisons in headers
James Simmons [Fri, 18 Nov 2016 16:48:42 +0000 (11:48 -0500)]
staging: lustre: libcfs: remove NULL comparisons in headers

Remove the NULL comparisions in the libcfs headers.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: remove whitespace in libcfs_fail.h
James Simmons [Fri, 18 Nov 2016 16:48:41 +0000 (11:48 -0500)]
staging: lustre: libcfs: remove whitespace in libcfs_fail.h

One last white space is still left in libcfs_fail.h. Lets
remove it.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: use bit macro in libcfs headers
James Simmons [Fri, 18 Nov 2016 16:48:40 +0000 (11:48 -0500)]
staging: lustre: libcfs: use bit macro in libcfs headers

Use the BIT macros instead of (1 << ***) in libcfs
headers.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: correct spelling in libcfs_cpu.h
James Simmons [Fri, 18 Nov 2016 16:48:39 +0000 (11:48 -0500)]
staging: lustre: libcfs: correct spelling in libcfs_cpu.h

Spell the word destroy correctly.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: remove blank line in header
James Simmons [Fri, 18 Nov 2016 16:48:38 +0000 (11:48 -0500)]
staging: lustre: libcfs: remove blank line in header

The header libcfs_fail.h has a extra blank line that
is not needed.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: name parameters for function prototypes
James Simmons [Fri, 18 Nov 2016 16:48:37 +0000 (11:48 -0500)]
staging: lustre: libcfs: name parameters for function prototypes

Give the parameters names for the function
prototypes in the libcfs headers.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: remove header's bare unsigned use
James Simmons [Fri, 18 Nov 2016 16:48:36 +0000 (11:48 -0500)]
staging: lustre: libcfs: remove header's bare unsigned use

Fixup the libcfs headers to use the proper unsigned int
instead of raw unsigned.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
7 years agostaging: lustre: libcfs: fixup all header block comments
James Simmons [Fri, 18 Nov 2016 16:48:35 +0000 (11:48 -0500)]
staging: lustre: libcfs: fixup all header block comments

Properly format the incorrect comments sections
that were reported by checkpatch.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>