ASoC: Automatically assign the default readable()/volatile() functions

Ensure that all calls to readable_register()/volatile_register() go via
the snd_soc_codec function pointers.

If the default register access table has been given but no functions
for handling readable()/volatile() registers, use the default ones provided
by soc-cache.

Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

2 files changed, 14 insertions, 3 deletions

diff --git a/include/sound/soc.h b/include/sound/soc.h
index 97d1832bb9df..accb8a16c165 100644
--- a/include/sound/soc.h
+++ b/include/sound/soc.h
@@ -480,6 +480,8 @@ struct snd_soc_codec {
         int num_dai;
         enum snd_soc_compress_type compress_type;
         size_t reg_size;        /* reg_cache_size * reg_word_size */
+        int (*volatile_register)(struct snd_soc_codec *, unsigned int);
+        int (*readable_register)(struct snd_soc_codec *, unsigned int);
 
         /* runtime */
         struct snd_ac97 *ac97;  /* for ad-hoc ac97 devices */
diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
index b5e5758456bd..30d76e8bc9df 100644
--- a/sound/soc/soc-core.c
+++ b/sound/soc/soc-core.c
@@ -84,7 +84,7 @@ static ssize_t soc_codec_reg_show(struct snd_soc_codec *codec, char *buf)
 
         count += sprintf(buf, "%s registers\n", codec->name);
         for (i = 0; i < codec->driver->reg_cache_size; i += step) {
-                if (codec->driver->readable_register && !codec->driver->readable_register(codec, i))
+                if (codec->readable_register && !codec->readable_register(codec, i))
                         continue;
 
                 count += sprintf(buf + count, "%2x: ", i);
@@ -2029,8 +2029,8 @@ static int soc_new_pcm(struct snd_soc_pcm_runtime *rtd, int num)
  */
 int snd_soc_codec_volatile_register(struct snd_soc_codec *codec, int reg)
 {
-        if (codec->driver->volatile_register)
-                return codec->driver->volatile_register(codec, reg);
+        if (codec->volatile_register)
+                return codec->volatile_register(codec, reg);
         else
                 return 0;
 }
@@ -3489,6 +3489,8 @@ 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->dapm.bias_level = SND_SOC_BIAS_OFF;
         codec->dapm.dev = dev;
         codec->dapm.codec = codec;
@@ -3517,6 +3519,13 @@ int snd_soc_register_codec(struct device *dev,
                 }
         }
 
+        if (codec_drv->reg_access_size && codec_drv->reg_access_default) {
+                if (!codec->volatile_register)
+                        codec->volatile_register = snd_soc_default_volatile_register;
+                if (!codec->readable_register)
+                        codec->readable_register = snd_soc_default_readable_register;
+        }
+
         for (i = 0; i < num_dai; i++) {
                 fixup_codec_formats(&dai_drv[i].playback);
                 fixup_codec_formats(&dai_drv[i].capture);