summaryrefslogtreecommitdiff
path: root/sound/pci
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2009-06-08 13:50:18 +0200
committerTakashi Iwai <tiwai@suse.de>2009-06-08 13:50:18 +0200
commitd436dd063be605dc29f17b2cb0b99a852db89bed (patch)
tree811df221765d5610ae06c59f345a2c21a3d247bf /sound/pci
parent54de6bc8b2437f642844cecb8d183df2368ffceb (diff)
ALSA: ctxfi - Make volume controls more intuitive
Change the volume control to dB scale (as the raw data seems so). Also added the TLV dB-scale information. Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci')
-rw-r--r--sound/pci/ctxfi/ctmixer.c49
1 files changed, 32 insertions, 17 deletions
diff --git a/sound/pci/ctxfi/ctmixer.c b/sound/pci/ctxfi/ctmixer.c
index fac783f585fc..796156e4bd38 100644
--- a/sound/pci/ctxfi/ctmixer.c
+++ b/sound/pci/ctxfi/ctmixer.c
@@ -18,11 +18,12 @@
#include "ctmixer.h"
#include "ctamixer.h"
+#include <linux/slab.h>
#include <sound/core.h>
#include <sound/control.h>
#include <sound/asoundef.h>
#include <sound/pcm.h>
-#include <linux/slab.h>
+#include <sound/tlv.h>
enum CT_SUM_CTL {
SUM_IN_F,
@@ -292,6 +293,7 @@ set_switch_state(struct ct_mixer *mixer,
mixer->switch_state &= ~(0x1 << (type - SWH_MIXER_START));
}
+#if 0 /* not used */
/* Map integer value ranging from 0 to 65535 to 14-bit float value ranging
* from 2^-6 to (1+1023/1024) */
static unsigned int uint16_to_float14(unsigned int x)
@@ -331,6 +333,12 @@ static unsigned int float14_to_uint16(unsigned int x)
return x;
}
+#endif /* not used */
+
+#define VOL_SCALE 0x1c
+#define VOL_MAX 0x100
+
+static const DECLARE_TLV_DB_SCALE(ct_vol_db_scale, -6400, 25, 1);
static int ct_alsa_mix_volume_info(struct snd_kcontrol *kcontrol,
struct snd_ctl_elem_info *uinfo)
@@ -338,8 +346,7 @@ static int ct_alsa_mix_volume_info(struct snd_kcontrol *kcontrol,
uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER;
uinfo->count = 2;
uinfo->value.integer.min = 0;
- uinfo->value.integer.max = 43690;
- uinfo->value.integer.step = 128;
+ uinfo->value.integer.max = VOL_MAX;
return 0;
}
@@ -349,15 +356,18 @@ static int ct_alsa_mix_volume_get(struct snd_kcontrol *kcontrol,
{
struct ct_atc *atc = snd_kcontrol_chip(kcontrol);
enum CT_AMIXER_CTL type = get_amixer_index(kcontrol->private_value);
- struct amixer *amixer = NULL;
- int i = 0;
+ struct amixer *amixer;
+ int i, val;
for (i = 0; i < 2; i++) {
amixer = ((struct ct_mixer *)atc->mixer)->
amixers[type*CHN_NUM+i];
- /* Convert 14-bit float-point scale to 16-bit integer volume */
- ucontrol->value.integer.value[i] =
- (float14_to_uint16(amixer->ops->get_scale(amixer)) & 0xffff);
+ val = amixer->ops->get_scale(amixer) / VOL_SCALE;
+ if (val < 0)
+ val = 0;
+ else if (val > VOL_MAX)
+ val = VOL_MAX;
+ ucontrol->value.integer.value[i] = val;
}
return 0;
@@ -369,16 +379,19 @@ static int ct_alsa_mix_volume_put(struct snd_kcontrol *kcontrol,
struct ct_atc *atc = snd_kcontrol_chip(kcontrol);
struct ct_mixer *mixer = atc->mixer;
enum CT_AMIXER_CTL type = get_amixer_index(kcontrol->private_value);
- struct amixer *amixer = NULL;
- int i = 0, j = 0, change = 0, val = 0;
+ struct amixer *amixer;
+ int i, j, val, oval, change = 0;
for (i = 0; i < 2; i++) {
- /* Convert 16-bit integer volume to 14-bit float-point scale */
- val = (ucontrol->value.integer.value[i] & 0xffff);
+ val = ucontrol->value.integer.value[i];
+ if (val < 0)
+ val = 0;
+ else if (val > VOL_MAX)
+ val = VOL_MAX;
+ val *= VOL_SCALE;
amixer = mixer->amixers[type*CHN_NUM+i];
- if ((float14_to_uint16(amixer->ops->get_scale(amixer)) & 0xff80)
- != (val & 0xff80)) {
- val = uint16_to_float14(val);
+ oval = amixer->ops->get_scale(amixer);
+ if (val != oval) {
amixer->ops->set_scale(amixer, val);
amixer->ops->commit_write(amixer);
change = 1;
@@ -398,11 +411,13 @@ static int ct_alsa_mix_volume_put(struct snd_kcontrol *kcontrol,
}
static struct snd_kcontrol_new vol_ctl = {
- .access = SNDRV_CTL_ELEM_ACCESS_READWRITE,
+ .access = SNDRV_CTL_ELEM_ACCESS_READWRITE |
+ SNDRV_CTL_ELEM_ACCESS_TLV_READ,
.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
.info = ct_alsa_mix_volume_info,
.get = ct_alsa_mix_volume_get,
- .put = ct_alsa_mix_volume_put
+ .put = ct_alsa_mix_volume_put,
+ .tlv = { .p = ct_vol_db_scale },
};
static void