OSDN Git Service

ASoC: simple-card-utils: add simple_props_to_xxx() macro
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Sun, 11 Apr 2021 23:52:32 +0000 (08:52 +0900)
committerMark Brown <broonie@kernel.org>
Tue, 13 Apr 2021 11:32:52 +0000 (12:32 +0100)
We shouldn't use dai_props->cpus/codecs/cpu_dai/codec_dai/codec_conf
directly, because these are array to supporting multi CPU/Codec/Platform.
This patch adds asoc_link_to_xxx() macro for it.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/87k0p8wf9b.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
include/sound/simple_card_utils.h

index e366e43..78b6cf0 100644 (file)
@@ -74,32 +74,45 @@ struct asoc_simple_priv {
 #define simple_priv_to_dev(priv)       (simple_priv_to_card(priv)->dev)
 #define simple_priv_to_link(priv, i)   (simple_priv_to_card(priv)->dai_link + (i))
 
+#define simple_props_to_dlc_cpu(props, i)      ((props)->cpus + i)
+#define simple_props_to_dlc_codec(props, i)    ((props)->codecs + i)
+#define simple_props_to_dlc_platform(props, i) ((props)->platforms + i)
+
+#define simple_props_to_dai_cpu(props, i)      ((props)->cpu_dai + i)
+#define simple_props_to_dai_codec(props, i)    ((props)->codec_dai + i)
+#define simple_props_to_codec_conf(props, i)   ((props)->codec_conf + i)
+
 #define for_each_prop_dlc_cpus(props, i, cpu)                          \
        for ((i) = 0;                                                   \
-            ((i) < (props)->num.cpus) && ((cpu) = &(props)->cpus[i]);  \
+            ((i) < (props)->num.cpus) &&                               \
+                    ((cpu) = simple_props_to_dlc_cpu(props, i));       \
             (i)++)
-#define for_each_prop_dlc_codecs(props, i, codec)                              \
+#define for_each_prop_dlc_codecs(props, i, codec)                      \
        for ((i) = 0;                                                   \
-            ((i) < (props)->num.codecs) && ((codec) = &(props)->codecs[i]); \
+            ((i) < (props)->num.codecs) &&                             \
+                    ((codec) = simple_props_to_dlc_codec(props, i));   \
             (i)++)
 #define for_each_prop_dlc_platforms(props, i, platform)                        \
        for ((i) = 0;                                                   \
-            ((i) < (props)->num.platforms) && ((platform) = &(props)->platforms[i]); \
+            ((i) < (props)->num.platforms) &&                          \
+                    ((platform) = simple_props_to_dlc_platform(props, i)); \
             (i)++)
 #define for_each_prop_codec_conf(props, i, conf)                       \
        for ((i) = 0;                                                   \
             ((i) < (props)->num.codecs) &&                             \
                     (props)->codec_conf &&                             \
-                    ((conf) = &(props)->codec_conf[i]);                \
+                    ((conf) = simple_props_to_codec_conf(props, i));   \
             (i)++)
 
 #define for_each_prop_dai_cpu(props, i, cpu)                           \
        for ((i) = 0;                                                   \
-            ((i) < (props)->num.cpus) && ((cpu) = &(props)->cpu_dai[i]); \
+            ((i) < (props)->num.cpus) &&                               \
+                    ((cpu) = simple_props_to_dai_cpu(props, i));       \
             (i)++)
 #define for_each_prop_dai_codec(props, i, codec)                       \
        for ((i) = 0;                                                   \
-            ((i) < (props)->num.codecs) && ((codec) = &(props)->codec_dai[i]); \
+            ((i) < (props)->num.codecs) &&                             \
+                    ((codec) = simple_props_to_dai_codec(props, i));   \
             (i)++)
 
 struct link_info {