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

2017-06-05 Thread David Kershner
This patchset moves drivers/staging/unisys/include to include/linux/visorbus, and moves drivers/staging/unisys/visorbus to drivers/virt/visorbus. David Kershner (3): include: linux: visorbus: Add visorbus to include/linux directory Documentation: Move visorbus documentation from staging to

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

2017-06-05 Thread David Kershner
This patch moves visorbus from drivers/staging/visorbus to drivers/virt. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- MAINTAINERS | 1 + drivers/staging/unisys/Kconfig | 1 - drivers/staging/unisys

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

2017-06-05 Thread David Kershner
clude folder. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- MAINTAINERS | 1 + drivers/staging/unisys/visorbus/Makefile | 2 -- drivers/staging/unisys/visorbus/controlvmchannel.h

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

2017-06-05 Thread David Kershner
This patch simply does a git mv of the drivers/staging/unisys/Documentation directory to Documentation. It also 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

[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 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 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 v4 08/29] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-06-08 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 v4 03/29] staging: unisys: visorbus: remove unused module parameters

2016-06-08 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 v4 10/29] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-08 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 v4 23/29] staging: unisys: visorbus: Remove notifier-related code from visorbus

2016-06-08 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 v4 06/29] staging: unisys: visornic: Correct comment spelling mistake

2016-06-08 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 v4 09/29] staging: unisys: visorinput: remove unnecessary locking

2016-06-08 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 v4 02/29] staging: unisys: visorbus: check parahotplug_request_complete_result

2016-06-08 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 v4 13/29] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-06-08 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 v4 20/29] staging: unisys: visorbus: Rectify commenting in visorchipset.c

2016-06-08 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 v4 12/29] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-06-08 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 v4 01/29] staging: unisys: visorbus: remove return values for write_vbus functions

2016-06-08 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 v4 26/29] staging: unisys: Move vbushelper.h to visorbus directory

2016-06-08 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 v4 19/29] staging: unisys: visorbus: fix visorchannel.c comments

2016-06-08 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 v4 16/29] staging: unisys: visorbus: remove unused parameter from function

2016-06-08 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 v4 25/29] staging: unisys: visorbus: fix visorbus_private.h comments

2016-06-08 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 v4 21/29] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-06-08 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 v4 28/29] Documentation: Move visorbus documentation from staging to Documentation/

2016-06-08 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 v4 22/29] staging: unisys: visorbus: rectify kerneldoc comment for struct

2016-06-08 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 v4 15/29] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-06-08 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 v4 18/29] staging: unisys: visorbus: fix commenting in visorbus_main.c

2016-06-08 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 v4 29/29] drivers: Add visorbus to the drivers/virt directory

2016-06-08 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 v4 27/29] include: linux: visorbus: Add visorbus to include/linux directory

2016-06-08 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 v4 24/29] staging: unisys: visorbus: Rename function to follow existing convention

2016-06-08 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 v4 14/29] staging: unisys: visorbus: Remove unused functions

2016-06-08 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 v4 11/29] staging: unisys: visorbus: remove periodic_work.h/.c

2016-06-08 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 v4 17/29] staging: unisys: visorbus: fix commenting in vbusdevinfo.h

2016-06-08 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 v4 04/29] staging: unisys: visorbus: remove unused struct

2016-06-08 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 v4 05/29] staging: unisys: visorbus: modify format string to match argument

2016-06-08 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 v4 07/29] staging: unisys: include: Remove thread-related enum members

2016-06-08 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 v4 00/29] Fixed issues raised by tglx, then move visorbus to drivers/virt

2016-06-08 Thread David Kershner
staging: unisys: visorbus: Move visorbus-unique functions to private header staging: unisys: visorbus: rectify kerneldoc comment for struct staging: unisys: visorbus: Remove notifier-related code from visorbus staging: unisys: visorbus: Rename function to follow existing convention David

[PATCH v3 22/30] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-06-04 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 v3 19/30] staging: unisys: visorbus: fix commenting in visorbus_main.c

2016-06-04 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 v3 27/30] staging: unisys: Move vbushelper.h to visorbus directory

2016-06-04 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 v3 24/30] staging: unisys: visorbus: Remove notifier-related code from visorbus

2016-06-04 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 v3 02/30] staging: unisys: visorchipset change -1 return value

2016-06-04 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return value to -EINVAL It also 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

[PATCH v3 25/30] staging: unisys: visorbus: Rename function to follow existing convention

2016-06-04 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 v3 28/30] include: linux: visorbus: Add visorbus to include/linux directory

2016-06-04 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 v3 14/30] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-06-04 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 v3 10/30] staging: unisys: visorinput: remove unnecessary locking

2016-06-04 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 v3 29/30] Documentation: Move visorbus documentation from staging to Documentation/

2016-06-04 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 v3 23/30] staging: unisys: visorbus: rectify kerneldoc comment for struct

2016-06-04 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 v3 16/30] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-06-04 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 v3 06/30] staging: unisys: visorbus: modify format string to match argument

2016-06-04 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 v3 30/30] drivers: Add visorbus to the drivers directory

2016-06-04 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 v3 12/30] staging: unisys: visorbus: remove periodic_work.h/.c

2016-06-04 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 v3 13/30] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-06-04 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 v3 05/30] staging: unisys: visorbus: remove unused struct

2016-06-04 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 v3 01/30] staging: unisys: visorbus change -1 return values

