On 03/19/2014 12:53 PM, Mark Brown wrote:
On Wed, Mar 19, 2014 at 12:47:59PM +0100, Lars-Peter Clausen wrote:
On 03/19/2014 12:10 PM, Mark Brown wrote:
No, we're trying to make soc-io smaller! Besides, if this stuff is getting cleaned up then:
We are trying to remove the ASoC level IO abstraction layer. In my opinion keeping everything related to this in one file, rather than hiding it among 5k lines of other code, makes it clear what is still left to do.
Like I've said before it is not obvious to me that removing the wrappers is a win - yes, things can obtain their regmap but if enough things need to do it then having a convenience wrapper which does the lookup at least doesn't hurt.
It's more about things like snd_soc_update_bits. The wrappers will stay, but can as you said be made static inline functions and moved to the header.