Re: [PATCH 1/2] ASoC: codecs: wcd9335: Add define for number of DAIs

From: Krzysztof Kozlowski
Date: Thu Dec 05 2024 - 07:41:12 EST


On 05/12/2024 13:33, Mark Brown wrote:
> On Thu, Dec 05, 2024 at 09:40:20AM +0100, Krzysztof Kozlowski wrote:
>> Number of DAIs in the codec is not really a binding, because it could
>> grow, e.g. when we implement missing features. Add the define to the
>> driver, which will replace the one in the binding header.
>
> This breaks an allmodconfig build:
>
> /build/stage/linux/sound/soc/codecs/wcd9335.c:162: error: "NUM_CODEC_DAIS" redef
> ined [-Werror]
> 162 | #define NUM_CODEC_DAIS (AIF4_PB + 1)
> |
> In file included from /build/stage/linux/sound/soc/codecs/wcd9335.c:28:
> /build/stage/linux/include/dt-bindings/sound/qcom,wcd9335.h:13: note: this is th
> e location of the previous definition
> 13 | #define NUM_CODEC_DAIS 7
> |

Apologies, last minute change hoping 6+1 equals 7, but obviously it does
not.

Best regards,
Krzysztof