[PATCH 15/28] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-06-10 Thread David Kershner
From: Bryan Thompson The STANDALONE_CLIENT define is no longer used by Unisys driver code. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging

[PATCH 08/28] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-06-10 Thread David Kershner
From: Tim Sell periodic_test_workqueue was an unused relic from the past, and was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/unisys

[PATCH 17/28] staging: unisys: visorbus: fix commenting in vbusdevinfo.h

2016-06-10 Thread David Kershner
: David Binder Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/vbusdeviceinfo.h | 65 1 file changed, 43 insertions(+), 22 deletions(-) diff --git a/drivers/staging/unisys/visorbus/vbusdeviceinfo.h b/drivers/staging/unisys

[PATCH 11/28] staging: unisys: visorbus: remove periodic_work.h/.c

2016-06-10 Thread David Kershner
From: Tim Sell These files were made no-longer-necessary by recent commits. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/periodic_work.h | 40 - drivers/staging/unisys/visorbus/Makefile| 1 - drivers/staging/unisys/visorbus

[PATCH 12/28] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-06-10 Thread David Kershner
From: Bryan Thompson visordriver_callback_lock is just a binary semaphore that logically makes more sense as a mutex. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 +- drivers/staging/unisys

[PATCH 01/28] staging: unisys: visorbus: remove return values for write_vbus functions

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch removes the return values from the write_vbus_* channel functions. Nobody was checking the return values and the value stored into the vbus info is not critical to the functioning of the device. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner

[PATCH 04/28] staging: unisys: visorbus: remove unused struct

2016-06-10 Thread David Kershner
From: David Binder Removes unused struct definition, channel_size_info, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 6 -- 1 file changed, 6 deletions(-) diff

[PATCH 05/28] staging: unisys: visorbus: modify format string to match argument

2016-06-10 Thread David Kershner
From: David Binder Modifies the format string of snprintf to expect an unsigned int instead of a signed one, per the supplied argument. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- 1 file changed

[PATCH 07/28] staging: unisys: include: Remove thread-related enum members

2016-06-10 Thread David Kershner
From: David Binder Code relating to ktheads was previously removed from s-Par driver code. This patch cleans up lingering remnants of kthreads by removing thread- related enum types. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys

[PATCH 10/28] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-10 Thread David Kershner
, because none of them ever perform any operations that would be invalid in atomic context. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 10 +++-- drivers/staging/unisys/visorbus/visorbus_main.c | 54 +++-- 2 files

[PATCH 25/28] staging: unisys: visorbus: fix visorbus_private.h comments

2016-06-10 Thread David Kershner
From: Tim Sell This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorbus_private.h: * Minor typos were corrected. * Useless comments were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus

[PATCH 19/28] staging: unisys: visorbus: fix visorchannel.c comments

2016-06-10 Thread David Kershner
typos were corrected. * Useless comments were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner Signed-off-by: David Binder --- drivers/staging/unisys/visorbus/visorchannel.c | 83 ++ 1 file changed, 71 insertions(+), 12 deletions(-) diff --git a/drivers

[PATCH 18/28] staging: unisys: visorbus: fix commenting in visorbus_main.c

2016-06-10 Thread David Kershner
tweaked so as to use appropriate conventions. Signed-off-by: David Binder Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 420 1 file changed, 289 insertions(+), 131 deletions(-) diff --git a/drivers/staging

[PATCH 22/28] staging: unisys: visorbus: rectify kerneldoc comment for struct

2016-06-10 Thread David Kershner
From: David Binder Fixes the kerneldoc comment for struct visor_device - the struct members were not listed with the appropriate @ prefix. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 34 +++ 1 file

[PATCH 13/28] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-06-10 Thread David Kershner
Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - drivers/staging/unisys/visorbus/visorchannel.c | 17 - drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 3 files changed, 20 deletions(-) diff --git a/drivers

[PATCH 23/28] staging: unisys: visorbus: Remove notifier-related code from visorbus

