Re: [PATCH 1/2] ALSA: ASoC: add STA32X codec driver

[Johannes Stezenbach <js@sig21.net>]

On Wed, Jun 15, 2011 at 04:05:16PM +0100, Mark Brown wrote:
> On Tue, Jun 14, 2011 at 09:27:05PM +0200, Daniel Mack wrote:
> 
> > +static const char *sta32x_limiter_ac_release_thr[] = {
> > +	"-inf", "-29dB", "-20dB", "-16dB", "-14dB", "-12dB", "-10dB", "-8dB",
> > +	"-7dB", "-6dB", "-5dB", "-4dB", "-3dB", "-2dB", "-1dB", "0dB" };
> > +static const char *sta32x_limiter_drc_attack_thr[] = {
> > +	"-31dB", "-29dB", "-27dB", "-25dB", "-23dB", "-21dB", "-19dB", "-17dB",
> > +	"-16dB", "-15dB", "-14dB", "-13dB", "-12dB", "-10dB", "-7dB", "-4dB" };
> > +static const char *sta32x_limiter_drc_release_thr[] = {
> > +	"-inf", "-38dB", "-36dB", "-33dB", "-31dB", "-30dB", "-28dB", "-26dB",
> > +	"-24dB", "-22dB", "-20dB", "-18dB", "-15dB", "-12dB", "-9dB", "-6dB" };
> > +
> 
> Doing these as regular volume TLVs will tend to work better in UIs.

The steps are not evenly distributed so I thought this is the
only way to describe the hardware correctly.  Or is there a
way to do controls which look like a slider in alsamixer
with steps as shown above?

Thanks for your review.


Johannes