ALSA: hda - Handle a few verbs as read-only
[pandora-kernel.git] / sound / hda / hdac_regmap.c
index 933907b..d401e5c 100644 (file)
 
 static bool hda_volatile_reg(struct device *dev, unsigned int reg)
 {
+       struct hdac_device *codec = dev_to_hdac_dev(dev);
        unsigned int verb = get_verb(reg);
 
        switch (verb) {
        case AC_VERB_GET_PROC_COEF:
+               return !codec->cache_coef;
        case AC_VERB_GET_COEF_INDEX:
        case AC_VERB_GET_PROC_STATE:
        case AC_VERB_GET_POWER_STATE:
@@ -60,6 +62,13 @@ static bool hda_writeable_reg(struct device *dev, unsigned int reg)
 {
        struct hdac_device *codec = dev_to_hdac_dev(dev);
        unsigned int verb = get_verb(reg);
+       int i;
+
+       for (i = 0; i < codec->vendor_verbs.used; i++) {
+               unsigned int *v = snd_array_elem(&codec->vendor_verbs, i);
+               if (verb == *v)
+                       return true;
+       }
 
        if (codec->caps_overwriting)
                return true;
@@ -68,6 +77,8 @@ static bool hda_writeable_reg(struct device *dev, unsigned int reg)
        case AC_VERB_GET_STREAM_FORMAT:
        case AC_VERB_GET_AMP_GAIN_MUTE:
                return true;
+       case AC_VERB_GET_PROC_COEF:
+               return codec->cache_coef;
        case 0xf00:
                break;
        default:
@@ -77,7 +88,6 @@ static bool hda_writeable_reg(struct device *dev, unsigned int reg)
        switch (verb) {
        case AC_VERB_GET_CONNECT_SEL:
        case AC_VERB_GET_SDI_SELECT:
-       case AC_VERB_GET_CONV:
        case AC_VERB_GET_PIN_WIDGET_CONTROL:
        case AC_VERB_GET_UNSOLICITED_RESPONSE: /* only as SET_UNSOLICITED_ENABLE */
        case AC_VERB_GET_BEEP_CONTROL:
@@ -85,14 +95,12 @@ static bool hda_writeable_reg(struct device *dev, unsigned int reg)
        case AC_VERB_GET_DIGI_CONVERT_1:
        case AC_VERB_GET_DIGI_CONVERT_2: /* only for beep control */
        case AC_VERB_GET_VOLUME_KNOB_CONTROL:
-       case AC_VERB_GET_CONFIG_DEFAULT:
        case AC_VERB_GET_GPIO_MASK:
        case AC_VERB_GET_GPIO_DIRECTION:
        case AC_VERB_GET_GPIO_DATA: /* not for volatile read */
        case AC_VERB_GET_GPIO_WAKE_MASK:
        case AC_VERB_GET_GPIO_UNSOLICITED_RSP_MASK:
        case AC_VERB_GET_GPIO_STICKY_MASK:
-       case AC_VERB_GET_CVT_CHAN_COUNT:
                return true;
        }
 
@@ -112,19 +120,143 @@ static bool hda_readable_reg(struct device *dev, unsigned int reg)
        case AC_VERB_GET_CONNECT_LIST:
        case AC_VERB_GET_SUBSYSTEM_ID:
                return true;
+       /* below are basically writable, but disabled for reducing unnecessary
+        * writes at sync
+        */
+       case AC_VERB_GET_CONFIG_DEFAULT: /* usually just read */
+       case AC_VERB_GET_CONV: /* managed in PCM code */
+       case AC_VERB_GET_CVT_CHAN_COUNT: /* managed in HDMI CA code */
+               return true;
        }
 
        return hda_writeable_reg(dev, reg);
 }
 
+/*
+ * Stereo amp pseudo register:
+ * for making easier to handle the stereo volume control, we provide a
+ * fake register to deal both left and right channels by a single
+ * (pseudo) register access.  A verb consisting of SET_AMP_GAIN with
+ * *both* SET_LEFT and SET_RIGHT bits takes a 16bit value, the lower 8bit
+ * for the left and the upper 8bit for the right channel.
+ */
+static bool is_stereo_amp_verb(unsigned int reg)
+{
+       if (((reg >> 8) & 0x700) != AC_VERB_SET_AMP_GAIN_MUTE)
+               return false;
+       return (reg & (AC_AMP_SET_LEFT | AC_AMP_SET_RIGHT)) ==
+               (AC_AMP_SET_LEFT | AC_AMP_SET_RIGHT);
+}
+
+/* read a pseudo stereo amp register (16bit left+right) */
+static int hda_reg_read_stereo_amp(struct hdac_device *codec,
+                                  unsigned int reg, unsigned int *val)
+{
+       unsigned int left, right;
+       int err;
+
+       reg &= ~(AC_AMP_SET_LEFT | AC_AMP_SET_RIGHT);
+       err = snd_hdac_exec_verb(codec, reg | AC_AMP_GET_LEFT, 0, &left);
+       if (err < 0)
+               return err;
+       err = snd_hdac_exec_verb(codec, reg | AC_AMP_GET_RIGHT, 0, &right);
+       if (err < 0)
+               return err;
+       *val = left | (right << 8);
+       return 0;
+}
+
+/* write a pseudo stereo amp register (16bit left+right) */
+static int hda_reg_write_stereo_amp(struct hdac_device *codec,
+                                   unsigned int reg, unsigned int val)
+{
+       int err;
+       unsigned int verb, left, right;
+
+       verb = AC_VERB_SET_AMP_GAIN_MUTE << 8;
+       if (reg & AC_AMP_GET_OUTPUT)
+               verb |= AC_AMP_SET_OUTPUT;
+       else
+               verb |= AC_AMP_SET_INPUT | ((reg & 0xf) << 8);
+       reg = (reg & ~0xfffff) | verb;
+
+       left = val & 0xff;
+       right = (val >> 8) & 0xff;
+       if (left == right) {
+               reg |= AC_AMP_SET_LEFT | AC_AMP_SET_RIGHT;
+               return snd_hdac_exec_verb(codec, reg | left, 0, NULL);
+       }
+
+       err = snd_hdac_exec_verb(codec, reg | AC_AMP_SET_LEFT | left, 0, NULL);
+       if (err < 0)
+               return err;
+       err = snd_hdac_exec_verb(codec, reg | AC_AMP_SET_RIGHT | right, 0, NULL);
+       if (err < 0)
+               return err;
+       return 0;
+}
+
+/* read a pseudo coef register (16bit) */
+static int hda_reg_read_coef(struct hdac_device *codec, unsigned int reg,
+                            unsigned int *val)
+{
+       unsigned int verb;
+       int err;
+
+       if (!codec->cache_coef)
+               return -EINVAL;
+       /* LSB 8bit = coef index */
+       verb = (reg & ~0xfff00) | (AC_VERB_SET_COEF_INDEX << 8);
+       err = snd_hdac_exec_verb(codec, verb, 0, NULL);
+       if (err < 0)
+               return err;
+       verb = (reg & ~0xfffff) | (AC_VERB_GET_COEF_INDEX << 8);
+       return snd_hdac_exec_verb(codec, verb, 0, val);
+}
+
+/* write a pseudo coef register (16bit) */
+static int hda_reg_write_coef(struct hdac_device *codec, unsigned int reg,
+                             unsigned int val)
+{
+       unsigned int verb;
+       int err;
+
+       if (!codec->cache_coef)
+               return -EINVAL;
+       /* LSB 8bit = coef index */
+       verb = (reg & ~0xfff00) | (AC_VERB_SET_COEF_INDEX << 8);
+       err = snd_hdac_exec_verb(codec, verb, 0, NULL);
+       if (err < 0)
+               return err;
+       verb = (reg & ~0xfffff) | (AC_VERB_GET_COEF_INDEX << 8) |
+               (val & 0xffff);
+       return snd_hdac_exec_verb(codec, verb, 0, NULL);
+}
+
 static int hda_reg_read(void *context, unsigned int reg, unsigned int *val)
 {
        struct hdac_device *codec = context;
+       int verb = get_verb(reg);
+       int err;
 
        if (!codec_is_running(codec))
                return -EAGAIN;
        reg |= (codec->addr << 28);
-       return snd_hdac_exec_verb(codec, reg, 0, val);
+       if (is_stereo_amp_verb(reg))
+               return hda_reg_read_stereo_amp(codec, reg, val);
+       if (verb == AC_VERB_GET_PROC_COEF)
+               return hda_reg_read_coef(codec, reg, val);
+       err = snd_hdac_exec_verb(codec, reg, 0, val);
+       if (err < 0)
+               return err;
+       /* special handling for asymmetric reads */
+       if (verb == AC_VERB_GET_POWER_STATE) {
+               if (*val & AC_PWRST_ERROR)
+                       *val = -1;
+               else /* take only the actual state */
+                       *val = (*val >> 4) & 0x0f;
+       }
+       return 0;
 }
 
 static int hda_reg_write(void *context, unsigned int reg, unsigned int val)
@@ -138,7 +270,13 @@ static int hda_reg_write(void *context, unsigned int reg, unsigned int val)
 
        reg &= ~0x00080000U; /* drop GET bit */
        reg |= (codec->addr << 28);
+
+       if (is_stereo_amp_verb(reg))
+               return hda_reg_write_stereo_amp(codec, reg, val);
+
        verb = get_verb(reg);
+       if (verb == AC_VERB_SET_PROC_COEF)
+               return hda_reg_write_coef(codec, reg, val);
 
        switch (verb & 0xf00) {
        case AC_VERB_SET_AMP_GAIN_MUTE:
@@ -200,6 +338,7 @@ int snd_hdac_regmap_init(struct hdac_device *codec)
        if (IS_ERR(regmap))
                return PTR_ERR(regmap);
        codec->regmap = regmap;
+       snd_array_init(&codec->vendor_verbs, sizeof(unsigned int), 8);
        return 0;
 }
 EXPORT_SYMBOL_GPL(snd_hdac_regmap_init);
@@ -209,10 +348,30 @@ void snd_hdac_regmap_exit(struct hdac_device *codec)
        if (codec->regmap) {
                regmap_exit(codec->regmap);
                codec->regmap = NULL;
+               snd_array_free(&codec->vendor_verbs);
        }
 }
 EXPORT_SYMBOL_GPL(snd_hdac_regmap_exit);
 
+/**
+ * snd_hdac_regmap_add_vendor_verb - add a vendor-specific verb to regmap
+ * @codec: the codec object
+ * @verb: verb to allow accessing via regmap
+ *
+ * Returns zero for success or a negative error code.
+ */
+int snd_hdac_regmap_add_vendor_verb(struct hdac_device *codec,
+                                   unsigned int verb)
+{
+       unsigned int *p = snd_array_new(&codec->vendor_verbs);
+
+       if (!p)
+               return -ENOMEM;
+       *p = verb;
+       return 0;
+}
+EXPORT_SYMBOL_GPL(snd_hdac_regmap_add_vendor_verb);
+
 /*
  * helper functions
  */