Commit 69f3aa6a authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Rafael J. Wysocki
Browse files

thermal: of: Simplify thermal_of_should_bind with scoped for each OF child 



Use scoped for_each_child_of_node_scoped() when iterating over device
nodes to make code a bit simpler.

Reviewed-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://patch.msgid.link/20241010-b4-cleanup-h-of-node-put-thermal-v4-1-bfbe29ad81f4@linaro.org


Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent ac1f43c0

drivers/thermal/thermal_of.c

+2 −3
Original line number Diff line number Diff line
@@ -297,7 +297,7 @@ static bool thermal_of_should_bind(struct thermal_zone_device *tz, 
				   struct thermal_cooling_device *cdev,

				   struct cooling_spec *c)

{

	struct device_node *tz_np, *cm_np, *child;

	struct device_node *tz_np, *cm_np;

	bool result = false;



	tz_np = thermal_of_zone_get_by_name(tz);
@@ -311,7 +311,7 @@ static bool thermal_of_should_bind(struct thermal_zone_device *tz, 
		goto out;



	/* Look up the trip and the cdev in the cooling maps. */

	for_each_child_of_node(cm_np, child) {

	for_each_child_of_node_scoped(cm_np, child) {

		struct device_node *tr_np;

		int count, i;
@@ -330,7 +330,6 @@ static bool thermal_of_should_bind(struct thermal_zone_device *tz, 
				break;

		}



		of_node_put(child);

		break;

	}