The for_each_child_of_node_scoped() helper provides a scope-based
clean-up functionality to put the device_node automatically, and
as such, there is no need to call of_node_put() directly.

Thus, use this helper to simplify the code.

Signed-off-by: Ai Chao <aic...@kylinos.cn>
---
 sound/soc/meson/axg-card.c         |  3 +--
 sound/soc/meson/meson-card-utils.c | 14 ++++----------
 2 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/sound/soc/meson/axg-card.c b/sound/soc/meson/axg-card.c
index a2dfccb7990f..b4dca80e15e4 100644
--- a/sound/soc/meson/axg-card.c
+++ b/sound/soc/meson/axg-card.c
@@ -222,7 +222,6 @@ static int axg_card_parse_codecs_masks(struct snd_soc_card 
*card,
                                       struct axg_dai_link_tdm_data *be)
 {
        struct axg_dai_link_tdm_mask *codec_mask;
-       struct device_node *np;
 
        codec_mask = devm_kcalloc(card->dev, link->num_codecs,
                                  sizeof(*codec_mask), GFP_KERNEL);
@@ -231,7 +230,7 @@ static int axg_card_parse_codecs_masks(struct snd_soc_card 
*card,
 
        be->codec_masks = codec_mask;
 
-       for_each_child_of_node(node, np) {
+       for_each_child_of_node_scoped(node, np) {
                snd_soc_of_get_slot_mask(np, "dai-tdm-slot-rx-mask",
                                         &codec_mask->rx);
                snd_soc_of_get_slot_mask(np, "dai-tdm-slot-tx-mask",
diff --git a/sound/soc/meson/meson-card-utils.c 
b/sound/soc/meson/meson-card-utils.c
index cfc7f6e41ab5..cf08c8dedd91 100644
--- a/sound/soc/meson/meson-card-utils.c
+++ b/sound/soc/meson/meson-card-utils.c
@@ -137,7 +137,6 @@ int meson_card_set_be_link(struct snd_soc_card *card,
                           struct device_node *node)
 {
        struct snd_soc_dai_link_component *codec;
-       struct device_node *np;
        int ret, num_codecs;
 
        num_codecs = of_get_child_count(node);
@@ -154,12 +153,10 @@ int meson_card_set_be_link(struct snd_soc_card *card,
        link->codecs = codec;
        link->num_codecs = num_codecs;
 
-       for_each_child_of_node(node, np) {
+       for_each_child_of_node_scoped(node, np) {
                ret = meson_card_parse_dai(card, np, codec);
-               if (ret) {
-                       of_node_put(np);
+               if (ret)
                        return ret;
-               }
 
                codec++;
        }
@@ -198,7 +195,6 @@ static int meson_card_add_links(struct snd_soc_card *card)
 {
        struct meson_card *priv = snd_soc_card_get_drvdata(card);
        struct device_node *node = card->dev->of_node;
-       struct device_node *np;
        int num, i, ret;
 
        num = of_get_child_count(node);
@@ -212,12 +208,10 @@ static int meson_card_add_links(struct snd_soc_card *card)
                return ret;
 
        i = 0;
-       for_each_child_of_node(node, np) {
+       for_each_child_of_node_scoped(node, np) {
                ret = priv->match_data->add_link(card, np, &i);
-               if (ret) {
-                       of_node_put(np);
+               if (ret)
                        return ret;
-               }
 
                i++;
        }
-- 
2.47.1


Reply via email to