lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 3 Sep 2020 09:05:32 -0500 From: Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com> To: Vinod Koul <vkoul@...nel.org>, Bard Liao <yung-chuan.liao@...ux.intel.com> Cc: alsa-devel@...a-project.org, tiwai@...e.de, gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org, ranjani.sridharan@...ux.intel.com, hui.wang@...onical.com, broonie@...nel.org, srinivas.kandagatla@...aro.org, jank@...ence.com, mengdong.lin@...el.com, sanyog.r.kale@...el.com, rander.wang@...ux.intel.com, bard.liao@...el.com Subject: Re: [PATCH 0/7] ASoC: soundwire: Move sdw stream operations to 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. >> 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 >
Powered by blists - more mailing lists