staging: unisys: refactor visor_signal_remove()
authorBenjamin Romer <benjamin.romer@unisys.com>
Thu, 23 Oct 2014 18:30:06 +0000 (14:30 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 27 Oct 2014 02:33:09 +0000 (10:33 +0800)
Rename visor_signal_remove() to spar_signal_remove() and fix CamelCase
parameter names:

pChannel => ch
Queue => queue
pSignal => sig

Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/channels/channel.c
drivers/staging/unisys/channels/chanstub.c
drivers/staging/unisys/common-spar/include/channels/channel.h
drivers/staging/unisys/uislib/uisqueue.c

index 4737b77..0b9a7cf 100644 (file)
@@ -102,14 +102,13 @@ EXPORT_SYMBOL_GPL(spar_signal_insert);
  * 1 if the removal succeeds, 0 if the queue was empty.
  */
 unsigned char
-visor_signal_remove(struct channel_header __iomem *pChannel, u32 Queue,
-                   void *pSignal)
+spar_signal_remove(struct channel_header __iomem *ch, u32 queue, void *sig)
 {
        void __iomem *psource;
        unsigned int head, tail;
        struct signal_queue_header __iomem *pqhdr =
-           (struct signal_queue_header __iomem *) ((char __iomem *) pChannel +
-                                   readq(&pChannel->ch_space_offset)) + Queue;
+           (struct signal_queue_header __iomem *) ((char __iomem *) ch +
+                                   readq(&ch->ch_space_offset)) + queue;
 
        /* capture current head and tail */
        head = readl(&pqhdr->head);
@@ -127,7 +126,7 @@ visor_signal_remove(struct channel_header __iomem *pChannel, u32 Queue,
        /* copy signal from tail location to the area pointed to by pSignal */
        psource = (char __iomem *) pqhdr + readq(&pqhdr->sig_base_offset) +
                (tail * readl(&pqhdr->signal_size));
-       memcpy_fromio(pSignal, psource, readl(&pqhdr->signal_size));
+       memcpy_fromio(sig, psource, readl(&pqhdr->signal_size));
 
        mb(); /* channel synch */
        writel(tail, &pqhdr->tail);
@@ -136,7 +135,7 @@ visor_signal_remove(struct channel_header __iomem *pChannel, u32 Queue,
               &pqhdr->num_received);
        return 1;
 }
-EXPORT_SYMBOL_GPL(visor_signal_remove);
+EXPORT_SYMBOL_GPL(spar_signal_remove);
 
 /*
  * Routine Description:
index fae9ab3..b6fd126 100644 (file)
@@ -61,7 +61,7 @@ SignalRemove_withLock(struct channel_header __iomem *pChannel, u32 Queue,
        unsigned char result;
 
        spin_lock(lock);
-       result = visor_signal_remove(pChannel, Queue, pSignal);
+       result = spar_signal_remove(pChannel, Queue, pSignal);
        spin_unlock(lock);
        return result;
 }
index 16dd7a3..15c9ceb 100644 (file)
@@ -528,8 +528,8 @@ unsigned char spar_signal_insert(struct channel_header __iomem *ch, u32 queue,
 * empty.
 */
 
-unsigned char visor_signal_remove(struct channel_header __iomem *pChannel,
-                                 u32 Queue, void *pSignal);
+unsigned char spar_signal_remove(struct channel_header __iomem *ch, u32 queue,
+                                void *sig);
 
 /*
 * Routine Description:
index 66da828..f9f8442 100644 (file)
@@ -125,7 +125,7 @@ int
 uisqueue_get_cmdrsp(struct uisqueue_info *queueinfo,
                    void *cmdrsp, unsigned int whichqueue)
 {
-       if (!visor_signal_remove(queueinfo->chan, whichqueue, cmdrsp))
+       if (!spar_signal_remove(queueinfo->chan, whichqueue, cmdrsp))
                return 0;
 
        queueinfo->packets_received++;