staging: unisys: fix sig_read_data and sig_read_data functions
authorPrarit Bhargava <prarit@redhat.com>
Tue, 5 May 2015 22:36:19 +0000 (18:36 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 8 May 2015 13:26:01 +0000 (15:26 +0200)
The sig_read_data() and sig_write_data() functions are involved in 2 steps
of calls.  They really don't need to be and this makes for much simpler
code.

Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visorbus/visorchannel.c

index 30bedcc7241568e4daf1f67c65c61ea4c9fe5c43..d7ddc3af4638891c6589b516a29807686664ea29 100644 (file)
@@ -318,42 +318,38 @@ sig_read_header(struct visorchannel *channel, u32 queue,
        return TRUE;
 }
 
-static BOOL
-sig_do_data(struct visorchannel *channel, u32 queue,
-           struct signal_queue_header *sig_hdr, u32 slot, void *data,
-           BOOL is_write)
+static inline BOOL
+sig_read_data(struct visorchannel *channel, u32 queue,
+             struct signal_queue_header *sig_hdr, u32 slot, void *data)
 {
        int err;
        int signal_data_offset = SIG_DATA_OFFSET(&channel->chan_hdr, queue,
                                                 sig_hdr, slot);
-       if (is_write) {
-               err = visor_memregion_write(channel->memregion,
-                                           signal_data_offset,
-                                           data, sig_hdr->signal_size);
-               if (err)
-                       return FALSE;
-       } else {
-               err = visor_memregion_read(channel->memregion,
-                                          signal_data_offset,
-                                          data, sig_hdr->signal_size);
-               if (err)
-                       return FALSE;
-       }
-       return TRUE;
-}
 
-static inline BOOL
-sig_read_data(struct visorchannel *channel, u32 queue,
-             struct signal_queue_header *sig_hdr, u32 slot, void *data)
-{
-       return sig_do_data(channel, queue, sig_hdr, slot, data, FALSE);
+       err = visor_memregion_read(channel->memregion,
+                                  signal_data_offset,
+                                  data, sig_hdr->signal_size);
+       if (err)
+               return FALSE;
+
+       return TRUE;
 }
 
 static inline BOOL
 sig_write_data(struct visorchannel *channel, u32 queue,
               struct signal_queue_header *sig_hdr, u32 slot, void *data)
 {
-       return sig_do_data(channel, queue, sig_hdr, slot, data, TRUE);
+       int err;
+       int signal_data_offset = SIG_DATA_OFFSET(&channel->chan_hdr, queue,
+                                                sig_hdr, slot);
+
+       err = visor_memregion_write(channel->memregion,
+                                   signal_data_offset,
+                                   data, sig_hdr->signal_size);
+       if (err)
+               return FALSE;
+
+       return TRUE;
 }
 
 static BOOL