While checking cpus before dereferencing the pointer is required, it is
not necessary for codecs. 'codec' can't possibly be NULL in the loop

Reported-by: Dan Carpenter <[email protected]>
Signed-off-by: Jerome Brunet <[email protected]>
---
 sound/soc/meson/axg-card.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/sound/soc/meson/axg-card.c b/sound/soc/meson/axg-card.c
index 70bb0cbad233..14a8321744da 100644
--- a/sound/soc/meson/axg-card.c
+++ b/sound/soc/meson/axg-card.c
@@ -118,8 +118,7 @@ static void axg_card_clean_references(struct axg_card *priv)
                        if (link->cpus)
                                of_node_put(link->cpus->of_node);
                        for_each_link_codecs(link, j, codec)
-                               if (codec)
-                                       of_node_put(codec->of_node);
+                               of_node_put(codec->of_node);
                }
        }
 
-- 
2.21.0

Reply via email to