linux-kernel - Re: [PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Fri, 26 Jan 2024 15:44:15 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
 Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
 Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>,
 Banajit Goswami <bgoswami@...cinc.com>, linux-sound@...r.kernel.org,
 linux-kernel@...r.kernel.org, alsa-devel@...a-project.org
Subject: Re: [PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn
 widgets on SM8350+

On 26/01/2024 15:42, Srinivas Kandagatla wrote:
> 
> 
> On 26/01/2024 13:23, Krzysztof Kozlowski wrote:
>> On 26/01/2024 14:21, Srinivas Kandagatla wrote:
>>>
>>>
>>> On 25/01/2024 15:31, Krzysztof Kozlowski wrote:
>>>> Hi,
>>>>
>>>> Changelog in individual patches.
>>>>
>>>> v1:
>>>> https://lore.kernel.org/all/20230717140138.201745-1-krzysztof.kozlowski@linaro.org/
>>>>
>>>
>>> Unfortunately this is breaking mic on X13s.
>>
>> Did you update also your mixer? I was asking last time about this as well...
> 
> Updating to new mixer setting fixes the issue.
> 
> This is going to be a issue with the existing ucm which becomes broken 
> with this new patches.
> 
> Are you going to send patches to fix the ucm too?

I could, but we should really think whether affecting sc8280xp is worth.
AFAIK, the microphones work there, so we do not need to fix them even
though fix is correct. It's the other platforms which need the fix.

Best regards,
Krzysztof