Re: [PATCH v1] arm64: defconfig: enable WCD937X codec driver as module
From: Krzysztof Kozlowski
Date: Thu Jan 08 2026 - 02:50:19 EST
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@xxxxxxxxxxxxxxxx>
> ---
Look incomplete.
https://lore.kernel.org/all/20240916052352.1819088-1-quic_mohs@xxxxxxxxxxx/
https://lore.kernel.org/all/20241010052646.2597418-1-quic_mohs@xxxxxxxxxxx/
https://lore.kernel.org/all/20241007082312.2136358-1-quic_mohs@xxxxxxxxxxx/
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?
Best regards,
Krzysztof