From patchwork Fri Sep 14 08:17:42 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [v2, 1/7] dt: introduce of_get_child_by_name to get child node by name. Date: Thu, 13 Sep 2012 22:17:42 -0000 From: Srinivas KANDAGATLA X-Patchwork-Id: 183879 Message-Id: <1347610662-20105-1-git-send-email-srinivas.kandagatla@st.com> To: davem@davemloft.net, robherring2@gmail.com, bergner@us.ibm.com, devicetree-discuss@lists.ozlabs.org Cc: jassi.brar@samsung.com, srinivas.kandagatla@st.com, afleming@freescale.com, netdev@vger.kernel.org, linuxppc-dev@lists.ozlabs.org From: Srinivas Kandagatla This patch introduces of_get_child_by_name function to get a child node by its name in a given parent node. Without this patch each driver code has to iterate the parent and do a string compare, However having of_get_child_by_name libary function would avoid code duplication, errors and is more convenient. Signed-off-by: Srinivas Kandagatla --- drivers/of/base.c | 25 +++++++++++++++++++++++++ include/linux/of.h | 2 ++ 2 files changed, 27 insertions(+), 0 deletions(-) diff --git a/drivers/of/base.c b/drivers/of/base.c index d4a1c9a..7391f8a 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -391,6 +391,31 @@ struct device_node *of_get_next_available_child(const struct device_node *node, EXPORT_SYMBOL(of_get_next_available_child); /** + * of_get_child_by_name - Find the child node by name for a given parent + * @node: parent node + * @name: child name to look for. + * + * This function looks for child node for given matching name + * + * Returns a node pointer if found, with refcount incremented, use + * of_node_put() on it when done. + * Returns NULL if node is not found. + */ + +struct device_node *of_get_child_by_name(const struct device_node *node, + const char *name) +{ + struct device_node *child; + + for_each_child_of_node(node, child) + if (child->name && (of_node_cmp(child->name, name) == 0) + && of_node_get(child)) + break; + return child; +} +EXPORT_SYMBOL(of_get_child_by_name); + +/** * of_find_node_by_path - Find a node matching a full OF path * @path: The full path to match * diff --git a/include/linux/of.h b/include/linux/of.h index 1b11632..7b8e3cd 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -192,6 +192,8 @@ extern struct device_node *of_get_next_child(const struct device_node *node, struct device_node *prev); extern struct device_node *of_get_next_available_child( const struct device_node *node, struct device_node *prev); +extern struct device_node *of_get_child_by_name(const struct device_node *node, + const char *name); #define for_each_child_of_node(parent, child) \ for (child = of_get_next_child(parent, NULL); child != NULL; \