Merge remote-tracking branch 'asoc/topic/core' into asoc-next
authorMark Brown <broonie@kernel.org>
Mon, 8 Dec 2014 13:11:44 +0000 (13:11 +0000)
committerMark Brown <broonie@kernel.org>
Mon, 8 Dec 2014 13:11:44 +0000 (13:11 +0000)
1  2 
include/sound/soc.h
sound/soc/Makefile
sound/soc/soc-core.c
sound/soc/soc-pcm.c

diff --combined include/sound/soc.h
        {.reg = xreg, .rreg = xreg, .shift = shift_left, \
        .rshift = shift_right, .max = xmax, .platform_max = xmax, \
        .invert = xinvert, .autodisable = xautodisable})
+ #define SOC_DOUBLE_S_VALUE(xreg, shift_left, shift_right, xmin, xmax, xsign_bit, xinvert, xautodisable) \
+       ((unsigned long)&(struct soc_mixer_control) \
+       {.reg = xreg, .rreg = xreg, .shift = shift_left, \
+       .rshift = shift_right, .min = xmin, .max = xmax, .platform_max = xmax, \
+       .sign_bit = xsign_bit, .invert = xinvert, .autodisable = xautodisable})
  #define SOC_SINGLE_VALUE(xreg, xshift, xmax, xinvert, xautodisable) \
        SOC_DOUBLE_VALUE(xreg, xshift, xshift, xmax, xinvert, xautodisable)
  #define SOC_SINGLE_VALUE_EXT(xreg, xmax, xinvert) \
        .access = SNDRV_CTL_ELEM_ACCESS_TLV_READ | \
                  SNDRV_CTL_ELEM_ACCESS_READWRITE, \
        .tlv.p  = (tlv_array), \
