of: property: Document that of_graph_get_endpoint_by_regs needs of_node_put
authorMaxime Ripard <maxime.ripard@bootlin.com>
Fri, 15 Mar 2019 09:22:47 +0000 (10:22 +0100)
committerRob Herring <robh@kernel.org>
Wed, 10 Apr 2019 14:03:03 +0000 (09:03 -0500)
The node returned by of_graph_get_endpoint_by_regs has a reference taken,
and we need to put that reference back when done with the node.

However, the documentation for that node doesn't mention it, so let's make
sure it does.

Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
Signed-off-by: Rob Herring <robh@kernel.org>
drivers/of/property.c

index 8631efa1daa152efa428b483d355d3a99d96963e..d7fa75e31f22415c447f05fa99437e1591cb0a1a 100644 (file)
@@ -659,7 +659,7 @@ EXPORT_SYMBOL(of_graph_get_next_endpoint);
  *
  * Return: An 'endpoint' node pointer which is identified by reg and at the same
  * is the child of a port node identified by port_reg. reg and port_reg are
- * ignored when they are -1.
+ * ignored when they are -1. Use of_node_put() on the pointer when done.
  */
 struct device_node *of_graph_get_endpoint_by_regs(
        const struct device_node *parent, int port_reg, int reg)