]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
ASoC: codecs: wcd938x: use defines for entries in snd_soc_dai_driver array
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Thu, 19 Oct 2023 14:41:08 +0000 (16:41 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 25 Oct 2023 16:21:41 +0000 (17:21 +0100)
snd_soc_dai_driver array in wcd938x driver has two entries whose order
must match order of wcd938x->sdw_priv array.  The wcd938x_bind() and
wcd938x_codec_set_sdw_stream() rely on this order.  wcd938x->sdw_priv
array is indexed by enum with AIF1_PB and AIF1_CAP, so use the same
defines instead of raw numners for snd_soc_dai_driver array.
No functional impact.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20231019144108.42853-1-krzysztof.kozlowski@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wcd938x.c

index d27b919c63b4197e49331a57375ac784efbeb3d7..faf8d3f9b3c5d929d935ad4c5c63fe45371e4edc 100644 (file)
@@ -3394,7 +3394,7 @@ static const struct snd_soc_dai_ops wcd938x_sdw_dai_ops = {
 };
 
 static struct snd_soc_dai_driver wcd938x_dais[] = {
-       [0] = {
+       [AIF1_PB] = {
                .name = "wcd938x-sdw-rx",
                .playback = {
                        .stream_name = "WCD AIF1 Playback",
@@ -3407,7 +3407,7 @@ static struct snd_soc_dai_driver wcd938x_dais[] = {
                },
                .ops = &wcd938x_sdw_dai_ops,
        },
-       [1] = {
+       [AIF1_CAP] = {
                .name = "wcd938x-sdw-tx",
                .capture = {
                        .stream_name = "WCD AIF1 Capture",