On Fri, Jan 26, 2018 at 12:33:53AM +0000, Kuninori Morimoto wrote:
In existing drivers, the component which have compr_ops *was* originally "platform" component only. The main purpose why we want to replace platform (and codec) to component is that CPU/Codec/Platform categorize is no longer best match for modern device.
Thus, if we can replace all CPU/Codec/Platform into component, every device can have every feature, with no categorize ! In other words, we don't know how many device have it.
Yeah I think there is quite a lot more work to get to that point though, we need to understand how we will present this to user-space.
But anyway, we don't have such situation, and we don't know how to adjust to it at this point. Thus Yes !! I can 100% agree your opinion. Thank you for your help
One note is that "rtd->platform" itself will be removed if all platforms were replaced. Then, soc_compr_get_component() / soc_compr_get_ops() might be merged or adjusted.
Yes that was my thinking as well, once you remove that you should only need some minor refactoring to make the updates.
Thanks, Charles