Skip to contentSkip to conversation

Patch "PM / OPP: Add missing of_node_put(np)" has been added to the 4.9-stable tree

PATCH [email protected] 1 participant 1 message Nov 28, 2017

Subsystems (5)
lore
Share & tools
lore

[email protected]

author maintainer Applied
#0 lore raw

This is a note to let you know that I've just added the patch titled

PM / OPP: Add missing of_node_put(np)

to the 4.9-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
pm-opp-add-missing-of_node_put-np.patch
and it can be found in the queue-4.9 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.

Quoted 1 line

From 7978db344719dab1e56d05e6fc04aaaddcde0a5e Mon Sep 17 00:00:00 2001

From: Tobias Jordan <[email protected]>
Date: Wed, 4 Oct 2017 11:35:03 +0530
Subject: PM / OPP: Add missing of_node_put(np)

From: Tobias Jordan <[email protected]>

commit 7978db344719dab1e56d05e6fc04aaaddcde0a5e upstream.

The for_each_available_child_of_node() loop in _of_add_opp_table_v2() doesn't drop the reference to "np" on errors. Fix that.

Fixes 274659029c9d (PM / OPP: Add support to parse "operating-points-v2" bindings)
[ VK: Improved commit log. ]
---
 drivers/base/power/opp/of.c |    1 +
 1 file changed, 1 insertion(+)
drivers/base/power/opp/of.c +1 -0
@@ -348,6 +348,7 @@ static int _of_add_opp_table_v2(struct d
 		if (ret) {
 			dev_err(dev, "%s: Failed to add OPP, %d\n", __func__,
 				ret);
+			of_node_put(np);
 			goto free_table;
 		}
 	}

Patches currently in stable-queue which might be from [email protected] are

queue-4.9/pm-opp-add-missing-of_node_put-np.patch