summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* staging: speakup: speakup_soft.c Logical continuationWalt Feasel2016-11-231-2/+2
| | | | | | | | | 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>
* staging: speakup: speakup_soft.c Space around operatorWalt Feasel2016-11-231-14/+14
| | | | | | | | 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>
* staging: speakup: speakup_spkout.c Space around operatorWalt Feasel2016-11-231-12/+12
| | | | | | | | 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>
* staging: speakup: speakup_spkout.c comment modificationsWalt Feasel2016-11-231-4/+3
| | | | | | | Make modifications to comments Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: speakup: speakup_txprt.c Spaces preferred around operatorWalt Feasel2016-11-231-11/+11
| | | | | | | | 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>
* staging: speakup: speakup_txprt.c Comment modificationsWalt Feasel2016-11-231-4/+3
| | | | | | | Make modifications to comments Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: speakup: spk_priv_keyinfo.h Space around operatorWalt Feasel2016-11-231-22/+22
| | | | | | | | 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>
* staging: speakup: spk_priv_keyinfo.h align columnsWalt Feasel2016-11-231-53/+51
| | | | | | | Make modifications to align columns Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: speakup: spk_types.h Align parenthesisWalt Feasel2016-11-231-1/+1
| | | | | | | | 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>
* staging: speakup: spk_types.h Comment modificationsWalt Feasel2016-11-231-8/+6
| | | | | | | Make modifications to comment style Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: emxx_udc: emxx_udc.c Align parenthesisWalt Feasel2016-11-231-2/+2
| | | | | | | | 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>
* staging: emxx_udc: emxx_udc.c Spaces preferred around operatorsWalt Feasel2016-11-231-1/+1
| | | | | | | | 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>
* staging: emxx_udc: emxx_udc.c {} Single statement blocksWalt Feasel2016-11-231-2/+1
| | | | | | | | | 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>
* staging: emxx_udc: emxx_udc.c Logical continuationWalt Feasel2016-11-231-22/+15
| | | | | | | | | 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>
* staging: emxx_udc: emxx_udc.c Blank lines after {Walt Feasel2016-11-231-13/+0
| | | | | | | | | 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>
* staging: unisys: visorbus: bus_configure add error handlingDavid Kershner2016-11-231-16/+29
| | | | | | | | | 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>
* staging: unisys: visorbus: return EAGAIN when emptyDavid Kershner2016-11-231-1/+2
| | | | | | | | | | 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>
* staging: unisys: visorbus: device_responder add error handlingDavid Kershner2016-11-231-4/+4
| | | | | | | | | 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>
* staging: unisys: visorbus: device_changestate_responder add error handlingDavid Kershner2016-11-231-6/+5
| | | | | | | | | 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>
* staging: unisys: visorbus: bus_responder add error handlingDavid Kershner2016-11-231-4/+4
| | | | | | | | | 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>
* staging: unisys: visorbus: save_crash_message add error handlingDavid Kershner2016-11-231-22/+28
| | | | | | | | 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>
* staging: unisys: visorbus: controlvm_respond_physdev_changestate add error ↵David Kershner2016-11-231-5/+3
| | | | | | | | | | 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>
* staging: unisys: visorbus: controlvm_respond add error handlingDavid Kershner2016-11-231-6/+4
| | | | | | | | | 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>
* staging: unisys: visorbus: chipset_init add error handlingDavid Kershner2016-11-231-2/+6
| | | | | | | | | 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>
* staging: unisys: visorbus: controlvm_respond_chipset_init add error handlingDavid Kershner2016-11-231-5/+3
| | | | | | | | | | 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>
* staging: unisys: visorbus: visorbus_private rename 'local' var to 'dest'Erik Arfvidson2016-11-231-2/+2
| | | | | | | | | 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>
* staging: unisys: visorbus: visorchannel.c rename 'local' variable to 'dest'Erik Arfvidson2016-11-231-5/+5
| | | | | | | | 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>
* staging: unisys: include: fix pound definesErik Arfvidson2016-11-231-10/+4
| | | | | | | | | | 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>
* staging: unisys: include: fixed iochannel.h commentsErik Arfvidson2016-11-231-147/+176
| | | | | | | | | | | | | | | | | | | | | | 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>
* staging: slicoss: change some macros into inline functionsSergio Paracuellos2016-11-232-9/+17
| | | | | | | | | | | 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>
* Staging: rtl8712: rtl871x_*: Removed unnecessary else statements after a ↵Shiva Kerdel2016-11-212-4/+4
| | | | | | | | | | 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>
* staging: rtl8712: Fix coding style warnings on Block commentsVijai Kumar K2016-11-216-113/+101
| | | | | | | | 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>
* staging/lustre: Use proper number of bytes in copy_from_userJian Yu2016-11-211-2/+9
| | | | | | | | | | | | | 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>
* staging: vt6656: Fix coding style warnings on Block commentsVijai Kumar K2016-11-211-3/+3
| | | | | | | | 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>
* staging: vt6655: Fixed coding style warnings on Block commentsVijai Kumar K2016-11-212-5/+8
| | | | | | | | 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>
* staging: dgnc: dgnc_tty: Remove blank linesVijai Kumar K2016-11-211-2/+0
| | | | | | | 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>
* staging: skein: threefish_block.c Remove blank linesWalt Feasel2016-11-211-16/+0
| | | | | | | | | | 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>
* staging: dgnc: digi.h Spelling correctionWalt Feasel2016-11-211-1/+1
| | | | | | | Make spelling correction for 'regular' Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgnc: digi.h Comment style modificationsWalt Feasel2016-11-211-48/+47
| | | | | | | Make modifications to comment style Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgnc: dgnc_tty.c Spelling correctionsWalt Feasel2016-11-211-5/+5
| | | | | | | Make spelling corrections for 'transitions' and 'satisfy' Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgnc: dgnc_tty.c comment style modificationsWalt Feasel2016-11-211-143/+86
| | | | | | | Make modifications to comment style Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: dgnc: dgnc_tty.c Align on parenthesisWalt Feasel2016-11-211-7/+6
| | | | | | | | 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>
* staging: dgnc: dgnc_tty.c Space preferred aroundWalt Feasel2016-11-211-1/+1
| | | | | | | | 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>
* staging: slicoss: fix different address space warningsSergio Paracuellos2016-11-211-1/+1
| | | | | | | | | | 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>
* staging: speakup: synth.c Align parenthesisWalt Feasel2016-11-211-1/+1
| | | | | | | | 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>
* staging: speakup: synth.c Comment modificationsWalt Feasel2016-11-211-2/+3
| | | | | | | Make modifications to comment style Signed-off-by: Walt Feasel <waltfeasel@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: speakup: synth.c Logical continuationWalt Feasel2016-11-211-2/+2
| | | | | | | | | 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>
* staging: speakup: synth.c Blank line before }Walt Feasel2016-11-211-1/+0
| | | | | | | | 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>
* staging: speakup: synth.c Spaces around operatorsWalt Feasel2016-11-211-5/+5
| | | | | | | | 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>
* staging: speakup: thread.c align on parenthesisWalt Feasel2016-11-211-1/+1
| | | | | | | | 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>