Re: [PATCH 3/3] ASoC: fsl_ssi: remove register defaults

From: Timur Tabi
Date: Mon Jan 11 2016 - 10:45:42 EST


Mark Brown wrote:
That's possibly problematic because the flat cache will of necessity end
up with defaults (of 0 from the kzalloc()) for all the registers.
You'll still have default values in the cache, though some of the
behaviour around optimising syncs does change without them explicitly
given. It does deal with the allocation issue but given that the issue
was incorrect defaults I'd be a bit concerned.

Ok, I'm confused. Granted, all of this regcache stuff was added after I stopped working on this driver, so I'm out of the loop. But it appears that the regcache cannot properly handle an uninitialized cache. I would expect it to know to perform hard reads of any registers that are uninitialized.

If the regcache wants to have an initialized cache, then it should automatically perform reads an all non-volatile, non-precious registers at initialization.