[alsa-devel] [PATCH 000/127] ASoC: codec cleanup - codec probe/remove

[Mark Brown]

On Thu, Aug 11, 2016 at 01:03:05AM +0000, Kuninori Morimoto wrote:

> Hmm... If my understand was correct, my concern is this.
> If we merged all (CPU/Codec/Platform/etc...) features into component,
> it will be big size, and almost all feature are not needed for almost all devices.

If there are concrete issues we can look at them but let's do that when
they arise rather than early on.  We do currently have devices that span
all classes (CODECs that take audio streams in through their register
maps, SoCs with built in CODECS...) so there definitely are devices that
need the overlap.

> According to your (and Lars ?) idea, if component includes all features,
> then, current "codec" will be "component", and "platform" will be "component" too (?).
> If so, I think necessary patch in final stage is like this ?

> 	- static struct snd_soc_codec_driver xxxx = {
> 	+ static struct snd_soc_component_driver xxxx = {

That's the goal.

> If so my previous patch is doing pointless effort, and will be trouble ?

I'd more say it's premature at this point, it's something that should be
done for each driver once it is converted to not need anything that's
CODEC specific rather than something that should be done early on.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20160811/cc47d4de/attachment.sig>