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.