of: Provide a function to request and map memory
authorMatthias Brugger <matthias.bgg@gmail.com>
Fri, 18 Jul 2014 09:36:39 +0000 (11:36 +0200)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 23 Jul 2014 10:02:30 +0000 (12:02 +0200)
A call to of_iomap does not request the memory region. This patch adds the
function of_io_request_and_map which requests the memory region before
mapping it.

Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
Suggested-by: Thomas Gleixner <tglx@linutronix.de>
Suggested-by: Rob Herring <robh@kernel.org>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
drivers/of/address.c
include/linux/io.h
include/linux/of_address.h
lib/devres.c

index 5edfcb0da37d9e036dba3f0e9d6764dcd9bc28ef..e3718250d66e4f2b5fcb13c9936b7e28615440ec 100644 (file)
@@ -702,6 +702,42 @@ void __iomem *of_iomap(struct device_node *np, int index)
 }
 EXPORT_SYMBOL(of_iomap);
 
+/*
+ * of_io_request_and_map - Requests a resource and maps the memory mapped IO
+ *                        for a given device_node
+ * @device:    the device whose io range will be mapped
+ * @index:     index of the io range
+ * @name:      name of the resource
+ *
+ * Returns a pointer to the requested and mapped memory or an ERR_PTR() encoded
+ * error code on failure. Usage example:
+ *
+ *     base = of_io_request_and_map(node, 0, "foo");
+ *     if (IS_ERR(base))
+ *             return PTR_ERR(base);
+ */
+void __iomem *of_io_request_and_map(struct device_node *np, int index,
+                                       char *name)
+{
+       struct resource res;
+       void __iomem *mem;
+
+       if (of_address_to_resource(np, index, &res))
+               return IOMEM_ERR_PTR(-EINVAL);
+
+       if (!request_mem_region(res.start, resource_size(&res), name))
+               return IOMEM_ERR_PTR(-EBUSY);
+
+       mem = ioremap(res.start, resource_size(&res));
+       if (!mem) {
+               release_mem_region(res.start, resource_size(&res));
+               return IOMEM_ERR_PTR(-ENOMEM);
+       }
+
+       return mem;
+}
+EXPORT_SYMBOL(of_io_request_and_map);
+
 /**
  * of_dma_get_range - Get DMA range info
  * @np:                device node to get DMA range info
index b76e6e5458064551d53ae527281a8a353e9f6301..d5fc9b8d8b03c1add6747924f406db6649c9d232 100644 (file)
@@ -58,6 +58,8 @@ static inline void devm_ioport_unmap(struct device *dev, void __iomem *addr)
 }
 #endif
 
+#define IOMEM_ERR_PTR(err) (__force void __iomem *)ERR_PTR(err)
+
 void __iomem *devm_ioremap(struct device *dev, resource_size_t offset,
                            unsigned long size);
 void __iomem *devm_ioremap_nocache(struct device *dev, resource_size_t offset,
index c13b8782a4ebeb4e2ee2cc3b269cba43a345eeb4..fb7b7221e06309b8cbde7f4b2f996aeff4f77f8e 100644 (file)
@@ -109,7 +109,12 @@ static inline bool of_dma_is_coherent(struct device_node *np)
 extern int of_address_to_resource(struct device_node *dev, int index,
                                  struct resource *r);
 void __iomem *of_iomap(struct device_node *node, int index);
+void __iomem *of_io_request_and_map(struct device_node *device,
+                                       int index, char *name);
 #else
+
+#include <linux/io.h>
+
 static inline int of_address_to_resource(struct device_node *dev, int index,
                                         struct resource *r)
 {
@@ -120,6 +125,12 @@ static inline void __iomem *of_iomap(struct device_node *device, int index)
 {
        return NULL;
 }
+
+static inline void __iomem *of_io_request_and_map(struct device_node *device,
+                                       int index, char *name)
+{
+       return IOMEM_ERR_PTR(-EINVAL);
+}
 #endif
 
 #if defined(CONFIG_OF_ADDRESS) && defined(CONFIG_PCI)
index f562bf6ff71d64c76de34c41a1551c6d0f81481d..bb632484a86019c605945e6c36ff250a2e1dbe68 100644 (file)
@@ -86,8 +86,6 @@ void devm_iounmap(struct device *dev, void __iomem *addr)
 }
 EXPORT_SYMBOL(devm_iounmap);
 
-#define IOMEM_ERR_PTR(err) (__force void __iomem *)ERR_PTR(err)
-
 /**
  * devm_ioremap_resource() - check, request region, and ioremap resource
  * @dev: generic device to handle the resource for