2016-06-10 Thread David Kershner
files within the same driver, known as 'visorbus'. This eliminated the need for a registration mechanism, but it has remained nevertheless until now. For the sake of simplification, this registration mechanism is now being removed. Signed-off-by: David Binder Signed-off-by: Davi

[PATCH 14/28] staging: unisys: visorbus: Remove unused functions

2016-06-10 Thread David Kershner
From: Bryan Thompson Remove visorbus_clear_channel, visorchannel_signalqueue_slots_avail, visorchannel_signalqueue_max_slots, visorchannel_clear, and visorchannel_debug which are no longer called by any driver. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell

[PATCH 26/28] staging: unisys: Move vbushelper.h to visorbus directory

2016-06-10 Thread David Kershner
Only visorbus needs this header file so move it to visorbus directory. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/{include => visorbus}/vbushelper.h | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename drivers/staging/unisys/{include => vi

[PATCH 21/28] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-06-10 Thread David Kershner
From: David Binder Moves function prototypes that are unique to visorbus from include/visorbus.h to visorbus/visorbus_private.h. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 25

[PATCH 28/28] staging: unisys: visorinput: make lock_visor_dev a mutex

2016-06-10 Thread David Kershner
From: Tim Sell Since lock_visor_dev is a binary semaphore it makes more sense to use a mutex lock. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff

[PATCH 20/28] staging: unisys: visorbus: Rectify commenting in visorchipset.c

2016-06-10 Thread David Kershner
From: David Binder Adds kerneldoc formatting to appropriate functions. Other multi-line comments now use proper formatting. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 292 - 1

[PATCH 27/28] staging: unisys: visorinput: ensure proper locking wrt creation & ints

2016-06-10 Thread David Kershner
ned-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 74 +- 1 file changed, 49 insertions(+), 25 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/drivers/staging/unisys/visorinput/visorinput.c index f633985..c13e698 10

[PATCH 24/28] staging: unisys: visorbus: Rename function to follow existing convention

2016-06-10 Thread David Kershner
From: David Binder Renames visorchipset_device_pause_response to device_pause_response, thereby following the convention that other responder functions follow. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c| 4 ++-- drivers

[PATCH 16/28] staging: unisys: visorbus: remove unused parameter from function

2016-06-10 Thread David Kershner
From: Tim Sell The off parameter to visorchannel_create_guts() was never used, so it was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchannel.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH RESEND 00/28] staging: unisys: fix visorbus & visorinput issues raised by tglx

2016-06-10 Thread David Kershner
visorbus staging: unisys: visorbus: Rename function to follow existing convention David Kershner (1): staging: unisys: Move vbushelper.h to visorbus directory Erik Arfvidson (2): staging: unisys: visorbus: remove return values for write_vbus functions staging: unisys: visorbus

[PATCH RESEND 02/28] staging: unisys: visorbus: check parahotplug_request_complete_result

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch modifies the caller of parahotplug_request_complete() to check the return value and return appropriate result. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 5 - 1

[PATCH RESEND 04/28] staging: unisys: visorbus: remove unused struct

2016-06-10 Thread David Kershner
From: David Binder Removes unused struct definition, channel_size_info, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 6 -- 1 file changed, 6 deletions(-) diff

[PATCH RESEND 03/28] staging: unisys: visorbus: remove unused module parameters

2016-06-10 Thread David Kershner
From: David Binder Removes unused module parameters from visorbus_main.c, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 9 + 1 file changed, 1 insertion

[PATCH RESEND 01/28] staging: unisys: visorbus: remove return values for write_vbus functions

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch removes the return values from the write_vbus_* channel functions. Nobody was checking the return values and the value stored into the vbus info is not critical to the functioning of the device. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner

[PATCH RESEND 24/28] staging: unisys: visorbus: Rename function to follow existing convention

2016-06-10 Thread David Kershner
From: David Binder Renames visorchipset_device_pause_response to device_pause_response, thereby following the convention that other responder functions follow. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c| 4 ++-- drivers

