aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/staging/unisys (follow)
AgeCommit message (Collapse)AuthorFilesLines
2014-12-02staging: unisys: remove duplicate headerSudip Mukherjee2-3/+0
these header files were included multiple times Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-12-02staging: unisys: remove unneeded structureJulia Lawall1-3/+0
Delete a local structure that is only used to be initialized by memset. A semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier x,i; @@ { ... when any -struct i x; <+... when != x - memset(&x,...); ...+> } // </smpl> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-27staging: unisys: remove pointless macrosGreg Kroah-Hartman1-8/+0
These macros aren't used anywhere, so just remove them. Cc: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-27Revert "staging: unisys: fix CamelCase macro names in controlframework.h"Greg Kroah-Hartman1-2/+2
This reverts commit 75185f57f110a7c48d9d33a585320c93334adf0c because it modified the .gitignore file in the root of the tree, without saying it did so, which isn't acceptable. Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> Cc: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: dereferencing NULL pointerSudip Mukherjee1-1/+2
we can reach the label Away in total five situation and in four of of them pDevInfo is NULL. so we were basically dereferencing a NULL pointer. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: unneeded NULL checkSudip Mukherjee1-4/+2
the NULL check for memregion is not required as it has already been checked for NULL after kzalloc. so we can reach this part of the code only if memregion is not NULL. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Add braces to else arm of if/else statementBryan Thompson1-1/+2
Add braces to the else arm of an if/else block in visorchannel_funcs.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Rename goto label AwayBryan Thompson1-27/+27
Rename the goto label used throughout visorchannel_funcs.c from Away to cleanup. Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Adjust lines to contain a maximum of 80 charactersBryan Thompson1-6/+12
Split multiple logging lines to get them closer to 80 characters per line. Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Place logical continuation at the end of a lineBryan Thompson1-2/+2
Move the || logical continuation from the start of the second line of an if statement to the end of the first line. Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Remove unnecessary parenthesesBryan Thompson1-1/+1
Remove extraneous parentheses around a variable in visorchannel_funcs.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Remove space between cast and variableBryan Thompson1-11/+11
Remove the whitespace between a cast and the variable in visorchannel_funcs.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Fix alignment issuesBryan Thompson1-5/+6
Use the appropriate whitespace for multiline statements in visorchannel_funcs.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-07staging: unisys: visorchannel: Remove multiple blank linesBryan Thompson2-2/+0
Remove the instances of multiple blank lines in the visorchannel files. Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: remove unused functions from uislib.cBenjamin Romer1-76/+0
Delete uislib_client_add_vnic() and uislib_client_delete_vnic(), since these are never used. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: Remove extra blank lines from virthba filesKen Depro2-6/+0
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 <kenneth.depro@unisys.com> Signed-off-by: Ben Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: virtpci: Remove GET_BUS_DEV macroBryan Thompson1-15/+33
The GET_BUS_DEV macro contained flow control statements that are undesirable. This patch removes the macro and places the code in each function. Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: refactor visorutil_mod_init()Benjamin Romer1-3/+3
Fix the function name declaration so it is just one line, and add the missing brackets to the else clause in the if statement. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: refactor visorutil_spar_detect()Benjamin Romer1-3/+3
Fix the declaration line so it is just one single line, and add the missing brackets on the else clause in the if statement. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: fix spacing in visorkmodutils.cBenjamin Romer1-4/+0
Remove all the extraneous blank lines from visorkmodutils.c. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: fix bracketing in visor_periodic_work_stop()Benjamin Romer1-1/+2
The last if statement in this function is missing brackets on the else clause. Add them. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: fix spacing in periodic_work.cBenjamin Romer1-16/+0
Get rid of all extraneous blank lines in periodic_work.c. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: fix sizeof() in visor_memregion_create_overlapped()Benjamin Romer1-1/+1
Use the variable name rather than the type in the sizeof() call in this function. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: fix line over 80 characters in mapit()Benjamin Romer1-1/+2
The error output in mapit() in memregion_direct.c has variables past the 80 character limit. Move them to the next line, but the string constant will still go past the limit, and that's okay. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: remove extra blank lines in memregion_direct.cBenjamin Romer1-2/+0
Get rid of the extra blank lines in this file. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: remove unnecessary spaces in casts in memregion_direct.cBenjamin Romer1-3/+3
Get rid of all extra spaces between casts and their targets in this file. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: refactor visor_memregion_create()Benjamin Romer1-4/+4
Fix the sizeof() so it uses the variable name rather than the type, fix the argument alignment to the kzalloc(), and rename the goto label. Away => cleanup Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: remove typedef for MEMREGIONBenjamin Romer5-39/+40
Remove the typedef for this structure and just use struct memregion instead. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: split double assignment in visor_charqueue_create()Benjamin Romer1-1/+2
Split up the doubled assignment in visor_charqueue_create() into two separate assignments. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: refactor CHARQUEUEBenjamin Romer2-18/+20
Remove the typedef and just use struct charqueue instead. Update all references. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-05staging: unisys: fix line spacing in charqueue.cBenjamin Romer1-16/+0
Clean up the extraneous blank lines in charqueue.c. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: Remove unneeded spaces from channel.cKen Depro1-7/+7
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 <kenneth.depro@unisys.com> Signed-off-by: Ben Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase local variables in virtpci_device_add()Bryan Thompson1-7/+7
Rename the following local variables in virtpci_device_add() pIoChan changed to io_chan pDev changed to dev Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase local variables in fix_vbus_dev_info()Bryan Thompson1-8/+8
Rename the following local variables in fix_vbus_dev_info() Change pChan to chan Change devInfo to dev_info Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase parameters of fix_vbus_dev_info()Bryan Thompson1-3/+3
Rename the following fix_vbus_dev_info() parameters devNo changed to dev_no devType changed to dev_type Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase name fix_vbus_devInfo()Bryan Thompson1-7/+7
Rename fix_vbus_devInfo() to fix_vbus_dev_info(). Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase in delete_vbus_device()Bryan Thompson1-2/+2
Rename pDev to dev in delete_vbus_device() Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase write_vbus_devInfo()Bryan Thompson1-3/+3
Rename write_vbus_devInfo() to write_vbus_dev_info() in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase write_vbus_busInfo()Bryan Thompson1-5/+5
Rename write_vbus_busInfo() to write_vbus_bus_info() in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase write_vbus_chpInfo()Bryan Thompson1-5/+5
Rename write_vbus_chpInfo() to write_vbus_chp_info() in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase vbus driver info variableBryan Thompson1-4/+5
Rename BusDriverInfo variable to bus_driver_info in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase virtpci device list lockBryan Thompson1-16/+16
Rename VpcidevListLock to vpcidev_list_lock in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase virtpci device list headBryan Thompson1-16/+16
Rename VpcidevListHead to vpcidev_list_head in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: virtpci: Fix CamelCase Chipset_DriverInfoBryan Thompson1-4/+4
Rename Chipset_DriverInfo to chipset_driver_info in virtpci.c Signed-off-by: Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: remove unused macros from visorchipset.hBenjamin Romer1-13/+0
Delete the macros and related preprocessor code for DBG_GETFILE_PAYLOAD(), DBG_GETFILE(), and DBG_PUTFILE(). Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: remove visorchipset_controlvm_respond_reportEvent()Benjamin Romer1-2/+0
No one is using this definition so remove it. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: fix CamelCase in visorchipset_set_device_context()Benjamin Romer2-4/+4
Fix CamelCase parameter names: busNo => bus_no devNo => dev_no Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: fix CamelCase in visorchipset_set_bus_context()Benjamin Romer2-4/+4
Fix CamelCase parameter: busNo => bus_no Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: remove unused definitions from visorchipset.hBenjamin Romer1-5/+0
Delete visorchipset_get_switch_info() and visorchipset_get_externalport_info() as these functions were declared but not defined, or used. Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-11-03staging: unisys: fix CamelCase in visorchipset_get_device_info()Benjamin Romer2-7/+7
Fix CamelCase names: busNo => bus_no devNo => dev_no devInfo => dev_info Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>