ASoC: soc-cache: Ensure flat compression uses a copy of the defaults cache
authorDimitris Papastamos <dp@opensource.wolfsonmicro.com>
Wed, 12 Jan 2011 10:22:28 +0000 (10:22 +0000)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Wed, 12 Jan 2011 14:55:12 +0000 (14:55 +0000)
With the addition of the cache fallback functionality, it is necessary
to ensure that if the register defaults cache was marked as __devinitconst
and the LZO compression is not compiled in the kernel, the fallback to
flat compression will still use a copy of the defaults 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>
sound/soc/soc-cache.c

index 9e534429ea6326938a63d09beb9e2a65b2a7aab8..19b29fb3ca4bcfc0ddc2beb92b53e48b87f20612 100644 (file)
@@ -1400,8 +1400,8 @@ static int snd_soc_flat_cache_sync(struct snd_soc_codec *codec)
                ret = snd_soc_cache_read(codec, i, &val);
                if (ret)
                        return ret;
-               if (codec_drv->reg_cache_default)
-                       if (snd_soc_get_cache_val(codec_drv->reg_cache_default,
+               if (codec->reg_def_copy)
+                       if (snd_soc_get_cache_val(codec->reg_def_copy,
                                                  i, codec_drv->reg_word_size) == val)
                                continue;
                ret = snd_soc_write(codec, i, val);
@@ -1446,16 +1446,8 @@ static int snd_soc_flat_cache_init(struct snd_soc_codec *codec)
        codec_drv = codec->driver;
        reg_size = codec_drv->reg_cache_size * codec_drv->reg_word_size;
 
-       /*
-        * for flat compression, we don't need to keep a copy of the
-        * original defaults register cache as it will definitely not
-        * be marked as __devinitconst
-        */
-       kfree(codec->reg_def_copy);
-       codec->reg_def_copy = NULL;
-
-       if (codec_drv->reg_cache_default)
-               codec->reg_cache = kmemdup(codec_drv->reg_cache_default,
+       if (codec->reg_def_copy)
+               codec->reg_cache = kmemdup(codec->reg_def_copy,
                                           reg_size, GFP_KERNEL);
        else
                codec->reg_cache = kzalloc(reg_size, GFP_KERNEL);