[PATCH RESEND 17/28] staging: unisys: visorbus: fix commenting in vbusdevinfo.h

2016-06-10 Thread David Kershner
: David Binder Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/vbusdeviceinfo.h | 65 1 file changed, 43 insertions(+), 22 deletions(-) diff --git a/drivers/staging/unisys/visorbus/vbusdeviceinfo.h b/drivers/staging/unisys

[PATCH RESEND 21/28] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-06-10 Thread David Kershner
From: David Binder Moves function prototypes that are unique to visorbus from include/visorbus.h to visorbus/visorbus_private.h. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 25

[PATCH RESEND 05/28] staging: unisys: visorbus: modify format string to match argument

2016-06-10 Thread David Kershner
From: David Binder Modifies the format string of snprintf to expect an unsigned int instead of a signed one, per the supplied argument. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- 1 file changed

[PATCH RESEND 27/28] staging: unisys: visorinput: ensure proper locking wrt creation & ints

2016-06-10 Thread David Kershner
ned-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 74 +- 1 file changed, 49 insertions(+), 25 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/drivers/staging/unisys/visorinput/visorinput.c index f633985..c13e698 10

[PATCH RESEND 14/28] staging: unisys: visorbus: Remove unused functions

2016-06-10 Thread David Kershner
From: Bryan Thompson Remove visorbus_clear_channel, visorchannel_signalqueue_slots_avail, visorchannel_signalqueue_max_slots, visorchannel_clear, and visorchannel_debug which are no longer called by any driver. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell

[PATCH RESEND 12/28] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-06-10 Thread David Kershner
From: Bryan Thompson visordriver_callback_lock is just a binary semaphore that logically makes more sense as a mutex. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 +- drivers/staging/unisys

[PATCH RESEND 18/28] staging: unisys: visorbus: fix commenting in visorbus_main.c

2016-06-10 Thread David Kershner
tweaked so as to use appropriate conventions. Signed-off-by: David Binder Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 420 1 file changed, 289 insertions(+), 131 deletions(-) diff --git a/drivers/staging

[PATCH RESEND 07/28] staging: unisys: include: Remove thread-related enum members

2016-06-10 Thread David Kershner
From: David Binder Code relating to ktheads was previously removed from s-Par driver code. This patch cleans up lingering remnants of kthreads by removing thread- related enum types. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys

[PATCH RESEND 13/28] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-06-10 Thread David Kershner
Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - drivers/staging/unisys/visorbus/visorchannel.c | 17 - drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 3 files changed, 20 deletions(-) diff --git a/drivers

[PATCH RESEND 16/28] staging: unisys: visorbus: remove unused parameter from function

2016-06-10 Thread David Kershner
From: Tim Sell The off parameter to visorchannel_create_guts() was never used, so it was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchannel.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH RESEND 19/28] staging: unisys: visorbus: fix visorchannel.c comments

2016-06-10 Thread David Kershner
typos were corrected. * Useless comments were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner Signed-off-by: David Binder --- drivers/staging/unisys/visorbus/visorchannel.c | 83 ++ 1 file changed, 71 insertions(+), 12 deletions(-) diff --git a/drivers

[PATCH RESEND 10/28] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-10 Thread David Kershner
, because none of them ever perform any operations that would be invalid in atomic context. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 10 +++-- drivers/staging/unisys/visorbus/visorbus_main.c | 54 +++-- 2 files

[PATCH RESEND 06/28] staging: unisys: visornic: Correct comment spelling mistake

2016-06-10 Thread David Kershner
From: David Binder Fixes a comment spelling mistake in visornic. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visornic/visornic_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys

[PATCH RESEND 08/28] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-06-10 Thread David Kershner
From: Tim Sell periodic_test_workqueue was an unused relic from the past, and was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/unisys

[PATCH RESEND 25/28] staging: unisys: visorbus: fix visorbus_private.h comments

2016-06-10 Thread David Kershner
From: Tim Sell This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorbus_private.h: * Minor typos were corrected. * Useless comments were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus

