checks
with only justifiable exceptions.
Benjamin Romer (41):
staging: unisys: clean up enums in uisqueue.h
staging: unisys: clean up GUESTPART_MSG_TYPE typedef
staging: unisys: Clean up CamelCase in struct bus_info
staging: unisys: fix CamelCase in struct add_vbus_guestpart
staging
Clean up CamelCase names:
busNo => bus_no
devNo => dev_no
instGuid => inst_uuid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 4 ++--
drivers/staging/unisys/uislib/uislib.c| 18 +-
2 files changed, 11 insertions(+), 11 deletions
Fix CamelCase names:
Issue_VMCALL_IO_VISORSERIAL_ADDR => issue_vmcall_io_visorserial_addr
DiagChannelAddress => channel_addr
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/s
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/i
Clean up CamelCase names:
pSwitch => s
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisqueue.h
index 02cd
Fix CamelCase names:
chipset_driverInfo => chipset_driver_info
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uisutils.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/unisys/incl
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/i
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/i
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisutil
Clean up CamelCase names:
busNo => bus_no
deviceNo => device_no
devInstGuid => instance_uuid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 6 +++---
drivers/staging/unisys/uislib/uislib.c| 12 ++--
drivers/staging/unisys/virtpci/virtpc
Just cleaning up the extra blank line from here.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/timskmod.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/unisys/include/timskmod.h
b/drivers/staging/unisys/include/timskmod.h
index ee7e85f..0f59b3c 100644
Clean up CamelCase name:
NetworkPolicy => network_policy
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisqueue.h
in
Fix CamelCase names:
UisUtils_Registered_Services => uisutils_registered_services
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 2 +-
drivers/staging/unisys/uislib/uisutils.c | 10 +-
3 files changed
Fix the indentation around server_channel_init, so the parameters line
up nicely.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisutils.h
b/drivers/staging
Fix CamelCase names:
Issue_VMCALL_QUERY_GUEST_VIRTUAL_TIME_OFFSET =>
issue_vmcall_query_guest_virtual_time_offset
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/visorchipset/file.c | 2 +-
2 files changed, 2 insertions(+)
Fix CamelCase names:
Issue_VMCALL_CHANNEL_MISMATCH => issue_vmcall_channel_mismatch
ChannelName => chname
ItemName => item_name
SourceLineNumber => line_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 13 ++---
1 file changed, 6 inse
Switch GUESTPART_MSG_TYPE over to enum guestpart_msg_type.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include
Clean up CamelCase names:
issueInterruptIfEmpty => issue_irq_if_empty
interruptHandle => irq_handle
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueu
Fix CamelCase names:
switchTypeGuid => switch_uuid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uisutils.c | 10 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/unisys/incl
This function is not used anywhere, so remove it.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisqueue.h
index 77a2a73..95695d8
Clean up CamelCase names:
extPort => ext_port
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisqueue.h
index f4ba
Clean up CamelCase names:
Reserved1 => reserved1
Reserved2 => reserved2
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/i
Simply switch DRIVER_PC to enum driver_pc.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/guestlinuxdebug.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/guestlinuxdebug.h
b/drivers/staging/unisys/include
Fix CamelCase names:
BusDeviceInfo_Init => bus_device_info_init
pBusDeviceInfo => bus_device_info_ptr
deviceType => dev_type
driverName => driver_name
verTag => ver_tag
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/vbushelper.h| 26
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
instGuid => inst_uuid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 4 ++--
drivers/staging/unisys/uislib/uislib.c| 18 +-
2 files changed, 11 insertions(+), 11 deletions(-)
dif
Fix CamelCase names:
switchTypeGuid => switch_uuid
Server_Channel_Ok => server_channel_ok
channelBytes => channel_bytes
Server_Channel_Init => server_channel_init
clientStr => client_str
clientStrLen => client_str_len
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/
Just switch EVENT_PC over to enum event_pc.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/guestlinuxdebug.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/guestlinuxdebug.h
b/drivers/staging/unisys/include
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/i
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/unisys/include/uisu
Take out those doubled blank lines in guestlinuxdebug.h.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/guestlinuxdebug.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/unisys/include/guestlinuxdebug.h
b/drivers/staging/unisys/include/guestlinuxdebug.h
These functions contain CamelCase names but are not used. Delete the
prototypes from uisutils.h.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/unisys/include/uisutils.h
b/drivers/staging
Take out the doubled blank lines in uniklog.h, too.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uniklog.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uniklog.h
b/drivers/staging/unisys/include/uniklog.h
index 6178cc4..ecd1bdb 100644
Fix up CamelCase names:
busNo => bus_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 2 +-
drivers/staging/unisys/virtpci/virtpci.c | 6 +++---
3 files changed, 5 insertions(+), 5 deletions(-)
diff --gi
Fix CamelCase names:
UisnicControlChanFunc => uisnic_control_chan_func
UissdControlChanFunc => uissd_control_chan_func
VirtControlChanFunc => virt_control_chan_func
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 6 +++---
drivers/staging/unisys/uislib
The copyright character in vbushelper.h isn't UTF-8 - swap it out for a
good, old-fashioned (C).
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/vbushelper.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/include/vbushelper
Clean up CamelCase names:
uisqueue_InterlockedOr => uisqueue_interlocked_or
Target => tgt
Set => set
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 8
drivers/staging/unisys/uislib/uisqueue.c | 20 ++--
drivers/staging/unisy
Fix CamelCase names:
ReqHandlerDel => req_handler_del
switchTypeGuid => switch_uuid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uisutils.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/d
Fix CamelCase names:
Issue_VMCALL_UPDATE_PHYSICAL_TIME => issue_vmcall_update_physical_time
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/visorchipset/file.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --gi
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisqueue.h
Remove the MACARRAY typedef entirely and switch over to u8*[ETH_ALEN],
like everybody else seems to use.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisutils.h
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/unisys/include/uisu
Clean up CamelCase names:
issueInterruptIfEmpty => issue_irq_if_empty
interruptHandle => irq_handle
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 6 +++---
drivers/staging/unisys/uislib/uisqueue.c | 10 +-
2 files changed, 8 insertions(+), 8 del
.
Benjamin Romer (67):
staging: unisys: delete extra blank lines in periodic_work.h
staging: unisys: remove typedef for PERIODIC_WORK
staging: unisys: remove unused locking functions in timskmod.h
staging: unisys: remove stray blank line in timskmod.h
staging: unisys: fix spacing in timskmod.h
Take out those doubled blank lines in uisthread.h.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisthread.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/unisys/include/uisthread.h
b/drivers/staging/unisys/include/uisthread.h
index 2683777..aa86ade
Fix CamelCase names:
ReqHandlerFind => req_handler_find
switchTypeGuid => switch_uuid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 2 +-
drivers/staging/unisys/uislib/uisutils.c | 4 ++--
3 files chan
Clean up CamelCase names:
channelAddr => channel_addr
channelBytes => channel_bytes
channelTypeGuid => channel_uuid
devInstGuid => instance_uuid
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 12 +-
driv
ic => local_vnic
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 14 +++---
drivers/staging/unisys/uislib/uislib.c| 80 +++
2 files changed, 47 insertions(+), 47 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.
Fix CamelCase names:
Issue_VMCALL_IO_CONTROLVM_ADDR => issue_vmcall_io_controlvm_addr
ControlAddress => control_addr
ControlBytes => control_bytes
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 6 +++---
drivers/staging/unisys/vis
Fix CamelCase names:
ReqHandlerAdd => req_handler_add
Server_Channel_Ok => svr_channel_ok
channelBytes => channel_bytes
Server_Channel_Init => svr_channel_init
clientStr => client_str
ClientStrLen => client_str_len
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/inclu
Clean up CamelCase names:
busNo => bus_no
deviceCount => dev_count
busTypeGuid => bus_uuid
busInstGuid => instance_guid
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 8
drivers/staging/unisys/uislib/uislib.c| 8
drivers/st
Fix CamelCase names:
Issue_VMCALL_MEASUREMENT_DO_NOTHING =>
issue_vmcall_measurement_do_nothing
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/include/uisutils.h
Fix Camelcase name:
Issue_VMCALL_FATAL_BYE_BYE => issue_vmcall_fatal
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/include/uisutils.h
b/drivers/staging/unisys/incl
Fix CamelCase names:
Issue_VMCALL_IO_DIAG_ADDR => issue_vmcall_io_diag_addr
DiagChannelAddress => diag_channel_addr
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/i
Adds in some missing spaces in the max macro in timskmod.h.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/timskmod.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/include/timskmod.h
b/drivers/staging/unisys/include/timskmod.h
Fix CamelCase names:
switchTypeGuid => switch_uuid
clientStr => client_str
clientStrLent => client_str_len
chipset_driverInfo => chipset_driver_info
Server_Channel_Ok => server_channel_ok
Server_Channel_Init => server_channel_init
Signed-off-by: Benjamin Romer
---
drivers/stag
Fix CamelCase names:
busNo => bus_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/unisys/include/uisutils.h
b/driv
-by: Benjamin Romer
---
drivers/staging/unisys/channels/chanstub.c| 2 +-
drivers/staging/unisys/include/timskmod.h | 2 +-
drivers/staging/unisys/include/timskmodutils.h| 77 ---
drivers/staging/unisys/virtpci/virtpci.c | 2 +-
drivers/staging
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/i
Fix CamelCase names:
busNo => bus_no
instGuid => inst_uuid
channelAddr => channel_addr
nChannelBytes => n_channel_bytes
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 4 ++--
drivers/staging/unisys/uislib/uislib.c| 23 ---
2 f
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 +-
drivers/staging/unisys/uislib/uislib.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/unisys/i
Fix up CamelCase names:
busCount => bus_count
switch_count => switch_count
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/
Convert all references to the typedef to struct req_handler_info.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 14 +++---
drivers/staging/unisys/uislib/uislib.c| 2 +-
drivers/staging/unisys/uislib/uisutils.c | 18 +-
3 files
not use periodic_work as the parameter's name.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/periodic_work.h | 20 ++--
drivers/staging/unisys/visorutil/periodic_work.c | 137 +++
2 files changed, 76 insertions(+), 81 deletions(-)
diff --git a/dr
Delete a block of unused locking functions from timskmod.h. These
functions generated a lot of checkpatch warnings and were not being used
anyhow.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/timskmod.h | 88 ---
1 file changed, 88 deletions
Remove all unnecesary blank lines after braces in uisqueue.h.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include
Fix CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisqueue.h
Clean up CamelCase names:
busNo => bus_no
devNo => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uisqueue.h
b/drivers/staging/unisys/include/uisq
Clean up CamelCase names:
guestHandle => guest_handle
recvBusInterruptHandle => recv_bus_irq_handle
useG2GCopy => use_g2g_copy
Filler => filler
busNo => bus_no
dev_no => dev_no
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 12 ++--
From: Benjamin Romer
Remove the extra blank lines from before the first comment.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/periodic_work.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/unisys/include/periodic_work.h
b/drivers/staging/unisys
Fix all checkpatch.pl strict checks and warnings. The two typedefs were
removed and the doubled blank lines were corrected.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/guestlinuxdebug.h | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers
Remove a few double blank lines from uisthread.h to clean up all of the
checkpatch.pl output from this file.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisthread.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/unisys/include/uisthread.h
b/drivers
patches when those
structures are corrected.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 4 +-
drivers/staging/unisys/include/uisutils.h | 147 +++--
drivers/staging/unisys/uislib/uislib.c | 38 +++---
drivers
Fix all checkpatch.pl strict errors and checks. It was only a couple of
doubled blank lines.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uniklog.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/unisys/include/uniklog.h
b/drivers/staging/unisys/include
Eliminate all checkpatch.pl strict warnings and checks from timskmod.h. Some of
these warnings were generated by a block of unused locking macros, so the
entire set of macros were removed.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/timskmod.h | 91
-by: Benjamin Romer
---
drivers/staging/unisys/channels/chanstub.c| 2 +-
drivers/staging/unisys/include/timskmod.h | 2 +-
drivers/staging/unisys/include/timskmodutils.h| 77 ---
drivers/staging/unisys/virtpci/virtpci.c | 2 +-
drivers/staging
Fix many of the checkpatch.pl strict warnings and checks generated. The
remaining strict checks require large-scale refactoring of structures, so
these will be fixed in subequent patches.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisqueue.h | 54
-by: Benjamin Romer
---
drivers/staging/unisys/include/vbushelper.h| 24 +++---
drivers/staging/unisys/uislib/uisutils.c | 8
drivers/staging/unisys/virtpci/virtpci.c | 12 +--
.../unisys/visorchipset/visorchipset_main.c| 8
patches.
Benjamin Romer (9):
staging: unisys: clean up periodic_work.c and periodic_work.h
staging: unisys: clean up timskmod.h
staging: unisys: remove timskmodutils.h
staging: unisys: clean up uisqueue.h
staging: unisys: remove blank lines from uisthread.h
staging: unisys: clean up
Eliminate all checkpatch.pl warnings and checks from these files.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/periodic_work.h | 25 ++--
drivers/staging/unisys/visorutil/periodic_work.c | 155 +++
2 files changed, 83 insertions(+), 97 deletions
since a structure is being copied, not an array, and it
fixes the error message printout later in the function that was using the UUID
value in the channel, rather than the value copied from the channel, which
causes a misleading error to be reported.
Signed-off-by: Benjamin Romer
---
drivers
From: Benjamin Romer
There are several else clauses that aren't necessary listed by checkpatch.pl in
visorchipset_main.c. Remove these.
Signed-off-by: Benjamin Romer
---
.../unisys/visorchipset/visorchipset_main.c| 26 +-
1 file changed, 10 insertions(+
This patch series corrects checkpatch.pl errors in the visorchipset directory.
With these changes, all files in the directory should pass without errors, and
only justifiable warnings should be generated.
Benjamin Romer (4):
staging: unisys: fix spacing in file.c
staging: unisys: fix spacing
From: Benjamin Romer
There's a missing blank line between code and declarations in file.c.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/file.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/unisys/visorchipset/file.c
b/drivers/staging/u
From: Benjamin Romer
Fix the two spacing errors in parser.c.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/parser.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/unisys/visorchipset/parser.c
b/drivers/staging/unisys/visorchipset/parser.c
index
From: Benjamin Romer
Fix the missing blank lines between declarations and code in
visorchipset_main.c.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorchipset/visorchipset_main.c | 16
1 file changed, 16 insertions(+)
diff --git a/drivers/staging/unisys
The function uisutil_copy_fragsinfo_from_skb() in uisutils.c is messy and
over-indented. This patch fixes the function's indentation so it is cleaner.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uisutils.c | 48 +---
1 file changed, 25 inser
This patch series fixes all checkpatch.pl errors in the uislib directory. After
this series is applied, all files should pass without anything but justifiable
warnings.
Benjamin Romer (5):
staging: unisys: fix spacing in uislib.c
staging: unisys: simplify destroy_bus() in uislib.c
staging
Fix the missing blank line between code and declarations in uisthread.c.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uisthread.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/unisys/uislib/uisthread.c
b/drivers/staging/unisys/uislib/uisthread.c
index
Fix the line spacing between code and local variables in uislib.c.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uislib.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/staging/unisys/uislib/uislib.c
b/drivers/staging/unisys/uislib/uislib.c
index 4cb3487
Remove the extraneous return from the uislib module exit function.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uislib.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/unisys/uislib/uislib.c
b/drivers/staging/unisys/uislib/uislib.c
index 17cfc21..8b89fe3
The destroy_bus() function is heavily indented and many lines extend past the
right margin. This patch simplifies the function without changing its effects.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/uislib/uislib.c | 84 ++
1 file changed, 44
The comments for the large union CONTROLVM_MESSAGE_PACKET were inconsistently
placed and one extended past the 80 char limit. These are cleaned up.
Signed-off-by: Benjamin Romer
---
.../include/channels/controlvmchannel.h| 41 --
1 file changed, 30 insertions
Remove the do blocks around several macros and fix spacing and trailing
semicolons.
Signed-off-by: Benjamin Romer
---
.../unisys/common-spar/include/channels/iochannel.h| 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/unisys/common-spar
The CHANNEL_*_MISMATCH error message macros should not be inside of do blocks.
Signed-off-by: Benjamin Romer
---
.../staging/unisys/common-spar/include/channels/channel.h| 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/unisys/common-spar
This patch series finishes clean-up in the common-spar directory tree, so that
all files pass through checkpatch.pl cleanly, or with only justifiable warnings
remaining, like over-80-character lines for error messages.
Benjamin Romer (4):
staging: unisys: remove do{} while(0) in macros in
Two functions were missing space between declarations and code.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/common-spar/include/channels/channel.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/unisys/common-spar/include/channels/channel.h
b/drivers/staging
Two functions were missing space between declarations and code.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/common-spar/include/channels/channel.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/unisys/common-spar/include/channels/channel.h
b/drivers/staging
Correct a couple of missing blank lines in uisutils.h.
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/include/uisutils.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/unisys/include/uisutils.h
b/drivers/staging/unisys/include/uisutils.h
index a1c193c..f1a1b0d
in a future patchset. The remaining
typedefs in the headers are all for enumerations or structures specific to
s-Par.
Benjamin Romer (14):
staging: unisys: get rid of unused VMMIO types
staging: unisys: fix formatting in timskmod.h
staging: unisys: get rid of semaphore macros
staging
The comments for the large union CONTROLVM_MESSAGE_PACKET were inconsistently
placed and one extended past the 80 char limit. These are cleaned up.
Signed-off-by: Benjamin Romer
---
.../include/channels/controlvmchannel.h| 41 --
1 file changed, 30 insertions
901 - 1000 of 1097 matches
Mail list logo