[alsa-devel] [PATCH 04/13] ASoC: Move IO functions to soc-io.c
lars at metafoo.de
Wed Mar 19 12:57:57 CET 2014
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.
More information about the Alsa-devel