[PATCH RESEND 23/28] staging: unisys: visorbus: Remove notifier-related code from visorbus

2016-06-10 Thread David Kershner
files within the same driver, known as 'visorbus'. This eliminated the need for a registration mechanism, but it has remained nevertheless until now. For the sake of simplification, this registration mechanism is now being removed. Signed-off-by: David Binder Signed-off-by: Davi

[PATCH RESEND 11/28] staging: unisys: visorbus: remove periodic_work.h/.c

2016-06-10 Thread David Kershner
From: Tim Sell These files were made no-longer-necessary by recent commits. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/periodic_work.h | 40 - drivers/staging/unisys/visorbus/Makefile| 1 - drivers/staging/unisys/visorbus

[PATCH RESEND 15/28] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-06-10 Thread David Kershner
From: Bryan Thompson The STANDALONE_CLIENT define is no longer used by Unisys driver code. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging

[PATCH RESEND 28/28] staging: unisys: visorinput: make lock_visor_dev a mutex

2016-06-10 Thread David Kershner
From: Tim Sell Since lock_visor_dev is a binary semaphore it makes more sense to use a mutex lock. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff

[PATCH RESEND 09/28] staging: unisys: visorinput: remove unnecessary locking

2016-06-10 Thread David Kershner
red state in devdata->interrupts_enabled. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 57 +- 1 file changed, 47 insertions(+), 10 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/d

[PATCH RESEND 20/28] staging: unisys: visorbus: Rectify commenting in visorchipset.c

2016-06-10 Thread David Kershner
From: David Binder Adds kerneldoc formatting to appropriate functions. Other multi-line comments now use proper formatting. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 292 - 1

[PATCH RESEND 26/28] staging: unisys: Move vbushelper.h to visorbus directory

2016-06-10 Thread David Kershner
Only visorbus needs this header file so move it to visorbus directory. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/vbushelper.h | 46 drivers/staging/unisys/visorbus/vbushelper.h | 46 2 files

[PATCH RESEND 22/28] staging: unisys: visorbus: rectify kerneldoc comment for struct

2016-06-10 Thread David Kershner
From: David Binder Fixes the kerneldoc comment for struct visor_device - the struct members were not listed with the appropriate @ prefix. Signed-off-by: David Binder --- drivers/staging/unisys/include/visorbus.h | 34 +++ 1 file changed, 17 insertions(+), 17 deleti

[PATCH 1/3] include: linux: visorbus: Add visorbus to include/linux directory

2016-06-10 Thread David Kershner
folders. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/MAINTAINERS| 2 +- drivers/staging/unisys/visorbus/Makefile | 2 -- drivers/staging/unisys/visorbus/controlvmchannel.h| 2 +- drivers

[PATCH 2/3] Documentation: Move visorbus documentation from staging to Documentation/

2016-06-10 Thread David Kershner
This patch simple does a git mv of the drivers/staging/unisys/Documentation directory to Documentation. Renames overview.txt to visorbus.txt and renames sysfs-platform-visorchipset to the correct name sysfs-bus-visorbus. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- .../ABI/stable

[PATCH 3/3] drivers: Add visorbus to the drivers/virt directory

2016-06-10 Thread David Kershner
visorbus is currently located at drivers/staging/visorbus, this patch moves it to drivers/virt. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/Kconfig| 3 +-- drivers/staging/unisys/Makefile

[PATCH 0/3] move visorbus out of staging to drivers/virt/visorbus

2016-06-10 Thread David Kershner
This patchset comprises the last 3 patches of the previously-submitted patchset (but retracted): [PATCH v4 00/29] Fixed issues raised by tglx, then move visorbus to drivers/virt David Kershner (3): include: linux: visorbus: Add visorbus to include/linux direc

[PATCH 08/21] staging: unisys: visorbus: Convert visorchipset.h refs

2016-09-01 Thread David Kershner
From: David Binder visorbus_private.h was previously named visorchipset.h. This patch removes references to the old filename, and updates them with the new filename. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus

