staging: unisys: Finally remove the last remnants of memregion
authorJes Sorensen <Jes.Sorensen@redhat.com>
Tue, 5 May 2015 22:37:00 +0000 (18:37 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 8 May 2015 13:27:30 +0000 (15:27 +0200)
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/include/visorbus.h
drivers/staging/unisys/visorbus/visorchannel.c
drivers/staging/unisys/visorchannel/globals.h
drivers/staging/unisys/visorutil/Makefile
drivers/staging/unisys/visorutil/memregion.h [deleted file]
drivers/staging/unisys/visorutil/memregion_direct.c [deleted file]

index e1c666234b1f5521804baa02a33f5bf8c2796653..87ffff4657fe8a10bf58c5c21a5c5223cea2d867 100644 (file)
@@ -36,7 +36,6 @@
 
 #include "periodic_work.h"
 #include "channel.h"
-#include "memregion.h"
 
 #ifndef HOSTADDRESS
 #define HOSTADDRESS u64
index 564d32521a4b9eca8748a65de27a97a4a6a59af2..ed652a08e4e1f4b755461ee87ba085b4202c9216 100644 (file)
 
 /*
  *  This provides Supervisor channel communication primitives, which are
- *  independent of the mechanism used to access the channel data.  All channel
- *  data is accessed using the memregion abstraction.  (memregion has both
- *  a CM2 implementation and a direct memory implementation.)
+ *  independent of the mechanism used to access the channel data.
  */
 
-#include "memregion.h"
 #include "version.h"
 #include "visorbus.h"
 #include <linux/uuid.h>
@@ -30,7 +27,9 @@
 #define MYDRVNAME "visorchannel"
 
 struct visorchannel {
-       struct memregion memregion;     /* from visor_memregion_create() */
+       HOSTADDRESS physaddr;
+       ulong nbytes;
+       void __iomem *mapped;
        struct channel_header chan_hdr;
        uuid_le guid;
        ulong size;
@@ -69,14 +68,14 @@ visorchannel_create_guts(HOSTADDRESS physaddr, ulong channel_bytes,
        if (!request_mem_region(physaddr, size, MYDRVNAME))
                goto cleanup;
 
-       channel->memregion.mapped = ioremap_cache(physaddr, size);
-       if (!channel->memregion.mapped) {
+       channel->mapped = ioremap_cache(physaddr, size);
+       if (!channel->mapped) {
                release_mem_region(physaddr, size);
                goto cleanup;
        }
 
-       channel->memregion.physaddr = physaddr;
-       channel->memregion.nbytes = size;
+       channel->physaddr = physaddr;
+       channel->nbytes = size;
 
        err = visorchannel_read(channel, 0, &channel->chan_hdr,
                                sizeof(struct channel_header));
@@ -89,22 +88,19 @@ visorchannel_create_guts(HOSTADDRESS physaddr, ulong channel_bytes,
        if (uuid_le_cmp(guid, NULL_UUID_LE) == 0)
                guid = channel->chan_hdr.chtype;
 
-       iounmap(channel->memregion.mapped);
-       release_mem_region(channel->memregion.physaddr,
-                          channel->memregion.nbytes);
-       channel->memregion.mapped = NULL;
-       if (!request_mem_region(channel->memregion.physaddr, channel_bytes,
-                               MYDRVNAME))
+       iounmap(channel->mapped);
+       release_mem_region(channel->physaddr, channel->nbytes);
+       channel->mapped = NULL;
+       if (!request_mem_region(channel->physaddr, channel_bytes, MYDRVNAME))
                goto cleanup;
 
-       channel->memregion.mapped = ioremap_cache(channel->memregion.physaddr,
-                                                 channel_bytes);
-       if (!channel->memregion.mapped) {
-               release_mem_region(channel->memregion.physaddr, channel_bytes);
+       channel->mapped = ioremap_cache(channel->physaddr, channel_bytes);
+       if (!channel->mapped) {
+               release_mem_region(channel->physaddr, channel_bytes);
                goto cleanup;
        }
 
-       channel->memregion.nbytes = channel_bytes;
+       channel->nbytes = channel_bytes;
 
        channel->size = channel_bytes;
        channel->guid = guid;
@@ -137,10 +133,9 @@ visorchannel_destroy(struct visorchannel *channel)
 {
        if (!channel)
                return;
-       if (channel->memregion.mapped) {
-               iounmap(channel->memregion.mapped);
-               release_mem_region(channel->memregion.physaddr,
-                                  channel->memregion.nbytes);
+       if (channel->mapped) {
+               iounmap(channel->mapped);
+               release_mem_region(channel->physaddr, channel->nbytes);
        }
        kfree(channel);
 }
@@ -149,7 +144,7 @@ EXPORT_SYMBOL_GPL(visorchannel_destroy);
 HOSTADDRESS
 visorchannel_get_physaddr(struct visorchannel *channel)
 {
-       return channel->memregion.physaddr;
+       return channel->physaddr;
 }
 EXPORT_SYMBOL_GPL(visorchannel_get_physaddr);
 
@@ -200,10 +195,10 @@ int
 visorchannel_read(struct visorchannel *channel, ulong offset,
                  void *local, ulong nbytes)
 {
-       if (offset + nbytes > channel->memregion.nbytes)
+       if (offset + nbytes > channel->nbytes)
                return -EIO;
 
-       memcpy_fromio(local, channel->memregion.mapped + offset, nbytes);
+       memcpy_fromio(local, channel->mapped + offset, nbytes);
 
        return 0;
 }
@@ -216,7 +211,7 @@ visorchannel_write(struct visorchannel *channel, ulong offset,
        size_t chdr_size = sizeof(struct channel_header);
        size_t copy_size;
 
-       if (offset + nbytes > channel->memregion.nbytes)
+       if (offset + nbytes > channel->nbytes)
                return -EIO;
 
        if (offset < chdr_size) {
@@ -224,7 +219,7 @@ visorchannel_write(struct visorchannel *channel, ulong offset,
                memcpy(&channel->chan_hdr + offset, local, copy_size);
        }
 
-       memcpy_toio(channel->memregion.mapped + offset, local, nbytes);
+       memcpy_toio(channel->mapped + offset, local, nbytes);
 
        return 0;
 }
index 0ed8e1d8033a8d8ca6ae4a92fc3befa706183c2e..6530413958d43e49a046ac0b9c795b3f27afecd4 100644 (file)
@@ -19,7 +19,6 @@
 #define __VISORCHANNEL_GLOBALS_H__
 
 #include "timskmod.h"
-#include "memregion.h"
 #include "version.h"
 
 #define MYDRVNAME "visorchannel"
index d9ab5a36e3bf0b026274540d2f23f6617e0b471c..f29978772ca348225f416a0c0d7d5e8851f95ae3 100644 (file)
@@ -4,6 +4,6 @@
 
 obj-$(CONFIG_UNISYS_VISORUTIL) += visorutil.o
 
-visorutil-y := charqueue.o  periodic_work.o  memregion_direct.o visorkmodutils.o
+visorutil-y := charqueue.o periodic_work.o visorkmodutils.o
 
 ccflags-y += -Idrivers/staging/unisys/include
diff --git a/drivers/staging/unisys/visorutil/memregion.h b/drivers/staging/unisys/visorutil/memregion.h
deleted file mode 100644 (file)
index 62036cd..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-/* memregion.h
- *
- * Copyright (C) 2010 - 2013 UNISYS CORPORATION
- * All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or (at
- * your option) any later version.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
- * NON INFRINGEMENT.  See the GNU General Public License for more
- * details.
- */
-
-#ifndef __MEMREGION_H__
-#define __MEMREGION_H__
-
-#include "timskmod.h"
-
-/* struct memregion is an opaque structure to users.
- * Fields are declared only in the implementation .c files.
- */
-struct memregion {
-       HOSTADDRESS physaddr;
-       ulong nbytes;
-       void __iomem *mapped;
-};
-
-void memregion_dump(struct memregion *memregion, char *s,
-                   ulong off, ulong len, struct seq_file *seq);
-
-#endif
diff --git a/drivers/staging/unisys/visorutil/memregion_direct.c b/drivers/staging/unisys/visorutil/memregion_direct.c
deleted file mode 100644 (file)
index 818f6a8..0000000
+++ /dev/null
@@ -1,26 +0,0 @@
-/* memregion_direct.c
- *
- * Copyright (C) 2010 - 2013 UNISYS CORPORATION
- * All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or (at
- * your option) any later version.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
- * NON INFRINGEMENT.  See the GNU General Public License for more
- * details.
- */
-
-/*
- *  This is an implementation of memory regions that can be used to read/write
- *  channel memory (in main memory of the host system) from code running in
- *  a virtual partition.
- */
-#include "timskmod.h"
-#include "memregion.h"
-
-#define MYDRVNAME "memregion"