ASoC: Remove IO register modifier callbacks
authorLars-Peter Clausen <lars@metafoo.de>
Tue, 18 Mar 2014 08:02:09 +0000 (09:02 +0100)
committerMark Brown <broonie@linaro.org>
Mon, 14 Apr 2014 16:22:43 +0000 (17:22 +0100)
There are no ASoC drivers left that use them and new drivers are supposed to use
regmap for this.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@linaro.org>
include/sound/soc.h
sound/soc/soc-cache.c
sound/soc/soc-core.c

index 14e7457e234717fa20a1d85d06ccc92f4aa3c9ca..a355d0f9a6f972e9ce455183fc97c9a0b8888e42 100644 (file)
@@ -393,12 +393,6 @@ int devm_snd_soc_register_component(struct device *dev,
                         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_codec_volatile_register(struct snd_soc_codec *codec,
-                                   unsigned int reg);
-int snd_soc_codec_readable_register(struct snd_soc_codec *codec,
-                                   unsigned int reg);
-int snd_soc_codec_writable_register(struct snd_soc_codec *codec,
-                                   unsigned int reg);
 int snd_soc_codec_set_cache_io(struct snd_soc_codec *codec,
                               struct regmap *regmap);
 int snd_soc_cache_sync(struct snd_soc_codec *codec);
@@ -692,9 +686,6 @@ struct snd_soc_codec {
        struct list_head list;
        struct list_head card_list;
        int num_dai;
-       int (*volatile_register)(struct snd_soc_codec *, unsigned int);
-       int (*readable_register)(struct snd_soc_codec *, unsigned int);
-       int (*writable_register)(struct snd_soc_codec *, unsigned int);
 
        /* runtime */
        struct snd_ac97 *ac97;  /* for ad-hoc ac97 devices */
@@ -756,11 +747,6 @@ struct snd_soc_codec_driver {
        /* codec IO */
        unsigned int (*read)(struct snd_soc_codec *, unsigned int);
        int (*write)(struct snd_soc_codec *, unsigned int, unsigned int);
-       int (*display_register)(struct snd_soc_codec *, char *,
-                               size_t, unsigned int);
-       int (*volatile_register)(struct snd_soc_codec *, unsigned int);
-       int (*readable_register)(struct snd_soc_codec *, unsigned int);
-       int (*writable_register)(struct snd_soc_codec *, unsigned int);
        unsigned int reg_cache_size;
        short reg_cache_step;
        short reg_word_size;
index bfed3e4c45ffa708910e580bf02359a7b6b3c509..3fa77d5f9b75c0b7f442669301f4bf667eab8f7c 100644 (file)
@@ -162,8 +162,6 @@ static int snd_soc_flat_cache_sync(struct snd_soc_codec *codec)
                                                  i, codec_drv->reg_word_size) == val)
                                continue;
 
-               WARN_ON(!snd_soc_codec_writable_register(codec, i));
-
                ret = snd_soc_write(codec, i, val);
                if (ret)
                        return ret;
index 5c0ed393128505eb17a3a506ec9655926497b3ad..41bd243485205cff0042a4026b73208fe51b154f 100644 (file)
@@ -154,22 +154,15 @@ static ssize_t soc_codec_reg_show(struct snd_soc_codec *codec, char *buf,
                step = codec->driver->reg_cache_step;
 
        for (i = 0; i < codec->driver->reg_cache_size; i += step) {
-               if (!snd_soc_codec_readable_register(codec, i))
-                       continue;
-               if (codec->driver->display_register) {
-                       count += codec->driver->display_register(codec, buf + count,
-                                                        PAGE_SIZE - count, i);
-               } else {
-                       /* only support larger than PAGE_SIZE bytes debugfs
-                        * entries for the default case */
-                       if (p >= pos) {
-                               if (total + len >= count - 1)
-                                       break;
-                               format_register_str(codec, i, buf + total, len);
-                               total += len;
-                       }
-                       p += len;
+               /* only support larger than PAGE_SIZE bytes debugfs
+                * entries for the default case */
+               if (p >= pos) {
+                       if (total + len >= count - 1)
+                               break;
+                       format_register_str(codec, i, buf + total, len);
+                       total += len;
                }
+               p += len;
        }
 
        total = min(total, count - 1);
@@ -1979,60 +1972,6 @@ static struct platform_driver soc_driver = {
        .remove         = soc_remove,
 };
 
-/**
- * snd_soc_codec_volatile_register: Report if a register is volatile.
- *
- * @codec: CODEC to query.
- * @reg: Register to query.
- *
- * Boolean function indiciating if a CODEC register is volatile.
- */
-int snd_soc_codec_volatile_register(struct snd_soc_codec *codec,
-                                   unsigned int reg)
-{
-       if (codec->volatile_register)
-               return codec->volatile_register(codec, reg);
-       else
-               return 0;
-}
-EXPORT_SYMBOL_GPL(snd_soc_codec_volatile_register);
-
-/**
- * snd_soc_codec_readable_register: Report if a register is readable.
- *
- * @codec: CODEC to query.
- * @reg: Register to query.
- *
- * Boolean function indicating if a CODEC register is readable.
- */
-int snd_soc_codec_readable_register(struct snd_soc_codec *codec,
-                                   unsigned int reg)
-{
-       if (codec->readable_register)
-               return codec->readable_register(codec, reg);
-       else
-               return 1;
-}
-EXPORT_SYMBOL_GPL(snd_soc_codec_readable_register);
-
-/**
- * snd_soc_codec_writable_register: Report if a register is writable.
- *
- * @codec: CODEC to query.
- * @reg: Register to query.
- *
- * Boolean function indicating if a CODEC register is writable.
- */
-int snd_soc_codec_writable_register(struct snd_soc_codec *codec,
-                                   unsigned int reg)
-{
-       if (codec->writable_register)
-               return codec->writable_register(codec, reg);
-       else
-               return 1;
-}
-EXPORT_SYMBOL_GPL(snd_soc_codec_writable_register);
-
 /**
  * snd_soc_new_ac97_codec - initailise AC97 device
  * @codec: audio codec
@@ -4136,9 +4075,6 @@ int snd_soc_register_codec(struct device *dev,
 
        codec->write = codec_drv->write;
        codec->read = codec_drv->read;
-       codec->volatile_register = codec_drv->volatile_register;
-       codec->readable_register = codec_drv->readable_register;
-       codec->writable_register = codec_drv->writable_register;
        codec->component.ignore_pmdown_time = codec_drv->ignore_pmdown_time;
        codec->dapm.bias_level = SND_SOC_BIAS_OFF;
        codec->dapm.dev = dev;