]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
ASoC: codecs: lpass: Drop unused AIF_INVALID first DAI identifier
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Fri, 23 May 2025 12:18:12 +0000 (14:18 +0200)
committerMark Brown <broonie@kernel.org>
Sun, 8 Jun 2025 22:34:48 +0000 (23:34 +0100)
commitbb4a0f497bc19558ba7fe9feac814286fc7ebe85
tree04df26b20af4e176e831fbe562f973454005eded
parent19272b37aa4f83ca52bdf9c16d5d81bdd1354494
ASoC: codecs: lpass: Drop unused AIF_INVALID first DAI identifier

All four Qualcomm SoC macro codecs define DAI IDs in an enum starting
with AIF_INVALID=0, which is nowhere used in the code thus actual DAI
IDs start from 1.  Drivers do not have their own of_xlate_dai_name(),
thus snd_soc_get_dlc() expects the DTS to start numbering DAIs from 0,
which creates confusing debugging scenario, e.g. DTS should use
<&lpass_wsamacro 2> for WSA_MACRO_AIF_VI with dai->id=3.

This also wastes some space, because drivers allocate few arrays for all
DAIs and basically the [0] is never used.

Drop the confusing first AIF_INVALID DAI identifier so the enum with DAI
IDs will start from 0.  This has little functional impact and does not
affect the ABI, except saving a few bytes of memory per driver.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://patch.msgid.link/20250523121811.380045-2-krzysztof.kozlowski@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/lpass-rx-macro.c
sound/soc/codecs/lpass-tx-macro.c
sound/soc/codecs/lpass-va-macro.c
sound/soc/codecs/lpass-wsa-macro.c