diff options
| author | Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> | 2025-12-24 12:07:05 +0100 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2026-01-31 07:38:32 -0800 |
| commit | 6846757d4c851eb8693bbbfc2cde55d291ba1d29 (patch) | |
| tree | 5b0a72d3e88de772adf0b23e770ee108422d5997 /drivers | |
| parent | 0689522a92292ca057f26e99f39781c3cb26f483 (diff) | |
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 <krzysztof.kozlowski@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20251224110702.61746-6-krzysztof.kozlowski@oss.qualcomm.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/hwmon/nct7363.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/hwmon/nct7363.c b/drivers/hwmon/nct7363.c index 71cef794835d..4946d59ff807 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 */ |