Re: [PATCH v1] arm64: defconfig: enable WCD937X codec driver as module

[Krzysztof Kozlowski <krzk@kernel.org>]

On 08/01/2026 08:41, Krzysztof Kozlowski wrote:
> On 08/01/2026 08:34, Ajay Kumar Nandam wrote:
>> Enable the Qualcomm WCD937X SoundWire codec driver as a
>> loadable module in the arm64 defconfig. This is required for
>> audio support on platforms using the WCD937X codec over the
>> SoundWire interface.
> 
> Not a good reason. You wrote four lines of which all are redundant,
> because you still did not prove we need to have support for WCD937x
> Soundwire interface.
> 
>>
>> Signed-off-by: Ajay Kumar Nandam <ajay.nandam@oss.qualcomm.com>
>> ---
> 
> 
> Look incomplete.
> 
> https://lore.kernel.org/all/20240916052352.1819088-1-quic_mohs@quicinc.com/
> https://lore.kernel.org/all/20241010052646.2597418-1-quic_mohs@quicinc.com/
> https://lore.kernel.org/all/20241007082312.2136358-1-quic_mohs@quicinc.com/
> 
> https://lore.kernel.org/all/?q=CONFIG_SND_SOC_WCD937X_SDW
> 
> But anyway, you did the same as all patches before, so it won't be
> applied. There is really no point in sending the same three four times.
> Nothing will change, no one will apply it, unless you finally fix the
> address list.
> 
> You also Cc-ed here completely random addresses... E.g. why is this
> relevant to Xilinx?


And I already said all this so you could have build on previous feedback
instead of repeating exactly same two mistakes.

https://lore.kernel.org/all/638a44fc-c693-4270-8643-7cd75a3c6ba6@linaro.org/

Best regards,
Krzysztof