[PATCH 13/21] staging: unisys: visorbus: Merge controlvm completion status to controlvmchannel

2016-09-01 Thread David Kershner
The completion status are valid only for the controlvm channel so they should be moved into its header file. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/controlvmchannel.h | 76 .../unisys/visorbus/controlvmcompletionstatus.h

[PATCH 18/21] staging: unisys: visorbus: Move non global guestlinuxdebug to visorbus

2016-09-01 Thread David Kershner
Guestlinuxdebug.h is only used by visorbus, move the include file into its directory instead of the global one. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/{include => visorbus}/guestlinuxdebug.h | 0 1 file changed, 0 insertions(+), 0 deletions(-) ren

[PATCH 03/21] staging: unisys: visornic_main.c remove unused pound defines

2016-09-01 Thread David Kershner
From: Erik Arfvidson This removes two unused pound defines in the visornic_main.c file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys/visornic

[PATCH 21/21] staging: unisys: visorbus: cleanup include listings

2016-09-01 Thread David Kershner
Cleaned up include section of files so that they only include the files needed by them. If an include file includes the file already don't repeat it. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- drivers/staging/unisys/vis

[PATCH 07/21] staging: unisys: Move vbushelper.h contents to visorbus_private.h

2016-09-01 Thread David Kershner
From: Bryan Thompson The contents of vbushelper.h are now only used by visorbus, so it no longer needs to be a general include file and it can be incorporated in the visorbus private header. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers

[PATCH 09/21] staging: unisys: visorbus: Merge channel_guid.h with channel.h

2016-09-01 Thread David Kershner
From: David Binder Merges the contents of channel_guid.h with channel.h, as the former was not a proper header file. Therefore, the channel_guid.h is no longer needed. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include

[PATCH 04/21] staging: unisys: visorbus: remove g_devicechangestate_packet

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the unused struct controlvm_message_packet g_devicechangestate_packet. The struct is set but never called. Signed-off-by: Jon Frisch Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 1 file changed, 2 deletions

[PATCH 19/21] staging: unisys: remove check for GNUC

2016-09-01 Thread David Kershner
The check for GNUC is redundant and is not needed for vmcallinterface.h. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/vmcallinterface.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys/visorbus/vmcallinterface.h b/drivers

[PATCH 11/21] staging: unisys: visorinput: Remove ultrainputreport.h include

2016-09-01 Thread David Kershner
From: Bryan Thompson Stop ultrainputreport.h from including itself. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorinput/ultrainputreport.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys

[PATCH 06/21] staging: unisys: visorbus: remove putfile_buffer_entry

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the unused struct putfile_buffer_entry. Signed-off-by: Jon Frisch Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b

[PATCH 10/21] staging: unisys: visorinput: Update visorinput.c includes

2016-09-01 Thread David Kershner
From: Bryan Thompson Remove unnecessary channel.h include from visorinput.c. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorinput/visorinput.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/unisys

[PATCH 01/21] staging: unisys: visorchipset.c remove unused pound defines

2016-09-01 Thread David Kershner
From: Erik Arfvidson This removes three unused pound defines in the visorchipset.c file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/unisys/visorbus

[PATCH 15/21] staging: unisys: visorbus: merge vbusdeviceinfo.h into vbuschannel

2016-09-01 Thread David Kershner
vbusdeviceinfo.h was only used by vbuschannel.h so merge them together. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/vbuschannel.h | 212 ++- drivers/staging/unisys/visorbus/vbusdeviceinfo.h | 234 - drivers

[PATCH 12/21] staging: unisys: merge diagchannel.h to visorbus.h

2016-09-01 Thread David Kershner
From: Erik Arfvidson It is pointless to have a dedicated include for another include. So I merged them together. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/diagchannel.h | 38 --- drivers

[PATCH 00/21] Clean up header files and remove unused structures

2016-09-01 Thread David Kershner
staging: unisys: visorinput: Remove ultrainputreport.h include David Binder (2): staging: unisys: visorbus: Convert visorchipset.h refs staging: unisys: visorbus: Merge channel_guid.h with channel.h David Kershner (9): staging: unisys: visorbus: Merge controlvm completion status to

