ASoC: audio-graph-card2.c: use of_property_read_u32() for rate
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Fri, 1 Jul 2022 05:18:14 +0000 (05:18 +0000)
committerMark Brown <broonie@kernel.org>
Fri, 8 Jul 2022 10:51:41 +0000 (11:51 +0100)
Audio Graph Card2 is using of_get_property(), but it should use
of_property_read_u32() to getting rate. Otherwise the setting will be
strange value. This patch fixup it.

Fixes: c3a15c92a67b701 ("ASoC: audio-graph-card2: add Codec2Codec support")
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/87h741s961.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/generic/audio-graph-card2.c

index d34b29a49268e979f0a279df5ddfa3001f8616cb..8e0628e6f2a0fd46cb725e6d449f280f92a0b7f1 100644 (file)
@@ -856,7 +856,7 @@ int audio_graph2_link_c2c(struct asoc_simple_priv *priv,
        struct device_node *port0, *port1, *ports;
        struct device_node *codec0_port, *codec1_port;
        struct device_node *ep0, *ep1;
-       u32 val;
+       u32 val = 0;
        int ret = -EINVAL;
 
        /*
@@ -880,7 +880,8 @@ int audio_graph2_link_c2c(struct asoc_simple_priv *priv,
        ports = of_get_parent(port0);
        port1 = of_get_next_child(ports, lnk);
 
-       if (!of_get_property(ports, "rate", &val)) {
+       of_property_read_u32(ports, "rate", &val);
+       if (!val) {
                struct device *dev = simple_priv_to_dev(priv);
 
                dev_err(dev, "Codec2Codec needs rate settings\n");