ASoC: soc-core: tidyup strcmp() param on snd_soc_is_matching_dai()
authorKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Thu, 22 Feb 2024 05:51:11 +0000 (05:51 +0000)
committerMark Brown <broonie@kernel.org>
Thu, 22 Feb 2024 19:49:41 +0000 (19:49 +0000)
commitb1724c00f0d9224c50a4fab6a85be8e2155a9a1b
tree52421beb9a317b0f9b6e47f0e030085d40911873
parent253ce07d2a091e98ef53e700e7fa221b28c4f964
ASoC: soc-core: tidyup strcmp() param on snd_soc_is_matching_dai()

snd_soc_is_matching_dai() checks DAI name, which is paired function
with snd_soc_dai_name_get().

It checks dlc->dai_name and dai->name (A) or dai->driver_name (B) or
dai->component->name (C)

static int snd_soc_is_matching_dai(...)
{
...
if (strcmp(dlc->dai_name, dai->name) == 0)
   ~~~~~~~~~~~~~  ^^^^^^^^^(A)
if (...
    strcmp(dai->driver->name, dlc->dai_name) == 0)
 (B)^^^^^^^^^^^^^^^^  ~~~~~~~~~~~~~
if (...
    strcmp(dlc->dai_name, dai->component->name) == 0)
   ~~~~~~~~~~~~~  ^^^^^^^^^^^^^^^^^^(C)
...
}

But (B) part order is different with (A) and (C) (= ^^^^ and ~~~~).
This is not a big deal, but confusable to read. Fixup it.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://msgid.link/r/87wmqxjbcg.wl-kuninori.morimoto.gx@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/soc-core.c