From 6846757d4c851eb8693bbbfc2cde55d291ba1d29 Mon Sep 17 00:00:00 2001 From: Krzysztof Kozlowski Date: Wed, 24 Dec 2025 12:07:05 +0100 Subject: [PATCH] hwmon: (nct7363) Simplify with scoped for each OF child loop Use scoped for-each loop when iterating over device nodes to make code a bit simpler. Signed-off-by: Krzysztof Kozlowski Link: https://lore.kernel.org/r/20251224110702.61746-6-krzysztof.kozlowski@oss.qualcomm.com Signed-off-by: Guenter Roeck --- drivers/hwmon/nct7363.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/hwmon/nct7363.c b/drivers/hwmon/nct7363.c index 71cef794835df..4946d59ff807b 100644 --- a/drivers/hwmon/nct7363.c +++ b/drivers/hwmon/nct7363.c @@ -396,7 +396,6 @@ static const struct regmap_config nct7363_regmap_config = { static int nct7363_probe(struct i2c_client *client) { struct device *dev = &client->dev; - struct device_node *child; struct nct7363_data *data; struct device *hwmon_dev; int ret; @@ -409,12 +408,10 @@ static int nct7363_probe(struct i2c_client *client) if (IS_ERR(data->regmap)) return PTR_ERR(data->regmap); - for_each_child_of_node(dev->of_node, child) { + for_each_child_of_node_scoped(dev->of_node, child) { ret = nct7363_present_pwm_fanin(dev, child, data); - if (ret) { - of_node_put(child); + if (ret) return ret; - } } /* Initialize the chip */ -- 2.47.3