2016-06-04 Thread David Kershner
From: Erik Arfvidson This patch removes the vague -1 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 v3 03/30] staging: unisys: iovmcall_gnuc.h change -1 return values

2016-06-04 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EPERM. This operation is not supported is a good alternative to -1 because the return is basically telling the caller that the processor doesn't support vmcall operations. Signed-off-by: Erik Arfvidson Signed-off-by:

[PATCH v3 18/30] staging: unisys: visorbus: fix commenting in vbusdevinfo.h

2016-06-04 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 v3 15/30] staging: unisys: visorbus: Remove unused functions

2016-06-04 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 v3 17/30] staging: unisys: visorbus: remove unused parameter from function

2016-06-04 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 v3 07/30] staging: unisys: visornic: Correct comment spelling mistake

2016-06-04 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 v3 26/30] staging: unisys: visorbus: fix visorbus_private.h comments

2016-06-04 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 v3 04/30] staging: unisys: visorbus: remove unused module parameters

2016-06-04 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 v3 08/30] staging: unisys: include: Remove thread-related enum members

2016-06-04 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 v3 09/30] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-06-04 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 v3 20/30] staging: unisys: visorbus: fix visorchannel.c comments

2016-06-04 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 v3 11/30] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-04 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 v3 21/30] staging: unisys: visorbus: Rectify commenting in visorchipset.c

2016-06-04 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 v3 00/30] Fixed issues raised by tglx, then move visorbus to drivers/virt

2016-06-04 Thread David Kershner
convention David Kershner (4): staging: unisys: Move vbushelper.h to visorbus directory include: linux: visorbus: Add visorbus to include/linux directory Documentation: Move visorbus documentation from staging to Documentation/ drivers: Add visorbus to the drivers directory Erik

[PATCH v2 10/27] staging: unisys: visorinput: remove unnecessary locking

2016-05-31 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 v2 20/27] staging: unisys: visorbus: make visorchannel function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorchannel.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 23/27] staging: unisys: visorbus: Add kerneldoc-style comments for visorbus API

2016-05-31 Thread David Kershner
From: David Binder Adds kerneldoc-style comments for those functions which may be used outside of the visorbus driver. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 127 drivers

[PATCH v2 24/27] staging: unisys: Move vbushelper.h to visorbus directory

2016-05-31 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 v2 11/27] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-05-31 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 v2 21/27] staging: unisys: visorbus: make visorchipset function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorchipset.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 19/27] staging: unisys: visorbus: make visorbus_private.h function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorbus_private.h now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 18/27] staging: unisys: visorbus: make function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorbus_main.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 15/27] staging: unisys: visorbus: Remove unused functions

2016-05-31 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 v2 22/27] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-05-31 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 v2 00/27] Fixed issues raised by tglx, then move visorbus to drivers/virt

2016-05-31 Thread David Kershner
kerneldoc-like staging: unisys: visorbus: make visorchipset function descriptions more kerneldoc-like staging: unisys: visorbus: Move visorbus-unique functions to private header staging: unisys: visorbus: Add kerneldoc-style comments for visorbus API David Kershner (4): staging

[PATCH v2 09/27] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-05-31 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 v2 07/27] staging: unisys: visornic: Correct comment spelling mistake

2016-05-31 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 v2 01/27] staging: unisys: visorbus change -1 return values

2016-05-31 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EFAULT since it would be the most appropriate, given that this error would only occur in an unexpected bad offset field. Resulting in a bad address. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by

[PATCH v2 02/27] staging: unisys: visorchipset change -1 return value

2016-05-31 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return value to -EINVAL Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 05/27] staging: unisys: visorbus: remove unused struct

2016-05-31 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 v2 14/27] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-05-31 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 v2 08/27] staging: unisys: include: Remove thread-related enum members

2016-05-31 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 v2 03/27] staging: unisys: iovmcall_gnuc.h change -1 return values

2016-05-31 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EPERM. This operation is not supported is a good alternative to -1 because the return is basically telling the caller that the processor doesn't support vmcall operations. Signed-off-by: Erik Arfvidson Signed-off-by:

[PATCH v2 17/27] staging: unisys: visorbus: vbusdeviceinfo function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in vbusdeviceinfo.h now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 16/27] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-05-31 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 v2 12/27] staging: unisys: visorbus: remove periodic_work.h/.c

2016-05-31 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 v2 13/27] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-05-31 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 | 3 ++- drivers/staging/unisys

[PATCH v2 26/27] Documentation: Move visorbus documentation from staging to Documentation/

2016-05-31 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 v2 04/27] staging: unisys: visorbus: remove unused module parameters

2016-05-31 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 v2 27/27] drivers: Add visorbus to the drivers directory

2016-05-31 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 v2 25/27] include: linux: visorbus: Add visorbus to include/linux directory

2016-05-31 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 v2 06/27] staging: unisys: visorbus: modify format string to match argument

2016-05-31 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 01/24] staging: unisys: visorbus: remove unused module parameters

2016-05-24 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 --- drivers/staging/unisys/visorbus/visorbus_main.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff

[PATCH 03/24] staging: unisys: visorbus: modify format string to match argument

2016-05-24 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 --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 02/24] staging: unisys: visorbus: remove unused struct

2016-05-24 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 --- drivers/staging/unisys/visorbus/visorbus_main.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging

  1   2   >