[PATCH 17/21] staging: unisys: visorbus: remove include for vbuschannel.h

2016-09-01 Thread David Kershner
visorbus_main.c does not need to include vbuschannel.h. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b/drivers/staging/unisys/visorbus

[PATCH 05/21] staging: unisys: visorbus: remove unused lists

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the following lists that were never used: LIST_HEAD(bus_info_list) LIST_HEAD(dev_info_list) LIST_HEAD(putfile_request_list) Signed-off-by: Jon Frisch Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 10 -- 1 file

[PATCH 20/21] staging: unisys: visorbus: merge guestlinuxdebug with vmcallinterface.h

2016-09-01 Thread David Kershner
Merge guestlinuxdebug into vmcallinterface.h since it is dependent on it Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/guestlinuxdebug.h | 180 -- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - drivers/staging/unisys

[PATCH 02/21] staging: unisys: visorbus_main.c remove unused pound defines

2016-09-01 Thread David Kershner
From: Erik Arfvidson This removes two unused pound defines in the visorbus_main.c file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/unisys/visorbus

[PATCH 16/21] staging: unisys: visorbus: Merge iovmcall_gnuc into vmcallinterface

2016-09-01 Thread David Kershner
iovmcall_gnuc was only being referenced by vmcallinterface, so merge them together. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/iovmcall_gnuc.h | 48 --- drivers/staging/unisys/visorbus/vmcallinterface.h | 35

[PATCH 14/21] staging: unisys: visorbus: Reorder functions to avoid including header

2016-09-01 Thread David Kershner
Visorchannel does not need to include visorbus_private.h; all it was using it for was function prototypes. A simple reorder causes it to no longer need to include it. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchannel.c | 245

[PATCH v2 05/21] staging: unisys: visorbus: remove unused lists

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the following lists that were never used: LIST_HEAD(bus_info_list) LIST_HEAD(dev_info_list) LIST_HEAD(putfile_request_list) Signed-off-by: Jon Frisch Signed-off-by: David Kershner Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus

[PATCH v2 02/21] staging: unisys: visorbus_main.c remove unused pound defines

2016-09-01 Thread David Kershner
From: Erik Arfvidson This removes two unused pound defines in the visorbus_main.c file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/visorbus_main.c | 3 --- 1 file changed, 3 deletions(-) diff --git a

[PATCH v2 06/21] staging: unisys: visorbus: remove putfile_buffer_entry

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the unused struct putfile_buffer_entry. Signed-off-by: Jon Frisch Signed-off-by: David Kershner Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/visorchipset.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging

[PATCH v2 00/21] Clean up header files and remove unused structures

2016-09-01 Thread David Kershner
with channel.h David Kershner (9): staging: unisys: visorbus: Merge controlvm completion status to controlvmchannel staging: unisys: visorbus: Reorder functions to avoid including header staging: unisys: visorbus: merge vbusdeviceinfo.h into vbuschannel staging: unisys: visorbus: Merge

[PATCH v2 18/21] staging: unisys: visorbus: Move non global guestlinuxdebug to visorbus

2016-09-01 Thread David Kershner
Guestlinuxdebug.h is only used by visorbus, move the include file into its directory instead of the global one. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/{include => visorbus}/guestlinuxdebug.h | 0 1 file changed

[PATCH v2 15/21] staging: unisys: visorbus: merge vbusdeviceinfo.h into vbuschannel

2016-09-01 Thread David Kershner
vbusdeviceinfo.h was only used by vbuschannel.h so merge them together. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/vbuschannel.h | 212 ++- drivers/staging/unisys/visorbus/vbusdeviceinfo.h

[PATCH v2 14/21] staging: unisys: visorbus: Reorder functions to avoid including header

2016-09-01 Thread David Kershner
Visorchannel does not need to include visorbus_private.h; all it was using it for was function prototypes. A simple reorder causes it to no longer need to include it. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus

