diff options
author | Ken Cox <jkc@redhat.com> | 2014-03-13 15:39:22 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-03-17 14:45:44 -0700 |
commit | bd5b9b32d2826a7bcd0b97e698f82e2060f422b9 (patch) | |
tree | 0ce507c1ef218336015e2c8811b9ecf6bd92092e /drivers/staging/unisys | |
parent | 92a9e2d6bdd382b10f3709c8cf172d50c7a00a01 (diff) |
Staging: unisys: visorchipset: Clean up sparse warnings in visorchipset code.
Clean up code to get rid of sparse warnings due to accessing I/O space. Also
declared functions and variables as static if they are only used locally.
Signed-off-by: Ken Cox <jkc@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/unisys')
-rw-r--r-- | drivers/staging/unisys/visorchipset/controlvm_direct.c | 1 | ||||
-rw-r--r-- | drivers/staging/unisys/visorchipset/file.c | 1 | ||||
-rw-r--r-- | drivers/staging/unisys/visorchipset/visorchipset_main.c | 16 |
3 files changed, 10 insertions, 8 deletions
diff --git a/drivers/staging/unisys/visorchipset/controlvm_direct.c b/drivers/staging/unisys/visorchipset/controlvm_direct.c index 7fbc5892bcbc..b911ea85c093 100644 --- a/drivers/staging/unisys/visorchipset/controlvm_direct.c +++ b/drivers/staging/unisys/visorchipset/controlvm_direct.c @@ -21,6 +21,7 @@ #include "globals.h" #include "uisutils.h" +#include "controlvm.h" #define CURRENT_FILE_PC VISOR_CHIPSET_PC_controlvm_direct_c diff --git a/drivers/staging/unisys/visorchipset/file.c b/drivers/staging/unisys/visorchipset/file.c index b0d28a2b5231..7e8bc9824457 100644 --- a/drivers/staging/unisys/visorchipset/file.c +++ b/drivers/staging/unisys/visorchipset/file.c @@ -24,6 +24,7 @@ #include <linux/mm.h> #include <linux/fs.h> #include "uisutils.h" +#include "file.h" #define CURRENT_FILE_PC VISOR_CHIPSET_PC_file_c diff --git a/drivers/staging/unisys/visorchipset/visorchipset_main.c b/drivers/staging/unisys/visorchipset/visorchipset_main.c index 0e404e89ae96..e58b136d65e6 100644 --- a/drivers/staging/unisys/visorchipset/visorchipset_main.c +++ b/drivers/staging/unisys/visorchipset/visorchipset_main.c @@ -52,9 +52,9 @@ * message, we switch back to fast polling mode. */ #define MIN_IDLE_SECONDS 10 -ulong Poll_jiffies = POLLJIFFIES_CONTROLVMCHANNEL_FAST; -ulong Most_recent_message_jiffies; /* when we got our last - * controlvm message */ +static ulong Poll_jiffies = POLLJIFFIES_CONTROLVMCHANNEL_FAST; +static ulong Most_recent_message_jiffies; /* when we got our last + * controlvm message */ static inline char * NONULLSTR(char *s) { @@ -72,7 +72,7 @@ static U8 chipset_events[MAX_CHIPSET_EVENTS] = { 0, 0 }; static struct delayed_work Periodic_controlvm_work; static struct workqueue_struct *Periodic_controlvm_workqueue; -DEFINE_SEMAPHORE(NotifierLock); +static DEFINE_SEMAPHORE(NotifierLock); typedef struct { CONTROLVM_MESSAGE message; @@ -215,7 +215,7 @@ static const struct file_operations proc_bootToTool_fops = { }; typedef struct { - U8 *ptr; /* pointer to base address of payload pool */ + U8 __iomem *ptr; /* pointer to base address of payload pool */ U64 offset; /* offset from beginning of controlvm * channel to beginning of payload * pool */ U32 bytes; /* number of bytes in payload pool */ @@ -324,7 +324,7 @@ struct putfile_request { int completion_status; }; -atomic_t Visorchipset_cache_buffers_in_use = ATOMIC_INIT(0); +static atomic_t Visorchipset_cache_buffers_in_use = ATOMIC_INIT(0); struct parahotplug_request { struct list_head list; @@ -1343,7 +1343,7 @@ static int initialize_controlvm_payload_info(HOSTADDRESS phys_addr, U64 offset, U32 bytes, CONTROLVM_PAYLOAD_INFO *info) { - U8 *payload = NULL; + U8 __iomem *payload = NULL; int rc = CONTROLVM_RESP_SUCCESS; if (info == NULL) { @@ -1677,7 +1677,7 @@ parahotplug_request_complete(int id, U16 active) /* * Enables or disables a PCI device by kicking off a udev script */ -void +static void parahotplug_process_message(CONTROLVM_MESSAGE *inmsg) { struct parahotplug_request *req; |