drivers/of: Make of_io_request_and_map() "name" argument optional
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 12 Jun 2018 00:01:07 +0000 (10:01 +1000)
committerRob Herring <robh@kernel.org>
Tue, 10 Jul 2018 14:08:57 +0000 (08:08 -0600)
These days of_address_to_resource() puts a reasonable name
in the resource struct, thus make the "name" argument an
optional override.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Rob Herring <robh@kernel.org>
drivers/of/address.c

index 5334991..7ddbf0a 100644 (file)
@@ -846,7 +846,7 @@ EXPORT_SYMBOL(of_iomap);
  *                        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
+ * @name:      name "override" for the memory region request or NULL
  *
  * Returns a pointer to the requested and mapped memory or an ERR_PTR() encoded
  * error code on failure. Usage example:
@@ -856,7 +856,7 @@ EXPORT_SYMBOL(of_iomap);
  *             return PTR_ERR(base);
  */
 void __iomem *of_io_request_and_map(struct device_node *np, int index,
-                                       const char *name)
+                                   const char *name)
 {
        struct resource res;
        void __iomem *mem;
@@ -864,6 +864,8 @@ void __iomem *of_io_request_and_map(struct device_node *np, int index,
        if (of_address_to_resource(np, index, &res))
                return IOMEM_ERR_PTR(-EINVAL);
 
+       if (!name)
+               name = res.name;
        if (!request_mem_region(res.start, resource_size(&res), name))
                return IOMEM_ERR_PTR(-EBUSY);