[alsa-devel] cs42l52 alsa driver broken beep volume control

[Brian Austin]

On Thu, 20 Jun 2013, Lars-Peter Clausen wrote:

> Hi,
>
> I stumbled upon this while making some changes to the SOC_SINGLE_VALUE
> macro. For the Beep Volume control the cs42l52 driver sets the invert field
> to 7, which doesn't make much sense and is causing a build warning with my
> changes. I assume the intention was to actually use SOC_SINGLE_SX_TLV
> instead of SOC_SINGLE_TLV. But I'm not sure if setting min to 0x1f and max
> 0x7 is right either. Also according to the datasheet the tlv dB info seems
> to be wrong. The Beep Volume uses 2dB steps while the info uses 0.5dB steps.
>
> - Lars
>
>
Yes, I was supposed to use our SX Volume Control, and your correct the 
step size is 2dB from -65dB. I believe the values are correct but I'll 
have to do the math on it.

I can send a patch shortly.

Thanks,
Brian