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

Krzysztof Kozlowski posted 2 patches 1 year, 11 months ago
There is a newer version of this series
sound/soc/codecs/lpass-macro-common.h |   7 +
sound/soc/codecs/lpass-tx-macro.c     | 666 +++++++++++++++++++-------
2 files changed, 501 insertions(+), 172 deletions(-)
[PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+
Posted by Krzysztof Kozlowski 1 year, 11 months ago
Hi,

Changelog in individual patches.

v1:
https://lore.kernel.org/all/20230717140138.201745-1-krzysztof.kozlowski@linaro.org/

Best regards,
Krzysztof

Krzysztof Kozlowski (2):
  ASoC: codecs: tx-macro: split widgets per different LPASS versions
  ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+

 sound/soc/codecs/lpass-macro-common.h |   7 +
 sound/soc/codecs/lpass-tx-macro.c     | 666 +++++++++++++++++++-------
 2 files changed, 501 insertions(+), 172 deletions(-)

-- 
2.34.1
Re: [PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+
Posted by Srinivas Kandagatla 1 year, 11 months ago

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.

--srini

> Best regards,
> Krzysztof
> 
> Krzysztof Kozlowski (2):
>    ASoC: codecs: tx-macro: split widgets per different LPASS versions
>    ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+
> 
>   sound/soc/codecs/lpass-macro-common.h |   7 +
>   sound/soc/codecs/lpass-tx-macro.c     | 666 +++++++++++++++++++-------
>   2 files changed, 501 insertions(+), 172 deletions(-)
>
Re: [PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+
Posted by Krzysztof Kozlowski 1 year, 11 months ago
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...

Best regards,
Krzysztof
Re: [PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+
Posted by Srinivas Kandagatla 1 year, 11 months ago

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?


--srini
> 
> Best regards,
> Krzysztof
>
Re: [PATCH v2 0/2] ASoC: codecs: tx-macro: correct TX SMIC MUXn widgets on SM8350+
Posted by Krzysztof Kozlowski 1 year, 11 months ago
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