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