Re: [PATCH] ASoC: soc-ops: Correct the max value for clamp in soc_mixer_reg_to_ctl()

[Richard Fitzgerald <rf@opensource.cirrus.com>]

On 17/12/2025 12:38 pm, Péter Ujfalusi wrote:
> 
> 
> On 17/12/2025 14:36, Richard Fitzgerald wrote:
>> On 17/12/2025 12:20 pm, Péter Ujfalusi wrote:
>>>
>>>
>>> On 17/12/2025 14:16, Mark Brown wrote:
>>>> On Wed, Dec 17, 2025 at 02:06:23PM +0200, Peter Ujfalusi wrote:
>>>>> In 'normal' controls the mc->min is the minimum value the register can
>>>>> have, the mc->max is the maximum (the steps between are max - min).
>>>>
>>>> Have you seen:
>>>>
>>>>     https://lore.kernel.org/r/20251216134938.788625-1-
>>>> sbinding@opensource.cirrus.com
>>>
>>> No, I tried to look for possible fixes for this, but have not found it.
>>>
>>> I think my one liner is a bit simpler with the same result, but I'll let
>>> people decide which is better (and test on Cirrus side)
>>>
>> Does it pass the kunit tests for SX controls?
>> The ASoC kunit tests have specific tests for SX controls.
>> The original patch failed those tests, but it was merged anyway.
> 
> It passes my manual tests on cs42l43, not sure how to run the unit-test
> for SX, can you by chance test this?
> 
The easiest option is to run them from your kernel build tree like this:

make mrproper
./tools/testing/kunit/kunit.py run 
--kunitconfig=tools/testing/kunit/configs/all_tests.config