-       .info   = snd_soc_info_volsw_s8, .get = snd_soc_get_volsw_s8, \
-       .put    = snd_soc_put_volsw_s8, \
-       .private_value = (unsigned long)&(struct soc_mixer_control) \
-               {.reg = xreg, .min = xmin, .max = xmax, \
-                .platform_max = xmax} }
+       .info = snd_soc_info_volsw, .get = snd_soc_get_volsw,\
+       .put = snd_soc_put_volsw, \
+       .private_value = SOC_DOUBLE_S_VALUE(xreg, 0, 8, xmin, xmax, 7, 0, 0) }
  #define SOC_ENUM_DOUBLE(xreg, xshift_l, xshift_r, xitems, xtexts) \
  {     .reg = xreg, .shift_l = xshift_l, .shift_r = xshift_r, \
        .items = xitems, .texts = xtexts, \
@@@ -409,9 -412,13 +412,9 @@@ int devm_snd_soc_register_component(str
                         const struct snd_soc_component_driver *cmpnt_drv,
                         struct snd_soc_dai_driver *dai_drv, int num_dai);
  void snd_soc_unregister_component(struct device *dev);
 -int snd_soc_cache_sync(struct snd_soc_codec *codec);
  int snd_soc_cache_init(struct snd_soc_codec *codec);
  int snd_soc_cache_exit(struct snd_soc_codec *codec);
 -int snd_soc_cache_write(struct snd_soc_codec *codec,
 -                      unsigned int reg, unsigned int value);
 -int snd_soc_cache_read(struct snd_soc_codec *codec,
 -                     unsigned int reg, unsigned int *value);
 +
  int snd_soc_platform_read(struct snd_soc_platform *platform,
                                        unsigned int reg);
  int snd_soc_platform_write(struct snd_soc_platform *platform,
@@@ -541,12 -548,6 +544,6 @@@ int snd_soc_get_volsw_sx(struct snd_kco
        struct snd_ctl_elem_value *ucontrol);
  int snd_soc_put_volsw_sx(struct snd_kcontrol *kcontrol,
        struct snd_ctl_elem_value *ucontrol);
- int snd_soc_info_volsw_s8(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_info *uinfo);
- int snd_soc_get_volsw_s8(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol);
- int snd_soc_put_volsw_s8(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol);
  int snd_soc_info_volsw_range(struct snd_kcontrol *kcontrol,
        struct snd_ctl_elem_info *uinfo);
  int snd_soc_put_volsw_range(struct snd_kcontrol *kcontrol,
@@@ -787,11 -788,13 +784,11 @@@ struct snd_soc_codec 
        unsigned int ac97_registered:1; /* Codec has been AC97 registered */
        unsigned int ac97_created:1; /* Codec has been created by SoC */
        unsigned int cache_init:1; /* codec cache has been initialized */
 -      u32 cache_sync; /* Cache needs to be synced to hardware */
  
        /* codec IO */
        void *control_data; /* codec control (i2c/3wire) data */
        hw_write_t hw_write;
        void *reg_cache;
 -      struct mutex cache_rw_mutex;
  
        /* component */
        struct snd_soc_component component;
@@@ -854,8 -857,6 +851,6 @@@ struct snd_soc_platform_driver 
  
        int (*probe)(struct snd_soc_platform *);
        int (*remove)(struct snd_soc_platform *);
-       int (*suspend)(struct snd_soc_dai *dai);
-       int (*resume)(struct snd_soc_dai *dai);
        struct snd_soc_component_driver component_driver;
  
        /* pcm creation and destruction */
  
  struct snd_soc_dai_link_component {
        const char *name;
-       const struct device_node *of_node;
+       struct device_node *of_node;
        const char *dai_name;
  };
  
@@@ -888,8 -889,6 +883,6 @@@ struct snd_soc_platform 
        struct device *dev;
        const struct snd_soc_platform_driver *driver;
  
-       unsigned int suspended:1; /* platform is suspended */
        struct list_head list;
  
        struct snd_soc_component component;
@@@ -984,7 -983,7 +977,7 @@@ struct snd_soc_codec_conf 
         * DT/OF node, but not both.
         */
        const char *dev_name;
-       const struct device_node *of_node;
+       struct device_node *of_node;
  
        /*
         * optional map of kcontrol, widget and path name prefixes that are
@@@ -1001,7 -1000,7 +994,7 @@@ struct snd_soc_aux_dev 
         * DT/OF node, but not both.
         */
        const char *codec_name;
-       const struct device_node *codec_of_node;
+       struct device_node *codec_of_node;
  
        /* codec/machine specific init - e.g. add machine controls */
        int (*init)(struct snd_soc_component *component);
@@@ -1258,17 -1257,6 +1251,17 @@@ unsigned int snd_soc_read(struct snd_so
  int snd_soc_write(struct snd_soc_codec *codec, unsigned int reg,
        unsigned int val);
  
 +/**
 + * snd_soc_cache_sync() - Sync the register cache with the hardware
 + * @codec: CODEC to sync
 + *
 + * Note: This function will call regcache_sync()
 + */
 +static inline int snd_soc_cache_sync(struct snd_soc_codec *codec)
 +{
 +      return regcache_sync(codec->component.regmap);
 +}
 +
  /* component IO */
  int snd_soc_component_read(struct snd_soc_component *component,
        unsigned int reg, unsigned int *val);
diff --combined sound/soc/Makefile
@@@ -1,5 -1,5 +1,5 @@@
  snd-soc-core-objs := soc-core.o soc-dapm.o soc-jack.o soc-cache.o soc-utils.o
- snd-soc-core-objs += soc-pcm.o soc-compress.o soc-io.o soc-devres.o
+ snd-soc-core-objs += soc-pcm.o soc-compress.o soc-io.o soc-devres.o soc-ops.o
  
  ifneq ($(CONFIG_SND_SOC_GENERIC_DMAENGINE_PCM),)
  snd-soc-core-objs += soc-generic-dmaengine-pcm.o
@@@ -26,6 -26,7 +26,6 @@@ obj-$(CONFIG_SND_SOC) += kirkwood
  obj-$(CONFIG_SND_SOC) += pxa/
  obj-$(CONFIG_SND_SOC) += rockchip/
  obj-$(CONFIG_SND_SOC) += samsung/
 -obj-$(CONFIG_SND_SOC) += s6000/
  obj-$(CONFIG_SND_SOC) += sh/
  obj-$(CONFIG_SND_SOC) += sirf/
  obj-$(CONFIG_SND_SOC) += spear/
diff --combined sound/soc/soc-core.c
@@@ -309,6 -309,9 +309,6 @@@ static void soc_init_codec_debugfs(stru
  {
        struct snd_soc_codec *codec = snd_soc_component_to_codec(component);
  
 -      debugfs_create_bool("cache_sync", 0444, codec->component.debugfs_root,
 -                          &codec->cache_sync);
 -
        codec->debugfs_reg = debugfs_create_file("codec_reg", 0644,
                                                 codec->component.debugfs_root,
                                                 codec, &codec_reg_fops);
@@@ -589,17 -592,12 +589,12 @@@ int snd_soc_suspend(struct device *dev
  
        for (i = 0; i < card->num_rtd; i++) {
                struct snd_soc_dai *cpu_dai = card->rtd[i].cpu_dai;
-               struct snd_soc_platform *platform = card->rtd[i].platform;
  
                if (card->rtd[i].dai_link->ignore_suspend)
                        continue;
  
                if (cpu_dai->driver->suspend && !cpu_dai->driver->ac97_control)
                        cpu_dai->driver->suspend(cpu_dai);
-               if (platform->driver->suspend && !platform->suspended) {
-                       platform->driver->suspend(cpu_dai);
-                       platform->suspended = 1;
-               }
        }
  
        /* close any waiting streams and save state */
                                          SND_SOC_DAPM_STREAM_SUSPEND);
        }
  
-       /* Recheck all analogue paths too */
-       dapm_mark_io_dirty(&card->dapm);
+       /* Recheck all endpoints too, their state is affected by suspend */
+       dapm_mark_endpoints_dirty(card);
        snd_soc_dapm_sync(&card->dapm);
  
        /* suspend all CODECs */
                                if (codec->driver->suspend)
                                        codec->driver->suspend(codec);
                                codec->suspended = 1;
 -                              codec->cache_sync = 1;
                                if (codec->component.regmap)
                                        regcache_mark_dirty(codec->component.regmap);
                                /* deactivate pins to sleep state */
@@@ -771,17 -770,12 +766,12 @@@ static void soc_resume_deferred(struct 
  
        for (i = 0; i < card->num_rtd; i++) {
                struct snd_soc_dai *cpu_dai = card->rtd[i].cpu_dai;
-               struct snd_soc_platform *platform = card->rtd[i].platform;
  
                if (card->rtd[i].dai_link->ignore_suspend)
                        continue;
  
                if (cpu_dai->driver->resume && !cpu_dai->driver->ac97_control)
                        cpu_dai->driver->resume(cpu_dai);
-               if (platform->driver->resume && platform->suspended) {
-                       platform->driver->resume(cpu_dai);
-                       platform->suspended = 0;
-               }
        }
  
        if (card->resume_post)
        /* userspace can access us now we are back as we were before */
        snd_power_change_state(card->snd_card, SNDRV_CTL_POWER_D0);
  
-       /* Recheck all analogue paths too */
-       dapm_mark_io_dirty(&card->dapm);
+       /* Recheck all endpoints too, their state is affected by suspend */
+       dapm_mark_endpoints_dirty(card);
        snd_soc_dapm_sync(&card->dapm);
  }
  
@@@ -880,7 -874,7 +870,7 @@@ static struct snd_soc_dai *snd_soc_find
        list_for_each_entry(component, &component_list, list) {
                if (dlc->of_node && component->dev->of_node != dlc->of_node)
                        continue;
 -              if (dlc->name && strcmp(dev_name(component->dev), dlc->name))
 +              if (dlc->name && strcmp(component->name, dlc->name))
                        continue;
                list_for_each_entry(dai, &component->dai_list, list) {
                        if (dlc->dai_name && strcmp(dai->name, dlc->dai_name))
@@@ -1247,25 -1241,22 +1237,22 @@@ static int soc_probe_link_components(st
        return 0;
  }
  
- static int soc_probe_codec_dai(struct snd_soc_card *card,
-                              struct snd_soc_dai *codec_dai,
-                              int order)
+ static int soc_probe_dai(struct snd_soc_dai *dai, int order)
  {
        int ret;
  
-       if (!codec_dai->probed && codec_dai->driver->probe_order == order) {
-               if (codec_dai->driver->probe) {
-                       ret = codec_dai->driver->probe(codec_dai);
+       if (!dai->probed && dai->driver->probe_order == order) {
+               if (dai->driver->probe) {
+                       ret = dai->driver->probe(dai);
                        if (ret < 0) {
-                               dev_err(codec_dai->dev,
-                                       "ASoC: failed to probe CODEC DAI %s: %d\n",
-                                       codec_dai->name, ret);
+                               dev_err(dai->dev,
+                                       "ASoC: failed to probe DAI %s: %d\n",
+                                       dai->name, ret);
                                return ret;
                        }
                }
  
-               /* mark codec_dai as probed and add to card dai list */
-               codec_dai->probed = 1;
+               dai->probed = 1;
        }
  
        return 0;
@@@ -1315,40 -1306,22 +1302,22 @@@ static int soc_probe_link_dais(struct s
  {
        struct snd_soc_dai_link *dai_link = &card->dai_link[num];
        struct snd_soc_pcm_runtime *rtd = &card->rtd[num];
-       struct snd_soc_platform *platform = rtd->platform;
        struct snd_soc_dai *cpu_dai = rtd->cpu_dai;
        int i, ret;
  
        dev_dbg(card->dev, "ASoC: probe %s dai link %d late %d\n",
                        card->name, num, order);
  
-       /* config components */
-       cpu_dai->platform = platform;
-       cpu_dai->card = card;
-       for (i = 0; i < rtd->num_codecs; i++)
-               rtd->codec_dais[i]->card = card;
        /* set default power off timeout */
        rtd->pmdown_time = pmdown_time;
  
-       /* probe the cpu_dai */
-       if (!cpu_dai->probed &&
-                       cpu_dai->driver->probe_order == order) {
-               if (cpu_dai->driver->probe) {
-                       ret = cpu_dai->driver->probe(cpu_dai);
-                       if (ret < 0) {
-                               dev_err(cpu_dai->dev,
-                                       "ASoC: failed to probe CPU DAI %s: %d\n",
-                                       cpu_dai->name, ret);
-                               return ret;
-                       }
-               }
-               cpu_dai->probed = 1;
-       }
+       ret = soc_probe_dai(cpu_dai, order);
+       if (ret)
+               return ret;
  
        /* probe the CODEC DAI */
        for (i = 0; i < rtd->num_codecs; i++) {
-               ret = soc_probe_codec_dai(card, rtd->codec_dais[i], order);
+               ret = soc_probe_dai(rtd->codec_dais[i], order);
                if (ret)
                        return ret;
        }
@@@ -2322,1027 -2295,13 +2291,13 @@@ EXPORT_SYMBOL_GPL(snd_soc_add_card_cont
  int snd_soc_add_dai_controls(struct snd_soc_dai *dai,
        const struct snd_kcontrol_new *controls, int num_controls)
  {
-       struct snd_card *card = dai->card->snd_card;
+       struct snd_card *card = dai->component->card->snd_card;
  
        return snd_soc_add_controls(card, dai->dev, controls, num_controls,
                        NULL, dai);
  }
  EXPORT_SYMBOL_GPL(snd_soc_add_dai_controls);
  
- /**
-  * snd_soc_info_enum_double - enumerated double mixer info callback
-  * @kcontrol: mixer control
-  * @uinfo: control element information
-  *
-  * Callback to provide information about a double enumerated
-  * mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_info_enum_double(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_info *uinfo)
- {
-       struct soc_enum *e = (struct soc_enum *)kcontrol->private_value;
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED;
-       uinfo->count = e->shift_l == e->shift_r ? 1 : 2;
-       uinfo->value.enumerated.items = e->items;
-       if (uinfo->value.enumerated.item >= e->items)
-               uinfo->value.enumerated.item = e->items - 1;
-       strlcpy(uinfo->value.enumerated.name,
-               e->texts[uinfo->value.enumerated.item],
-               sizeof(uinfo->value.enumerated.name));
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_info_enum_double);
- /**
-  * snd_soc_get_enum_double - enumerated double mixer get callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to get the value of a double enumerated mixer.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_enum_double(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_enum *e = (struct soc_enum *)kcontrol->private_value;
-       unsigned int val, item;
-       unsigned int reg_val;
-       int ret;
-       ret = snd_soc_component_read(component, e->reg, &reg_val);
-       if (ret)
-               return ret;
-       val = (reg_val >> e->shift_l) & e->mask;
-       item = snd_soc_enum_val_to_item(e, val);
-       ucontrol->value.enumerated.item[0] = item;
-       if (e->shift_l != e->shift_r) {
-               val = (reg_val >> e->shift_l) & e->mask;
-               item = snd_soc_enum_val_to_item(e, val);
-               ucontrol->value.enumerated.item[1] = item;
-       }
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_enum_double);
- /**
-  * snd_soc_put_enum_double - enumerated double mixer put callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to set the value of a double enumerated mixer.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_put_enum_double(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_enum *e = (struct soc_enum *)kcontrol->private_value;
-       unsigned int *item = ucontrol->value.enumerated.item;
-       unsigned int val;
-       unsigned int mask;
-       if (item[0] >= e->items)
-               return -EINVAL;
-       val = snd_soc_enum_item_to_val(e, item[0]) << e->shift_l;
-       mask = e->mask << e->shift_l;
-       if (e->shift_l != e->shift_r) {
-               if (item[1] >= e->items)
-                       return -EINVAL;
-               val |= snd_soc_enum_item_to_val(e, item[1]) << e->shift_r;
-               mask |= e->mask << e->shift_r;
-       }
-       return snd_soc_component_update_bits(component, e->reg, mask, val);
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_enum_double);
- /**
-  * snd_soc_read_signed - Read a codec register and interprete as signed value
-  * @component: component
-  * @reg: Register to read
-  * @mask: Mask to use after shifting the register value
-  * @shift: Right shift of register value
-  * @sign_bit: Bit that describes if a number is negative or not.
-  * @signed_val: Pointer to where the read value should be stored
-  *
-  * This functions reads a codec register. The register value is shifted right
-  * by 'shift' bits and masked with the given 'mask'. Afterwards it translates
-  * the given registervalue into a signed integer if sign_bit is non-zero.
-  *
-  * Returns 0 on sucess, otherwise an error value
-  */
- static int snd_soc_read_signed(struct snd_soc_component *component,
-       unsigned int reg, unsigned int mask, unsigned int shift,
-       unsigned int sign_bit, int *signed_val)
- {
-       int ret;
-       unsigned int val;
-       ret = snd_soc_component_read(component, reg, &val);
-       if (ret < 0)
-               return ret;
-       val = (val >> shift) & mask;
-       if (!sign_bit) {
-               *signed_val = val;
-               return 0;
-       }
-       /* non-negative number */
-       if (!(val & BIT(sign_bit))) {
-               *signed_val = val;
-               return 0;
-       }
-       ret = val;
-       /*
-        * The register most probably does not contain a full-sized int.
-        * Instead we have an arbitrary number of bits in a signed
-        * representation which has to be translated into a full-sized int.
-        * This is done by filling up all bits above the sign-bit.
-        */
-       ret |= ~((int)(BIT(sign_bit) - 1));
-       *signed_val = ret;
-       return 0;
- }
- /**
-  * snd_soc_info_volsw - single mixer info callback
-  * @kcontrol: mixer control
-  * @uinfo: control element information
-  *
-  * Callback to provide information about a single mixer control, or a double
-  * mixer control that spans 2 registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_info_volsw(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_info *uinfo)
- {
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       int platform_max;
-       if (!mc->platform_max)
-               mc->platform_max = mc->max;
-       platform_max = mc->platform_max;
-       if (platform_max == 1 && !strstr(kcontrol->id.name, " Volume"))
-               uinfo->type = SNDRV_CTL_ELEM_TYPE_BOOLEAN;
-       else
-               uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER;
-       uinfo->count = snd_soc_volsw_is_stereo(mc) ? 2 : 1;
-       uinfo->value.integer.min = 0;
-       uinfo->value.integer.max = platform_max - mc->min;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_info_volsw);
- /**
-  * snd_soc_get_volsw - single mixer get callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to get the value of a single mixer control, or a double mixer
-  * control that spans 2 registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_volsw(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int reg2 = mc->rreg;
-       unsigned int shift = mc->shift;
-       unsigned int rshift = mc->rshift;
-       int max = mc->max;
-       int min = mc->min;
-       int sign_bit = mc->sign_bit;
-       unsigned int mask = (1 << fls(max)) - 1;
-       unsigned int invert = mc->invert;
-       int val;
-       int ret;
-       if (sign_bit)
-               mask = BIT(sign_bit + 1) - 1;
-       ret = snd_soc_read_signed(component, reg, mask, shift, sign_bit, &val);
-       if (ret)
-               return ret;
-       ucontrol->value.integer.value[0] = val - min;
-       if (invert)
-               ucontrol->value.integer.value[0] =
-                       max - ucontrol->value.integer.value[0];
-       if (snd_soc_volsw_is_stereo(mc)) {
-               if (reg == reg2)
-                       ret = snd_soc_read_signed(component, reg, mask, rshift,
-                               sign_bit, &val);
-               else
-                       ret = snd_soc_read_signed(component, reg2, mask, shift,
-                               sign_bit, &val);
-               if (ret)
-                       return ret;
-               ucontrol->value.integer.value[1] = val - min;
-               if (invert)
-                       ucontrol->value.integer.value[1] =
-                               max - ucontrol->value.integer.value[1];
-       }
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_volsw);
- /**
-  * snd_soc_put_volsw - single mixer put callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to set the value of a single mixer control, or a double mixer
-  * control that spans 2 registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_put_volsw(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int reg2 = mc->rreg;
-       unsigned int shift = mc->shift;
-       unsigned int rshift = mc->rshift;
-       int max = mc->max;
-       int min = mc->min;
-       unsigned int sign_bit = mc->sign_bit;
-       unsigned int mask = (1 << fls(max)) - 1;
-       unsigned int invert = mc->invert;
-       int err;
-       bool type_2r = false;
-       unsigned int val2 = 0;
-       unsigned int val, val_mask;
-       if (sign_bit)
-               mask = BIT(sign_bit + 1) - 1;
-       val = ((ucontrol->value.integer.value[0] + min) & mask);
-       if (invert)
-               val = max - val;
-       val_mask = mask << shift;
-       val = val << shift;
-       if (snd_soc_volsw_is_stereo(mc)) {
-               val2 = ((ucontrol->value.integer.value[1] + min) & mask);
-               if (invert)
-                       val2 = max - val2;
-               if (reg == reg2) {
-                       val_mask |= mask << rshift;
-                       val |= val2 << rshift;
-               } else {
-                       val2 = val2 << shift;
-                       type_2r = true;
-               }
-       }
-       err = snd_soc_component_update_bits(component, reg, val_mask, val);
-       if (err < 0)
-               return err;
-       if (type_2r)
-               err = snd_soc_component_update_bits(component, reg2, val_mask,
-                       val2);
-       return err;
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_volsw);
- /**
-  * snd_soc_get_volsw_sx - single mixer get callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to get the value of a single mixer control, or a double mixer
-  * control that spans 2 registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_volsw_sx(struct snd_kcontrol *kcontrol,
-                     struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-           (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int reg2 = mc->rreg;
-       unsigned int shift = mc->shift;
-       unsigned int rshift = mc->rshift;
-       int max = mc->max;
-       int min = mc->min;
-       int mask = (1 << (fls(min + max) - 1)) - 1;
-       unsigned int val;
-       int ret;
-       ret = snd_soc_component_read(component, reg, &val);
-       if (ret < 0)
-               return ret;
-       ucontrol->value.integer.value[0] = ((val >> shift) - min) & mask;
-       if (snd_soc_volsw_is_stereo(mc)) {
-               ret = snd_soc_component_read(component, reg2, &val);
-               if (ret < 0)
-                       return ret;
-               val = ((val >> rshift) - min) & mask;
-               ucontrol->value.integer.value[1] = val;
-       }
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_volsw_sx);
- /**
-  * snd_soc_put_volsw_sx - double mixer set callback
-  * @kcontrol: mixer control
-  * @uinfo: control element information
-  *
-  * Callback to set the value of a double mixer control that spans 2 registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_put_volsw_sx(struct snd_kcontrol *kcontrol,
-                        struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-           (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int reg2 = mc->rreg;
-       unsigned int shift = mc->shift;
-       unsigned int rshift = mc->rshift;
-       int max = mc->max;
-       int min = mc->min;
-       int mask = (1 << (fls(min + max) - 1)) - 1;
-       int err = 0;
-       unsigned int val, val_mask, val2 = 0;
-       val_mask = mask << shift;
-       val = (ucontrol->value.integer.value[0] + min) & mask;
-       val = val << shift;
-       err = snd_soc_component_update_bits(component, reg, val_mask, val);
-       if (err < 0)
-               return err;
-       if (snd_soc_volsw_is_stereo(mc)) {
-               val_mask = mask << rshift;
-               val2 = (ucontrol->value.integer.value[1] + min) & mask;
-               val2 = val2 << rshift;
-               err = snd_soc_component_update_bits(component, reg2, val_mask,
-                       val2);
-       }
-       return err;
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_volsw_sx);
- /**
-  * snd_soc_info_volsw_s8 - signed mixer info callback
-  * @kcontrol: mixer control
-  * @uinfo: control element information
-  *
-  * Callback to provide information about a signed mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_info_volsw_s8(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_info *uinfo)
- {
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       int platform_max;
-       int min = mc->min;
-       if (!mc->platform_max)
-               mc->platform_max = mc->max;
-       platform_max = mc->platform_max;
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER;
-       uinfo->count = 2;
-       uinfo->value.integer.min = 0;
-       uinfo->value.integer.max = platform_max - min;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_info_volsw_s8);
- /**
-  * snd_soc_get_volsw_s8 - signed mixer get callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to get the value of a signed mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_volsw_s8(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       unsigned int reg = mc->reg;
-       unsigned int val;
-       int min = mc->min;
-       int ret;
-       ret = snd_soc_component_read(component, reg, &val);
-       if (ret)
-               return ret;
-       ucontrol->value.integer.value[0] =
-               ((signed char)(val & 0xff))-min;
-       ucontrol->value.integer.value[1] =
-               ((signed char)((val >> 8) & 0xff))-min;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_volsw_s8);
- /**
-  * snd_soc_put_volsw_sgn - signed mixer put callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to set the value of a signed mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_put_volsw_s8(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       unsigned int reg = mc->reg;
-       int min = mc->min;
-       unsigned int val;
-       val = (ucontrol->value.integer.value[0]+min) & 0xff;
-       val |= ((ucontrol->value.integer.value[1]+min) & 0xff) << 8;
-       return snd_soc_component_update_bits(component, reg, 0xffff, val);
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_volsw_s8);
- /**
-  * snd_soc_info_volsw_range - single mixer info callback with range.
-  * @kcontrol: mixer control
-  * @uinfo: control element information
-  *
-  * Callback to provide information, within a range, about a single
-  * mixer control.
-  *
-  * returns 0 for success.
-  */
- int snd_soc_info_volsw_range(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_info *uinfo)
- {
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       int platform_max;
-       int min = mc->min;
-       if (!mc->platform_max)
-               mc->platform_max = mc->max;
-       platform_max = mc->platform_max;
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER;
-       uinfo->count = snd_soc_volsw_is_stereo(mc) ? 2 : 1;
-       uinfo->value.integer.min = 0;
-       uinfo->value.integer.max = platform_max - min;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_info_volsw_range);
- /**
-  * snd_soc_put_volsw_range - single mixer put value callback with range.
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to set the value, within a range, for a single mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_put_volsw_range(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       unsigned int reg = mc->reg;
-       unsigned int rreg = mc->rreg;
-       unsigned int shift = mc->shift;
-       int min = mc->min;
-       int max = mc->max;
-       unsigned int mask = (1 << fls(max)) - 1;
-       unsigned int invert = mc->invert;
-       unsigned int val, val_mask;
-       int ret;
-       if (invert)
-               val = (max - ucontrol->value.integer.value[0]) & mask;
-       else
-               val = ((ucontrol->value.integer.value[0] + min) & mask);
-       val_mask = mask << shift;
-       val = val << shift;
-       ret = snd_soc_component_update_bits(component, reg, val_mask, val);
-       if (ret < 0)
-               return ret;
-       if (snd_soc_volsw_is_stereo(mc)) {
-               if (invert)
-                       val = (max - ucontrol->value.integer.value[1]) & mask;
-               else
-                       val = ((ucontrol->value.integer.value[1] + min) & mask);
-               val_mask = mask << shift;
-               val = val << shift;
-               ret = snd_soc_component_update_bits(component, rreg, val_mask,
-                       val);
-       }
-       return ret;
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_volsw_range);
- /**
-  * snd_soc_get_volsw_range - single mixer get callback with range
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback to get the value, within a range, of a single mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_volsw_range(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int rreg = mc->rreg;
-       unsigned int shift = mc->shift;
-       int min = mc->min;
-       int max = mc->max;
-       unsigned int mask = (1 << fls(max)) - 1;
-       unsigned int invert = mc->invert;
-       unsigned int val;
-       int ret;
-       ret = snd_soc_component_read(component, reg, &val);
-       if (ret)
-               return ret;
-       ucontrol->value.integer.value[0] = (val >> shift) & mask;
-       if (invert)
-               ucontrol->value.integer.value[0] =
-                       max - ucontrol->value.integer.value[0];
-       else
-               ucontrol->value.integer.value[0] =
-                       ucontrol->value.integer.value[0] - min;
-       if (snd_soc_volsw_is_stereo(mc)) {
-               ret = snd_soc_component_read(component, rreg, &val);
-               if (ret)
-                       return ret;
-               ucontrol->value.integer.value[1] = (val >> shift) & mask;
-               if (invert)
-                       ucontrol->value.integer.value[1] =
-                               max - ucontrol->value.integer.value[1];
-               else
-                       ucontrol->value.integer.value[1] =
-                               ucontrol->value.integer.value[1] - min;
-       }
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_volsw_range);
- /**
-  * snd_soc_limit_volume - Set new limit to an existing volume control.
-  *
-  * @codec: where to look for the control
-  * @name: Name of the control
-  * @max: new maximum limit
-  *
-  * Return 0 for success, else error.
-  */
- int snd_soc_limit_volume(struct snd_soc_codec *codec,
-       const char *name, int max)
- {
-       struct snd_card *card = codec->component.card->snd_card;
-       struct snd_kcontrol *kctl;
-       struct soc_mixer_control *mc;
-       int found = 0;
-       int ret = -EINVAL;
-       /* Sanity check for name and max */
-       if (unlikely(!name || max <= 0))
-               return -EINVAL;
-       list_for_each_entry(kctl, &card->controls, list) {
-               if (!strncmp(kctl->id.name, name, sizeof(kctl->id.name))) {
-                       found = 1;
-                       break;
-               }
-       }
-       if (found) {
-               mc = (struct soc_mixer_control *)kctl->private_value;
-               if (max <= mc->max) {
-                       mc->platform_max = max;
-                       ret = 0;
-               }
-       }
-       return ret;
- }
- EXPORT_SYMBOL_GPL(snd_soc_limit_volume);
- int snd_soc_bytes_info(struct snd_kcontrol *kcontrol,
-                      struct snd_ctl_elem_info *uinfo)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_bytes *params = (void *)kcontrol->private_value;
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_BYTES;
-       uinfo->count = params->num_regs * component->val_bytes;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_bytes_info);
- int snd_soc_bytes_get(struct snd_kcontrol *kcontrol,
-                     struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_bytes *params = (void *)kcontrol->private_value;
-       int ret;
-       if (component->regmap)
-               ret = regmap_raw_read(component->regmap, params->base,
-                                     ucontrol->value.bytes.data,
-                                     params->num_regs * component->val_bytes);
-       else
-               ret = -EINVAL;
-       /* Hide any masked bytes to ensure consistent data reporting */
-       if (ret == 0 && params->mask) {
-               switch (component->val_bytes) {
-               case 1:
-                       ucontrol->value.bytes.data[0] &= ~params->mask;
-                       break;
-               case 2:
-                       ((u16 *)(&ucontrol->value.bytes.data))[0]
-                               &= cpu_to_be16(~params->mask);
-                       break;
-               case 4:
-                       ((u32 *)(&ucontrol->value.bytes.data))[0]
-                               &= cpu_to_be32(~params->mask);
-                       break;
-               default:
-                       return -EINVAL;
-               }
-       }
-       return ret;
- }
- EXPORT_SYMBOL_GPL(snd_soc_bytes_get);
- int snd_soc_bytes_put(struct snd_kcontrol *kcontrol,
-                     struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_bytes *params = (void *)kcontrol->private_value;
-       int ret, len;
-       unsigned int val, mask;
-       void *data;
-       if (!component->regmap || !params->num_regs)
-               return -EINVAL;
-       len = params->num_regs * component->val_bytes;
-       data = kmemdup(ucontrol->value.bytes.data, len, GFP_KERNEL | GFP_DMA);
-       if (!data)
-               return -ENOMEM;
-       /*
-        * If we've got a mask then we need to preserve the register
-        * bits.  We shouldn't modify the incoming data so take a
-        * copy.
-        */
-       if (params->mask) {
-               ret = regmap_read(component->regmap, params->base, &val);
-               if (ret != 0)
-                       goto out;
-               val &= params->mask;
-               switch (component->val_bytes) {
-               case 1:
-                       ((u8 *)data)[0] &= ~params->mask;
-                       ((u8 *)data)[0] |= val;
-                       break;
-               case 2:
-                       mask = ~params->mask;
-                       ret = regmap_parse_val(component->regmap,
-                                                       &mask, &mask);
-                       if (ret != 0)
-                               goto out;
-                       ((u16 *)data)[0] &= mask;
-                       ret = regmap_parse_val(component->regmap,
-                                                       &val, &val);
-                       if (ret != 0)
-                               goto out;
-                       ((u16 *)data)[0] |= val;
-                       break;
-               case 4:
-                       mask = ~params->mask;
-                       ret = regmap_parse_val(component->regmap,
-                                                       &mask, &mask);
-                       if (ret != 0)
-                               goto out;
-                       ((u32 *)data)[0] &= mask;
-                       ret = regmap_parse_val(component->regmap,
-                                                       &val, &val);
-                       if (ret != 0)
-                               goto out;
-                       ((u32 *)data)[0] |= val;
-                       break;
-               default:
-                       ret = -EINVAL;
-                       goto out;
-               }
-       }
-       ret = regmap_raw_write(component->regmap, params->base,
-                              data, len);
- out:
-       kfree(data);
-       return ret;
- }
- EXPORT_SYMBOL_GPL(snd_soc_bytes_put);
- int snd_soc_bytes_info_ext(struct snd_kcontrol *kcontrol,
-                       struct snd_ctl_elem_info *ucontrol)
- {
-       struct soc_bytes_ext *params = (void *)kcontrol->private_value;
-       ucontrol->type = SNDRV_CTL_ELEM_TYPE_BYTES;
-       ucontrol->count = params->max;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_bytes_info_ext);
- int snd_soc_bytes_tlv_callback(struct snd_kcontrol *kcontrol, int op_flag,
-                               unsigned int size, unsigned int __user *tlv)
- {
-       struct soc_bytes_ext *params = (void *)kcontrol->private_value;
-       unsigned int count = size < params->max ? size : params->max;
-       int ret = -ENXIO;
-       switch (op_flag) {
-       case SNDRV_CTL_TLV_OP_READ:
-               if (params->get)
-                       ret = params->get(tlv, count);
-               break;
-       case SNDRV_CTL_TLV_OP_WRITE:
-               if (params->put)
-                       ret = params->put(tlv, count);
-               break;
-       }
-       return ret;
- }
- EXPORT_SYMBOL_GPL(snd_soc_bytes_tlv_callback);
- /**
-  * snd_soc_info_xr_sx - signed multi register info callback
-  * @kcontrol: mreg control
-  * @uinfo: control element information
-  *
-  * Callback to provide information of a control that can
-  * span multiple codec registers which together
-  * forms a single signed value in a MSB/LSB manner.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_info_xr_sx(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_info *uinfo)
- {
-       struct soc_mreg_control *mc =
-               (struct soc_mreg_control *)kcontrol->private_value;
-       uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER;
-       uinfo->count = 1;
-       uinfo->value.integer.min = mc->min;
-       uinfo->value.integer.max = mc->max;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_info_xr_sx);
- /**
-  * snd_soc_get_xr_sx - signed multi register get callback
-  * @kcontrol: mreg control
-  * @ucontrol: control element information
-  *
-  * Callback to get the value of a control that can span
-  * multiple codec registers which together forms a single
-  * signed value in a MSB/LSB manner. The control supports
-  * specifying total no of bits used to allow for bitfields
-  * across the multiple codec registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_xr_sx(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mreg_control *mc =
-               (struct soc_mreg_control *)kcontrol->private_value;
-       unsigned int regbase = mc->regbase;
-       unsigned int regcount = mc->regcount;
-       unsigned int regwshift = component->val_bytes * BITS_PER_BYTE;
-       unsigned int regwmask = (1<<regwshift)-1;
-       unsigned int invert = mc->invert;
-       unsigned long mask = (1UL<<mc->nbits)-1;
-       long min = mc->min;
-       long max = mc->max;
-       long val = 0;
-       unsigned int regval;
-       unsigned int i;
-       int ret;
-       for (i = 0; i < regcount; i++) {
-               ret = snd_soc_component_read(component, regbase+i, &regval);
-               if (ret)
-                       return ret;
-               val |= (regval & regwmask) << (regwshift*(regcount-i-1));
-       }
-       val &= mask;
-       if (min < 0 && val > max)
-               val |= ~mask;
-       if (invert)
-               val = max - val;
-       ucontrol->value.integer.value[0] = val;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_xr_sx);
- /**
-  * snd_soc_put_xr_sx - signed multi register get callback
-  * @kcontrol: mreg control
-  * @ucontrol: control element information
-  *
-  * Callback to set the value of a control that can span
-  * multiple codec registers which together forms a single
-  * signed value in a MSB/LSB manner. The control supports
-  * specifying total no of bits used to allow for bitfields
-  * across the multiple codec registers.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_put_xr_sx(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mreg_control *mc =
-               (struct soc_mreg_control *)kcontrol->private_value;
-       unsigned int regbase = mc->regbase;
-       unsigned int regcount = mc->regcount;
-       unsigned int regwshift = component->val_bytes * BITS_PER_BYTE;
-       unsigned int regwmask = (1<<regwshift)-1;
-       unsigned int invert = mc->invert;
-       unsigned long mask = (1UL<<mc->nbits)-1;
-       long max = mc->max;
-       long val = ucontrol->value.integer.value[0];
-       unsigned int i, regval, regmask;
-       int err;
-       if (invert)
-               val = max - val;
-       val &= mask;
-       for (i = 0; i < regcount; i++) {
-               regval = (val >> (regwshift*(regcount-i-1))) & regwmask;
-               regmask = (mask >> (regwshift*(regcount-i-1))) & regwmask;
-               err = snd_soc_component_update_bits(component, regbase+i,
-                               regmask, regval);
-               if (err < 0)
-                       return err;
-       }
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_xr_sx);
- /**
-  * snd_soc_get_strobe - strobe get callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback get the value of a strobe mixer control.
-  *
-  * Returns 0 for success.
-  */
- int snd_soc_get_strobe(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int shift = mc->shift;
-       unsigned int mask = 1 << shift;
-       unsigned int invert = mc->invert != 0;
-       unsigned int val;
-       int ret;
-       ret = snd_soc_component_read(component, reg, &val);
-       if (ret)
-               return ret;
-       val &= mask;
-       if (shift != 0 && val != 0)
-               val = val >> shift;
-       ucontrol->value.enumerated.item[0] = val ^ invert;
-       return 0;
- }
- EXPORT_SYMBOL_GPL(snd_soc_get_strobe);
- /**
-  * snd_soc_put_strobe - strobe put callback
-  * @kcontrol: mixer control
-  * @ucontrol: control element information
-  *
-  * Callback strobe a register bit to high then low (or the inverse)
-  * in one pass of a single mixer enum control.
-  *
-  * Returns 1 for success.
-  */
- int snd_soc_put_strobe(struct snd_kcontrol *kcontrol,
-       struct snd_ctl_elem_value *ucontrol)
- {
-       struct snd_soc_component *component = snd_kcontrol_chip(kcontrol);
-       struct soc_mixer_control *mc =
-               (struct soc_mixer_control *)kcontrol->private_value;
-       unsigned int reg = mc->reg;
-       unsigned int shift = mc->shift;
-       unsigned int mask = 1 << shift;
-       unsigned int invert = mc->invert != 0;
-       unsigned int strobe = ucontrol->value.enumerated.item[0] != 0;
-       unsigned int val1 = (strobe ^ invert) ? mask : 0;
-       unsigned int val2 = (strobe ^ invert) ? 0 : mask;
-       int err;
-       err = snd_soc_component_update_bits(component, reg, mask, val1);
-       if (err < 0)
-               return err;
-       return snd_soc_component_update_bits(component, reg, mask, val2);
- }
- EXPORT_SYMBOL_GPL(snd_soc_put_strobe);
  /**
   * snd_soc_dai_set_sysclk - configure DAI system or master clock.
   * @dai: DAI
diff --combined sound/soc/soc-pcm.c
@@@ -654,6 -654,8 +654,8 @@@ static int soc_pcm_close(struct snd_pcm
                        codec_dai->rate = 0;
        }
  
+       snd_soc_dai_digital_mute(cpu_dai, 1, substream->stream);
        if (cpu_dai->driver->ops->shutdown)
                cpu_dai->driver->ops->shutdown(substream, cpu_dai);
  
@@@ -772,6 -774,7 +774,7 @@@ static int soc_pcm_prepare(struct snd_p
        for (i = 0; i < rtd->num_codecs; i++)
                snd_soc_dai_digital_mute(rtd->codec_dais[i], 0,
                                         substream->stream);
+       snd_soc_dai_digital_mute(cpu_dai, 0, substream->stream);
  
  out:
        mutex_unlock(&rtd->pcm_mutex);
@@@ -1522,36 -1525,13 +1525,36 @@@ static void dpcm_set_fe_runtime(struct 
                dpcm_init_runtime_hw(runtime, &cpu_dai_drv->capture);
  }
  
 +static int dpcm_fe_dai_do_trigger(struct snd_pcm_substream *substream, int cmd);
 +
 +/* Set FE's runtime_update state; the state is protected via PCM stream lock
 + * for avoiding the race with trigger callback.
 + * If the state is unset and a trigger is pending while the previous operation,
 + * process the pending trigger action here.
 + */
 +static void dpcm_set_fe_update_state(struct snd_soc_pcm_runtime *fe,
 +                                   int stream, enum snd_soc_dpcm_update state)
 +{
 +      struct snd_pcm_substream *substream =
 +              snd_soc_dpcm_get_substream(fe, stream);
 +
 +      snd_pcm_stream_lock_irq(substream);
 +      if (state == SND_SOC_DPCM_UPDATE_NO && fe->dpcm[stream].trigger_pending) {
 +              dpcm_fe_dai_do_trigger(substream,
 +                                     fe->dpcm[stream].trigger_pending - 1);
 +              fe->dpcm[stream].trigger_pending = 0;
 +      }
 +      fe->dpcm[stream].runtime_update = state;
 +      snd_pcm_stream_unlock_irq(substream);
 +}
 +
  static int dpcm_fe_dai_startup(struct snd_pcm_substream *fe_substream)
  {
        struct snd_soc_pcm_runtime *fe = fe_substream->private_data;
        struct snd_pcm_runtime *runtime = fe_substream->runtime;
        int stream = fe_substream->stream, ret = 0;
  
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_FE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_FE);
  
        ret = dpcm_be_dai_startup(fe, fe_substream->stream);
        if (ret < 0) {
        dpcm_set_fe_runtime(fe_substream);
        snd_pcm_limit_hw_rates(runtime);
  
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
        return 0;
  
  unwind:
        dpcm_be_dai_startup_unwind(fe, fe_substream->stream);
  be_err:
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
        return ret;
  }
  
@@@ -1626,7 -1606,7 +1629,7 @@@ static int dpcm_fe_dai_shutdown(struct 
        struct snd_soc_pcm_runtime *fe = substream->private_data;
        int stream = substream->stream;
  
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_FE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_FE);
  
        /* shutdown the BEs */
        dpcm_be_dai_shutdown(fe, substream->stream);
        dpcm_dapm_stream_event(fe, stream, SND_SOC_DAPM_STREAM_STOP);
  
        fe->dpcm[stream].state = SND_SOC_DPCM_STATE_CLOSE;
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
        return 0;
  }
  
@@@ -1664,10 -1644,6 +1667,10 @@@ int dpcm_be_dai_hw_free(struct snd_soc_
                if (!snd_soc_dpcm_can_be_free_stop(fe, be, stream))
                                continue;
  
 +              /* do not free hw if this BE is used by other FE */
 +              if (be->dpcm[stream].users > 1)
 +                      continue;
 +
                if ((be->dpcm[stream].state != SND_SOC_DPCM_STATE_HW_PARAMS) &&
                    (be->dpcm[stream].state != SND_SOC_DPCM_STATE_PREPARE) &&
                    (be->dpcm[stream].state != SND_SOC_DPCM_STATE_HW_FREE) &&
@@@ -1692,7 -1668,7 +1695,7 @@@ static int dpcm_fe_dai_hw_free(struct s
        int err, stream = substream->stream;
  
        mutex_lock_nested(&fe->card->mutex, SND_SOC_CARD_CLASS_RUNTIME);
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_FE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_FE);
  
        dev_dbg(fe->dev, "ASoC: hw_free FE %s\n", fe->dai_link->name);
  
        err = dpcm_be_dai_hw_free(fe, stream);
  
        fe->dpcm[stream].state = SND_SOC_DPCM_STATE_HW_FREE;
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
  
        mutex_unlock(&fe->card->mutex);
        return 0;
@@@ -1800,7 -1776,7 +1803,7 @@@ static int dpcm_fe_dai_hw_params(struc
        int ret, stream = substream->stream;
  
        mutex_lock_nested(&fe->card->mutex, SND_SOC_CARD_CLASS_RUNTIME);
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_FE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_FE);
  
        memcpy(&fe->dpcm[substream->stream].hw_params, params,
                        sizeof(struct snd_pcm_hw_params));
                fe->dpcm[stream].state = SND_SOC_DPCM_STATE_HW_PARAMS;
  
  out:
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
        mutex_unlock(&fe->card->mutex);
        return ret;
  }
@@@ -1937,7 -1913,7 +1940,7 @@@ int dpcm_be_dai_trigger(struct snd_soc_
  }
  EXPORT_SYMBOL_GPL(dpcm_be_dai_trigger);
  
 -static int dpcm_fe_dai_trigger(struct snd_pcm_substream *substream, int cmd)
 +static int dpcm_fe_dai_do_trigger(struct snd_pcm_substream *substream, int cmd)
  {
        struct snd_soc_pcm_runtime *fe = substream->private_data;
        int stream = substream->stream, ret;
        return ret;
  }
  
 +static int dpcm_fe_dai_trigger(struct snd_pcm_substream *substream, int cmd)
 +{
 +      struct snd_soc_pcm_runtime *fe = substream->private_data;
 +      int stream = substream->stream;
 +
 +      /* if FE's runtime_update is already set, we're in race;
 +       * process this trigger later at exit
 +       */
 +      if (fe->dpcm[stream].runtime_update != SND_SOC_DPCM_UPDATE_NO) {
 +              fe->dpcm[stream].trigger_pending = cmd + 1;
 +              return 0; /* delayed, assuming it's successful */
 +      }
 +
 +      /* we're alone, let's trigger */
 +      return dpcm_fe_dai_do_trigger(substream, cmd);
 +}
 +
  int dpcm_be_dai_prepare(struct snd_soc_pcm_runtime *fe, int stream)
  {
        struct snd_soc_dpcm *dpcm;
@@@ -2071,7 -2030,7 +2074,7 @@@ static int dpcm_fe_dai_prepare(struct s
  
        dev_dbg(fe->dev, "ASoC: prepare FE %s\n", fe->dai_link->name);
  
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_FE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_FE);
  
        /* there is no point preparing this FE if there are no BEs */
        if (list_empty(&fe->dpcm[stream].be_clients)) {
        fe->dpcm[stream].state = SND_SOC_DPCM_STATE_PREPARE;
  
  out:
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
        mutex_unlock(&fe->card->mutex);
  
        return ret;
@@@ -2245,11 -2204,11 +2248,11 @@@ static int dpcm_run_new_update(struct s
  {
        int ret;
  
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_BE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_BE);
        ret = dpcm_run_update_startup(fe, stream);
        if (ret < 0)
                dev_err(fe->dev, "ASoC: failed to startup some BEs\n");
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
  
        return ret;
  }
@@@ -2258,11 -2217,11 +2261,11 @@@ static int dpcm_run_old_update(struct s
  {
        int ret;
  
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_BE;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_BE);
        ret = dpcm_run_update_shutdown(fe, stream);
        if (ret < 0)
                dev_err(fe->dev, "ASoC: failed to shutdown some BEs\n");
 -      fe->dpcm[stream].runtime_update = SND_SOC_DPCM_UPDATE_NO;
 +      dpcm_set_fe_update_state(fe, stream, SND_SOC_DPCM_UPDATE_NO);
  
        return ret;
  }