1 From f4fcfdda2fd8834c62dcb9bfddcf1f89d190b70e Mon Sep 17 00:00:00 2001
2 From: "Rob Herring (Arm)" <robh@kernel.org>
3 Date: Wed, 23 Apr 2025 14:42:13 -0500
4 Subject: [PATCH] of: reserved_mem: Add functions to parse "memory-region"
6 Drivers with "memory-region" properties currently have to do their own
7 parsing of "memory-region" properties. The result is all the drivers
8 have similar patterns of a call to parse "memory-region" and then get
9 the region's address and size. As this is a standard property, it should
10 have common functions for drivers to use. Add new functions to count the
11 number of regions and retrieve the region's address as a resource.
13 Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>
14 Acked-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
15 Link: https://lore.kernel.org/r/20250423-dt-memory-region-v2-v2-1-2fbd6ebd3c88@kernel.org
16 Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
18 drivers/of/of_reserved_mem.c | 80 +++++++++++++++++++++++++++++++++
19 include/linux/of_reserved_mem.h | 26 +++++++++++
20 2 files changed, 106 insertions(+)
22 --- a/drivers/of/of_reserved_mem.c
23 +++ b/drivers/of/of_reserved_mem.c
25 #define pr_fmt(fmt) "OF: reserved mem: " fmt
27 #include <linux/err.h>
28 +#include <linux/ioport.h>
29 #include <linux/libfdt.h>
31 #include <linux/of_fdt.h>
32 @@ -514,3 +515,82 @@ struct reserved_mem *of_reserved_mem_loo
35 EXPORT_SYMBOL_GPL(of_reserved_mem_lookup);
38 + * of_reserved_mem_region_to_resource() - Get a reserved memory region as a resource
39 + * @np: node containing 'memory-region' property
40 + * @idx: index of 'memory-region' property to lookup
41 + * @res: Pointer to a struct resource to fill in with reserved region
43 + * This function allows drivers to lookup a node's 'memory-region' property
44 + * entries by index and return a struct resource for the entry.
46 + * Returns 0 on success with @res filled in. Returns -ENODEV if 'memory-region'
47 + * is missing or unavailable, -EINVAL for any other error.
49 +int of_reserved_mem_region_to_resource(const struct device_node *np,
50 + unsigned int idx, struct resource *res)
52 + struct reserved_mem *rmem;
57 + struct device_node __free(device_node) *target = of_parse_phandle(np, "memory-region", idx);
58 + if (!target || !of_device_is_available(target))
61 + rmem = of_reserved_mem_lookup(target);
65 + resource_set_range(res, rmem->base, rmem->size);
66 + res->name = rmem->name;
69 +EXPORT_SYMBOL_GPL(of_reserved_mem_region_to_resource);
72 + * of_reserved_mem_region_to_resource_byname() - Get a reserved memory region as a resource
73 + * @np: node containing 'memory-region' property
74 + * @name: name of 'memory-region' property entry to lookup
75 + * @res: Pointer to a struct resource to fill in with reserved region
77 + * This function allows drivers to lookup a node's 'memory-region' property
78 + * entries by name and return a struct resource for the entry.
80 + * Returns 0 on success with @res filled in, or a negative error-code on
83 +int of_reserved_mem_region_to_resource_byname(const struct device_node *np,
85 + struct resource *res)
92 + idx = of_property_match_string(np, "memory-region-names", name);
96 + return of_reserved_mem_region_to_resource(np, idx, res);
98 +EXPORT_SYMBOL_GPL(of_reserved_mem_region_to_resource_byname);
101 + * of_reserved_mem_region_count() - Return the number of 'memory-region' entries
102 + * @np: node containing 'memory-region' property
104 + * This function allows drivers to retrieve the number of entries for a node's
105 + * 'memory-region' property.
107 + * Returns the number of entries on success, or negative error code on a
108 + * malformed property.
110 +int of_reserved_mem_region_count(const struct device_node *np)
112 + return of_count_phandle_with_args(np, "memory-region", NULL);
114 +EXPORT_SYMBOL_GPL(of_reserved_mem_region_count);
115 --- a/include/linux/of_reserved_mem.h
116 +++ b/include/linux/of_reserved_mem.h
119 struct of_phandle_args;
120 struct reserved_mem_ops;
123 struct reserved_mem {
125 @@ -40,6 +41,12 @@ int of_reserved_mem_device_init_by_name(
126 void of_reserved_mem_device_release(struct device *dev);
128 struct reserved_mem *of_reserved_mem_lookup(struct device_node *np);
129 +int of_reserved_mem_region_to_resource(const struct device_node *np,
130 + unsigned int idx, struct resource *res);
131 +int of_reserved_mem_region_to_resource_byname(const struct device_node *np,
132 + const char *name, struct resource *res);
133 +int of_reserved_mem_region_count(const struct device_node *np);
137 #define RESERVEDMEM_OF_DECLARE(name, compat, init) \
138 @@ -64,6 +71,25 @@ static inline struct reserved_mem *of_re
143 +static inline int of_reserved_mem_region_to_resource(const struct device_node *np,
145 + struct resource *res)
150 +static inline int of_reserved_mem_region_to_resource_byname(const struct device_node *np,
152 + struct resource *res)
157 +static inline int of_reserved_mem_region_count(const struct device_node *np)