From: Erik Arfvidson erik.arfvid...@unisys.com
this patch simply migrates file.h functionality to visorchipset.h
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/file.c | 5
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchannel.c | 15 ++-
drivers/staging/unisys/visorutil/memregion.h | 2
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchannel.c | 17 +++--
drivers/staging/unisys/visorutil/memregion.h| 1
From: Erik Arfvidson erik.arfvid...@unisys.com
This patch removes unused chipset feature
ULTRA_CHIPSET_FEATURE_PCIVBUS
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/common-spar/include/channels
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/globals.h | 2 --
drivers/staging/unisys/visorchipset/visorchipset.h | 2 ++
2 files changed, 2
From: Erik Arfvidson erik.arfvid...@unisys.com
this patch removes unused chipset command on the guest
CONTROLVM_CHIPSET_SHUTDOWN
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/common-spar/include
From: Don Zickus dzic...@redhat.com
Remove stale code.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c | 32 -
1 file changed, 32 deletions(-)
diff --git a/drivers
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchipset.c | 201 +
1 file changed, 4 insertions(+), 197 deletions(-)
diff --git
From: Don Zickus dzic...@redhat.com
Handling the sysfs attributes become easier to deal with when you can just
run container_of(dev) to get devdata.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus
'. :-)
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c
b/drivers/staging/unisys/visorbus
-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/visorbus.h | 1 -
drivers/staging/unisys/visorbus/visorchannel.c | 1 -
drivers/staging/unisys/visorbus/visorchipset.c | 1 -
3 files changed, 3 deletions(-)
diff --git a/drivers/staging/unisys/include/visorbus.h
b
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorutil/memregion.h| 6 +-
drivers/staging/unisys/visorutil/memregion_direct.c | 6 --
2 files
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchannel.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/staging
From: David Kershner david.kersh...@unisys.com
The visorchipset driver originally serviced both servers and
clients. This implementation is client only so remove some
more server side implementation.
Signed-off-by: David Kershner david.kersh...@unisys.com
Signed-off-by: Benjamin Romer
-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c | 75 +++--
1 file changed, 58 insertions(+), 17 deletions(-)
diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c
b/drivers/staging/unisys/visorbus/visorbus_main.c
index
From: Erik Arfvidson erik.arfvid...@unisys.com
This patch renames send_irq_handle to reserved1 since this variable
is unused in the guest side.
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/common-spar
From: Don Zickus dzic...@redhat.com
This was replaced by an earlier patch and it not needed.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c | 197
1 file changed
From: David Kershner david.kersh...@unisys.com
Move vbuschannel.h into visorbus.
Signed-off-by: David Kershner david.kersh...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../common-spar/include/channels/vbuschannel.h | 94 --
drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../staging/unisys/visorutil/memregion_direct.c| 30 --
1 file changed, 30 deletions(-)
diff --git a/drivers/staging
in case the kzalloc failed to allocate memory for the message header.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/visorbus.h | 5 +-
drivers/staging/unisys/visorbus/visorchipset.c | 67
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchannel.c | 6 +-
drivers/staging/unisys/visorutil/memregion.h| 1 -
drivers/staging
From: Erik Arfvidson erik.arfvid...@unisys.com
This patch reorganizes, aligns, and corrects grammar mistakes on
comments.
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../common-spar/include/channels/iochannel.h | 240
From: David Kershner david.kersh...@unisys.com
Get rid of unused values in the enum.
Signed-off-by: David Kershner david.kersh...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/diagchannel.h | 32 +++-
1 file changed
that here.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/visorbus.h | 2 ++
drivers/staging/unisys/visorbus/visorchannel.c | 9 +
2 files changed, 11 insertions(+)
diff --git a/drivers/staging
From: Prarit Bhargava pra...@redhat.com
These shouldn't be defined in the code and can be replaced with the
standard bool, true, and false usage that the kernel uses.
Signed-off-by: Prarit Bhargava pra...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging
From: Don Zickus dzic...@redhat.com
This header is needed by other drivers and should be in a global namespace.
In addition, functionally it is part of visorbus, so roll the contents
into that header file.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro
args for now.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/visorbus.h | 1 -
drivers/staging/unisys/visorbus/visorbus_main.c | 219 +++-
2 files changed, 65 insertions(+), 155
From: Don Zickus dzic...@redhat.com
Turn visorbus.h into a public header that all visor* drivers will
include.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/visorbus.h | 214
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/visorbus.h | 1 -
drivers/staging/unisys/visorbus/visorchannel.c | 7 ---
2 files changed, 8
as the bus_responder but with dev_info_clear.
Both of the above quirks should be verified that things still work
correctly.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c| 10 +--
drivers
From: Erik Arfvidson erik.arfvid...@unisys.com
This patch removes charqueue.[ch] since it no longer called
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorutil/Makefile| 2 +-
drivers/staging
Later patches will require PCI and ACPI support in the kernel, so add these
features to the Kconfig.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/Kconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/unisys/Kconfig b/drivers/staging
From: David Kershner david.kersh...@unisys.com
The function find_visor_device_by_channel was not used, get rid of it.
Signed-off-by: David Kershner david.kersh...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c | 16
-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchipset.c | 35 +-
1 file changed, 12 insertions(+), 23 deletions(-)
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c
b/drivers/staging/unisys/visorbus/visorchipset.c
index
directly
instead of the extern variable in timksmod.h.
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/periodic_work.h | 1 +
drivers/staging/unisys/include/timskmod.h | 94
From: Erik Arfvidson erik.arfvid...@unisys.com
This patch remove unused controlvmchannel.h #defines
Signed-off-by: Erik Arfvidson erik.arfvid...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../staging/unisys/common-spar/include/channels/controlvmchannel.h | 5
From: Jes Sorensen jes.soren...@redhat.com
This fixes the build breakage caused by recent removal of -I flags in
the Makefiles
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/guestlinuxdebug.h | 1
the functions static.
Signed-off-by: Prarit Bhargava pra...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/Makefile | 2 +-
.../staging/unisys/visorbus/devmajorminor_attr.c | 192 -
.../staging/unisys/visorbus
From: David Kershner david.kersh...@unisys.com
serialloopbacktest is no longer supported. Get rid of it.
Signed-off-by: David Kershner david.kersh...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c | 7 ---
drivers
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchipset.c | 18 --
1 file changed, 18 deletions(-)
diff --git a/drivers/staging/unisys
for_each_child_device or
something. Add that to the TODO list. :-)
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchipset.c | 121 +
1 file changed, 63 insertions(+), 58
From: Don Zickus dzic...@redhat.com
Now that the conversion to struct visor_device is complete,
remove all the cruft left behind.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c| 53
From: David Kershner david.kersh...@unisys.com
Signed-off-by: David Kershner david.kersh...@unisys.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/sparstop.h | 29 -
1 file changed, 29 deletions(-)
delete mode 100644
-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorchannel.c | 36 ++
drivers/staging/unisys/visorutil/memregion.h | 3 +-
.../staging/unisys/visorutil/memregion_direct.c| 9 ++
3 files changed, 20 insertions(+), 28 deletions
dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorbus/visorbus_main.c| 125 +
drivers/staging/unisys/visorbus/visorbus_private.h | 12 +-
2 files changed, 35 insertions(+), 102 deletions(-)
diff --git a/drivers
From: Don Zickus dzic...@redhat.com
Another conversion of the init/exit routines with various other
functions used during init thrown in here.
Most of the conversion is mechanical.
Signed-off-by: Don Zickus dzic...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers
This module is being removed completely, because it contained wrapper functions
and utility functions that were used in virtpci and virthba. Since these two
drivers are being rewritten to not use these wrappers and utilities, uislib
needs to go.
Signed-off-by: Benjamin Romer benjamin.ro
The virthba driver is being rewritten and will be renamed to visorhba, so delete
the old driver from the source tree.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
Reviewed-by: Don Zickus dzic...@redhat.com
Reviewed-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/unisys
visorbus. This driver will consolidate the functions needed to support new
virtual device drivers, so the functions in visorchannel and visorutil will be
moved over to visorbus. Additionally, the message handling done in visorchipset
will make its way here too.
Benjamin Romer (3):
staging: unisys
The s-Par drivers used to be out-of-tree, so they needed a parameter to
let them know we were going into a dump. This patch removes that code
and uses the built-in kernel function instead.
Reviewed-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
= UISLIB_PC_UISUTILS_C
Reviewed-by: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/include/guestlinuxdebug.h | 40
1 file changed, 20 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/unisys/include
Fix the CamelCase global names:
ControlVm_Pending_Msg = controlvm_pending_msg
ControlVm_Pending_Msg_Valid = controlvm_pending_msg_valid
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset
Rename the CamelCase global:
MajorDev = major_dev
Update references to use the corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers
Fix these CamelCased names:
BusDev_Server_Notifiers = busdev_server_notifiers
BusDev_Client_Notifiers = busdev_client_notifiers
Update references to use the fixed names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 40
Fix the CamelCase-named structure clearing functions in
visorchipset_main.c:
busInfo_clear = bus_info_clear
devInfo_clear = dev_info_clear
Update references to use the fixed names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset
Rename this variable:
Visorchipset_cache_buffers_in_use = visorchipset_cache_buffers_in_use
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 6 +++---
1 file changed, 3 insertions
Fix the variable's CamelCase name:
g_DeviceChangeStatePacket = g_devicechangestate_packet
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 10 +-
1 file changed, 5
Convert all references to PARSER_CONTEXT into struct parser_context.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 28 +++---
drivers/staging/unisys/visorchipset/parser.h | 22 -
.../unisys
The block of code is old and #if 0'd out, so just remove it.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 29 --
1 file changed, 29 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset
Rename CamelCased local variable names:
localSavedCrashMsgOffset = crash_msg_offset
localSavedCrashMsgCount = crash_msg_count
Update references to use the new names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
Just fix the spacing around the logical or operator in this function.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorchipset
Fix CamelCase names:
busNo = bus_no
pBusInfo = bus_info
Update all references to use the corrected names, and remove the goto
statements and labels entirely.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 17
Rename the CamelCase global structure
ControlVm_payload_info = controlvm_payload_info
Move the declaration to the struct definition, and update all references
to use the new name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset
Simplify the code a little and shorten the indentation levels by
reversing the check for time_after_eq().
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 22 --
1 file changed, 12 insertions(+), 10 deletions
Fix the CamelCase label:
Away = cleanup
Update all references to use the fixed name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers
Fix the CamelCase local variables:
payloadOffset = payload_offset
payloadBytes = payload_bytes
Update all references to use the new names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 10 +-
1 file changed, 5
And update all references to use the corrected names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 36 +++---
1 file changed, 18 insertions(+), 18 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset
All of the proc code using easyproc and procobjecttree support is gone,
so we can remove it from the source entirely.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorutil/Makefile | 3 +-
drivers/staging/unisys/visorutil/easyproc.c | 356
Rename the CamelCase variable:
Visorchipset_platform_device = visorchipset_platform_device
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
1 file changed
Rename the CamelCase local
toolAction = tool_action
In both of the sysfs functions for exposing this value, and update all
references to use the corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12
the definition and global structure
declaration.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 23 +++---
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset
Fix the CamelCase variable names:
busNo = bus_no
devNo = dev_no
pDevInfo = dev_info
Update all the references to the updated names, then remove the goto and
its label.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../staging/unisys/visorchipset/visorchipset_main.c | 21
Rename the CamelCase parameters:
busNo = bus_no
devNo = dev_no
needResponse = need_response
And the local variables
pBusInfo = bus_info
pDevInfo = dev_info
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset
Fix the CamelCased local variable
textId = text_id
In both of the textid sysfs functions, and update all references to use the
corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1 file changed
First, fix the CamelCase names:
busNo = bus_no
pBusInfo = bus_info
Update all references to use the corrected names, and remove the messy
goto and label.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 49
Fix CamelCase names:
parser_init_byteStream = parser_init_byte_stream
isLocal = local
tryAgain = retry
Update references to the changed function name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c| 4 ++--
drivers/staging
This set of patches cleans up most of the remaining checkpatch.pl issues in the
unisys tree.
Benjamin Romer (69):
staging: unisys: clean up NULL comparisons in visorchannel_funcs.c
staging: unisys: cleanup NULL comparisons in virthba.c
staging: unisys: fix CamelCase in virthba_probe
Fix the CamelCase names:
Parahotplug_request_list = parahotplug_request_list
Parahotplug_request_list_lock = parahotplug_request_list_lock
Update references to use the corrected names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c
Take out extra braces from if statements with only a single line of code
inside.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/uislib/uislib.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/unisys
Fix the CamelCase parameter names in the set of responder functions:
cmdId = cmd_id
busNo = bus_no
devNo = dev_no
responseState = response_state
Update all references to use the fixed names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c
Fix up the line spacing around braces in visorchipset_main.c.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b
Fix all of the bracing mistakes by adding missing braces, and removing
unnecessary braces.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../staging/unisys/visorchipset/visorchipset_main.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git
Fix the CamelCase global variable in visorchipset_main.c:
Most_recent_message_jiffies = most_recent_message_jiffies
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 8
Fix the name and references:
Poll_jiffies = poll_jiffies
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 22 ++
1 file changed, 10 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset
Fix this CamelCase function name:
parser_byteStream_get = parser_byte_stream_get
Update all references to use the fixed name. Fix the spacing in the typecast.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 5 ++---
drivers/staging
Fix camelcase name:
Controlvm_Payload_Bytes_Buffered = controlvm_payload_bytes_buffered
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/parser.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/unisys
Just remove the NULL from the check and test the pointer directly.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorchipset
Change pointer comparisons to NULL to just use the pointer directly.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/uislib/uislib.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/unisys/uislib/uislib.c
b
Fix the CamelCase names:
busNo = bus_no
devNo = dev_no
pDevInfo = dev_info
Update all references to use the new names, and remove the goto and
label entirely.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 27
Fix the CamelCase parameter name
msgHdr = msg_hdr
In all of the controlvm functions in visorchipset_main.c. Update all references
to use the corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 24
First, fix all CamelCase names:
busNo = bus_no
pBusInfo = bus_info
Away = cleanup
Fix the kzalloc() call so it uses the variable name rather than the
type, and update all of the references to the renamed variables and
labels.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
Fix CamelCase names:
localCrashCreateBusMsg = local_crash_bus_msg
localCrashCreateDevMsg = local_crash_dev_msg
localSavedCrashMsgOffset = local_crash_msg_offset
localSavedCrashMsgCount = local_crash_msg_count
Away = cleanup
Update all references to use the fixed names.
Signed-off-by: Benjamin
the 80 column limit.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging
line.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
.../unisys/visorchipset/visorchipset_main.c| 117 +++--
1 file changed, 64 insertions(+), 53 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b/drivers/staging/unisys
Rename the local CamelCase variable
efiSparIndication = efi_spar_indication
Update all references to use the corrected name.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 12 ++--
1 file changed, 6 insertions(+), 6
Fix CamelCase names:
g_diagpoolBusNo = g_diagpool_bus_no
g_diagpoolDevNo = g_diagpool_dev_no
Update all references to the changed names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
1 file changed, 8
Fix the CamelCase global names:
Putfile_buffer_list_pool = putfile_buffer_list_pool
Putfile_buffer_list_pool_name = putfile_buffer_list_pool_name
Putfile_request_list = putfile_request_list
Update all references to use the corrected names.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
Rename the label in both places:
Away = cleanup
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c
b
Take out the variable used to compute a comparison against NULL, and
just use the parameter directly.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/virtpci/virtpci.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/unisys
There is an unused variable in the visorchipset_bus_info structure that
used to be for a proc entry, so remove it, and the code that referenced
it. We don't need it anymore.
Signed-off-by: Benjamin Romer benjamin.ro...@unisys.com
---
drivers/staging/unisys/visorchipset/visorchipset.h | 1
401 - 500 of 1030 matches
Mail list logo