Skip to contentSkip to conversation

Staging: octeon: Add missing of_node_put after calling of_parse_phandle.

PATCH New Sandhya Bankar 1 participant 1 message Sep 18, 2016 Staging

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 Staging: octeon: Add missing of_node_put after calling of_parse_phandle.

of_node_put needs to be called when the device node which is got from of_parse_phandle is no longer used.
This patch is found by below coccinelle script:

@@
expression e,e1,e2;
@@
*e = of_parse_phandle(...)
... when != of_node_put(e)

    when != true e == NULL
    when != e2 = e

e = e1

---
Changes in v2:
* Correcting the commit message.

 drivers/staging/octeon/ethernet-mdio.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
drivers/staging/octeon/ethernet-mdio.c +1 -0
index 1fde9c8..691e4a5 100644
@@ -168,6 +168,7 @@ int cvm_oct_phy_setup_device(struct net_device *dev)
 
 	phydev = of_phy_connect(dev, phy_node, cvm_oct_adjust_link, 0,
 				PHY_INTERFACE_MODE_GMII);
+	of_node_put(phy_node);
 
 	if (!phydev)
 		return -ENODEV;
Signature · 1 line
1.7.1