Re: Failed to set cache compression type

[Lars-Peter Clausen <lars@metafoo.de>]

On 06/02/2014 04:02 PM, Jarkko Nikula wrote:
> On 06/02/2014 04:43 PM, Fabio Estevam wrote:
>> Hi,
>>
>> I am booting linux-next 20140602 on a mx6qsabresd and I get the following
>> probe
>> error:
>>
>> wm8962 0-001a: customer id 0 revision D
>> wm8962 0-001a: ASoC: Failed to set cache compression type: -22
>> imx-wm8962 sound: snd_soc_register_card failed (-22)
>> imx-wm8962: probe of sound failed with error -22
>>
>> What is the proper way to fix it?
>>
> I was looking at this too and it was introduced by the commit b59dce53ef61
> ("ASoC: cache: Fix possible ZERO_SIZE_PTR pointer dereferencing error.").

I think snd_soc_cache_init() should return 0 rather than -EINVAL if reg_size 
is 0.

>
> If I understood commit 657254714ad2 ("ASoC: io: Remove support for ASoC
> cache in conjunction with regmap") right should we remove remaining
> soc-cache use? All codec drivers seems to include regmap.h although some of
> them are still calling soc_cache_sync().

Yes, ASoC level support for caching is going to be removed, hopefully sooner 
than later. If a driver still uses soc_cache_sync() even though it has 
switched to regmap for IO that's a bug.

- Lars