diff options
author | Nishka Dasgupta <nishkadg.linux@gmail.com> | 2019-08-15 11:07:04 +0530 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2019-08-15 18:00:41 +0100 |
commit | 81eeb0a35c2e40bcaf122c6aae3be4f7d9abe201 (patch) | |
tree | 3f2929e1d1be3a8ccdc1ed17f7af3c0542048ef5 /drivers/regulator | |
parent | ff461ebfd4b7051d9b79ca023741e6c2d960a912 (diff) | |
download | linux-81eeb0a35c2e40bcaf122c6aae3be4f7d9abe201.tar.bz2 |
regulator: core: Add label to collate of_node_put() statements
In function of_get_child_regulator(), the loop for_each_child_of_node()
contains two mid-loop return statements, each preceded by a statement
putting child. In order to reduce this repetition, create a new label,
err_node_put, that puts child and then returns the required value;
edit the mid-loop return blocks to instead go to this new label.
Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com>
Link: https://lore.kernel.org/r/20190815053704.32156-1-nishkadg.linux@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
-rw-r--r-- | drivers/regulator/core.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 7a5d52948703..4a27a46ec6e7 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -380,16 +380,17 @@ static struct device_node *of_get_child_regulator(struct device_node *parent, if (!regnode) { regnode = of_get_child_regulator(child, prop_name); - if (regnode) { - of_node_put(child); - return regnode; - } + if (regnode) + goto err_node_put; } else { - of_node_put(child); - return regnode; + goto err_node_put; } } return NULL; + +err_node_put: + of_node_put(child); + return regnode; } /** |