[PATCH v2 07/21] staging: unisys: Move vbushelper.h contents to visorbus_private.h

2016-09-01 Thread David Kershner
From: Bryan Thompson The contents of vbushelper.h are now only used by visorbus, so it no longer needs to be a general include file and it can be incorporated in the visorbus private header. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg

[PATCH v2 11/21] staging: unisys: visorinput: Remove ultrainputreport.h include

2016-09-01 Thread David Kershner
From: Bryan Thompson Stop ultrainputreport.h from including itself. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorinput/ultrainputreport.h | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH v2 08/21] staging: unisys: visorbus: Convert visorchipset.h refs

2016-09-01 Thread David Kershner
From: David Binder visorbus_private.h was previously named visorchipset.h. This patch removes references to the old filename, and updates them with the new filename. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers

[PATCH v2 20/21] staging: unisys: visorbus: merge guestlinuxdebug with vmcallinterface.h

2016-09-01 Thread David Kershner
Merge guestlinuxdebug into vmcallinterface.h since it is dependent on it Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/guestlinuxdebug.h | 180 -- drivers/staging/unisys/visorbus/visorbus_main.c

[PATCH v2 10/21] staging: unisys: visorinput: Update visorinput.c includes

2016-09-01 Thread David Kershner
From: Bryan Thompson Remove unnecessary channel.h include from visorinput.c. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorinput/visorinput.c | 1 - 1 file changed, 1 deletion(-) diff --git

[PATCH v2 04/21] staging: unisys: visorbus: remove g_devicechangestate_packet

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the unused struct controlvm_message_packet g_devicechangestate_packet. The struct is set but never called. Signed-off-by: Jon Frisch Signed-off-by: David Kershner Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 1

[PATCH v2 12/21] staging: unisys: merge diagchannel.h to visorbus.h

2016-09-01 Thread David Kershner
From: Erik Arfvidson It is pointless to have a dedicated include for another include. So I merged them together. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/include/diagchannel.h | 38

[PATCH v2 09/21] staging: unisys: visorbus: Merge channel_guid.h with channel.h

2016-09-01 Thread David Kershner
From: David Binder Merges the contents of channel_guid.h with channel.h, as the former was not a proper header file. Therefore, the channel_guid.h is no longer needed. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman

[PATCH v2 03/21] staging: unisys: visornic_main.c remove unused pound defines

2016-09-01 Thread David Kershner
From: Erik Arfvidson This removes two unused pound defines in the visornic_main.c file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visornic/visornic_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a

[PATCH v2 17/21] staging: unisys: visorbus: remove include for vbuschannel.h

2016-09-01 Thread David Kershner
visorbus_main.c does not need to include vbuschannel.h. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b

[PATCH v2 16/21] staging: unisys: visorbus: Merge iovmcall_gnuc into vmcallinterface

2016-09-01 Thread David Kershner
iovmcall_gnuc was only being referenced by vmcallinterface, so merge them together. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/iovmcall_gnuc.h | 48 --- drivers/staging/unisys/visorbus

[PATCH v2 21/21] staging: unisys: visorbus: cleanup include listings

2016-09-01 Thread David Kershner
Cleaned up include section of files so that they only include the files needed by them. If an include file includes the file already don't repeat it. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/visorbus_main.

[PATCH v2 13/21] staging: unisys: visorbus: Merge controlvm completion status to controlvmchannel

2016-09-01 Thread David Kershner
The completion status are valid only for the controlvm channel so they should be moved into its header file. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/controlvmchannel.h | 76 .../unisys/visorbus

[PATCH v2 19/21] staging: unisys: remove check for GNUC

2016-09-01 Thread David Kershner
The check for GNUC is redundant and is not needed for vmcallinterface.h. Signed-off-by: David Kershner Reviewed-by: Tim Sell Reported-by: Greg Kroah-Hartman --- drivers/staging/unisys/visorbus/vmcallinterface.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys

<    1   2   3   4   5   6   7   8   9   10   >