On 03-09-20, 09:05, Pierre-Louis Bossart wrote:
On 9/3/20 5:42 AM, Vinod Koul wrote:
On 01-09-20, 23:02, Bard Liao wrote:
sdw stream operation APIs can be called once per stream. dailink callbacks are good places to call these APIs.
Again, please mention here if this is to be merged thru sdw tree or ASoC tree
Good point, I thought it wouldn't matter but it does. I just gave it a try and there seems to be a conflict on Mark's tree w/ drivers/soundwire/intel.c (likely due to missing patches already added to Vinod's tree).
So this should go to Vinod's tree with Mark's Acked-by tag on the ASoC changes.
Alternatively we can also split this in two, with ASoC-only and SoundWire-only patches in separate series if it's easier for maintainers. We would lose the rationale for the changes but that's not essential.
If there are no dependencies on each other, that is best preferred option. One should mention in cover-letter about the linked series though.
Pierre-Louis Bossart (7): ASoC: soc-dai: clarify return value for get_sdw_stream() soundwire: stream: fix NULL/IS_ERR confusion soundwire: intel: fix NULL/ERR_PTR confusion ASOC: Intel: sof_sdw: add dailink .trigger callback ASOC: Intel: sof_sdw: add dailink .prepare and .hw_free callback
These should be ASoC
Right. if you are fine with the content and this goes in your tree, can this be modified while applying? Or do want a v2?
soundwire: intel: remove .trigger operation soundwire: intel: remove stream handling from .prepare and .hw_free
drivers/soundwire/intel.c | 60 ++++------------------- drivers/soundwire/stream.c | 2 +- include/sound/soc-dai.h | 3 +- sound/soc/intel/boards/sof_sdw.c | 81 ++++++++++++++++++++++++++++++++ 4 files changed, 92 insertions(+), 54 deletions(-)
-- 2.17.1