From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keyon Jie Subject: Re: [Sound-open-firmware] [v4, 00/14] ASoC: Sound Open Firmware (SOF) core Date: Mon, 25 Feb 2019 11:05:15 +0800 Message-ID: <22872b53-338f-df35-8682-129b2db83b7e@linux.intel.com> References: <20190213220734.10471-1-pierre-louis.bossart@linux.intel.com> <1550520188-9253-1-git-send-email-xiaoxiang@xiaomi.com> <2ad7bf66-a0a9-b97d-e1b1-48e3f9c699c5@linaro.org> <20190221043925.GF2736@vkoul-mobl> <7a67b767-5127-1844-20c5-06ef5946e85d@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" To: xiang xiao Cc: Daniel Baluta , Kumar Gala , andriy.shevchenko@intel.com, Alan Cox , tiwai@suse.de, Arnaud POULIQUEN , linux-remoteproc@vger.kernel.org, alsa-devel@alsa-project.org, Pierre-Louis Bossart , liam.r.girdwood@linux.intel.com, Vinod Koul , broonie@kernel.org, Srinivas Kandagatla , Xiang Xiao , Bjorn Andersson , wendy.liang@xilinx.com, sound-open-firmware@alsa-project.org List-Id: alsa-devel@alsa-project.org CgpPbiAyMDE5LzIvMjMg5LiK5Y2IMjoyMSwgeGlhbmcgeGlhbyB3cm90ZToKPiBPbiBGcmksIEZl YiAyMiwgMjAxOSBhdCA3OjEzIFBNIEtleW9uIEppZSA8eWFuZy5qaWVAbGludXguaW50ZWwuY29t PiB3cm90ZToKPj4gVGhlbiB0aGlzIGxvb2tzIGRvYWJsZSwgdGhhbmtzIGZvciBzaGFyaW5nIFhp YW5nLgo+Pgo+PiBJIGhhdmUgc2V2ZXJhbCBxdWVzdGlvbnM6Cj4+IDEuIElzIHZpcnRpbyBkZXZp Y2UgYW5kIHZyaW5nIGJ1ZmZlciBhY2Nlc3Mgc3VwcG9ydCBpbiByZW1vdGUgcHJvYyhBdWRpbwo+ PiBEU1AgaW4gb3VyIGNhc2UpIHNpZGUgbWFuZGF0b3J5PyBXaGljaCBtZWFucyB3ZSBoYXZlIHRv IHBvcnRpbmcKPj4gbGlicmFyaWVzIGxpa2UgbGliQU1QIGludG8gRlcobm90IHVzZXMgWmVwaHly IG9yIGZyZWVSVE9TIHlldCk/Cj4gCj4gSWYgeW91IGdvIHRocm91Z2ggdmlyaXRpby9ycG1zZyBw YXRoLCB2aXJ0aW8vdnJpbmcgYXJlIHJlcXVpcmVkLAo+IGJ1dCBpZiB5b3UgaW1wbGVtZW50IHRo ZSBuZXcgcnBtc2dfZW5kcG9pbnRfb3BzLCBib3RoIHNob3VsZG4ndCBiZSBuZWVkZWQuCj4gQnV0 IGJvdGggT3BlbkFNUC9SUE1zZy1MaXRlIHN1cHBvcnQgdGhlIGJhcmUgbWV0YWwgZW52aXJvbm1l bnQsCj4gYWN0dWFsbHkgc2luY2UgYm90aCBsaWJyYXJ5IGRlZmluZSB0aGUgY2xlYW4gYW5kIHRo aW4gcG9ydCBsYXllciwgaXQncwo+IHZlcnkgZWFzeSB0byBwb3J0KEkganVzdCBzcGVuZCAyLTMg ZGF5cyBmb3IgTnV0dFggcG9ydCkuCgpUaGFua3MgZm9yIHNoYXJpbmcsIHRoYXQncyBnb29kIGlm IGl0IGlzIHBvc3NpYmxlIHRvIGdvIHdpdGhvdXQgdmlydGlvIApzdXBwb3J0IGF0IHRoZSBmaXJz dCBzdGVwLCB3ZSB3aWxsIGRvIGludmVzdGlnYXRpb24gYmFzZWQgb24gY29kZSB5b3UgCnNoYXJl ZC4KCj4gCj4+IDIuIEFib3V0IHRoZSByZXNvdXJjZSB0YWJsZSBpbiBGVyBiaW5hcnksIGlzIHRo aXMgbWFuZGF0b3J5IG9yIGl0IGNvdWxkCj4+IGJlIGVtcHR5Pwo+IAo+IEFnYWluIHZpcnRpby9y cG1zZyBwYXRoIGlzIHJlcXVpcmVkLCBidXQgY3VzdG9tIGltcGxlbWVudGF0aW9uIG1heW4ndCBu ZWVkZWQuCj4gQWN0dWFsbHksIHJlbW90ZXByb2MgbGF5ZXIgc3VwcG9ydCB0aGUgY3VzdG9tIGZp cm13YXJlIGZvcm1hdCwgc28gU09GCj4gY2FuIG1pZ3JhdGUgdG8gcmVtb3RlcHJvYyB3aXRob3V0 IGNoYW5naW5nIGZpcm13YXJlIGZvcm1hdC4KCkdvb2QgdG8ga25vdyB0aGlzLCB0aGFua3MuCgpU aGFua3MsCn5LZXlvbgoKPiAKPj4KPj4gQXMgeW91IG1heSBrbm93LCB3ZSB1c2VzIFNPRi1zaW1p bGFyIElQQyBmcmFtZXdvcmsod2hpY2ggaXMgcXVpdGUKPj4gZGlmZmVyZW50IHdpdGggcnBtc2cv cmVtb3RlcHJvYykgb24gSW50ZWwgU29DIHBsYXRmb3JtcyBmb3IgbG9uZyBvbiB0ZW5zCj4+IG9m IHBsYXRmb3JtcyBvZiBzZXZlcmFsIGdlbmVyYXRpb25zKHdlIGhhdmUgZW5hYmxlZCBhbmQgdmVy aWZpZWQgU09GIG9uCj4+IG1vc3Qgb2YgdGhlbSksIHNvIHRoZSB0cmFuc2l0aW9uIG1heSB0YWtl IHRpbWUuCj4+Cj4gCj4gWWVzLCBJIHVuZGVyc3RhbmQuIEJ1dCBJbnRlbCBzdGlsbCBjb3VsZCBw cm92aWRlIHRoZSBjdXJyZW50IHZlcnNpb24KPiB0byB0aGUgY3VzdG9tZXIgYmVmb3JlIHRoZSBu ZXcgSVBDIHZlcnNpb24gaXMgc3RhYmxlIGlmIG1hbnkgcGVvcGxlCj4gdGhpbmsgdGhlIGdlbmVy aWMgSVBDIGlzIHRoZSBnb29kIGRpcmVjdGlvbiB0byBnby4KPiAKPj4gVGhhbmtzLAo+PiB+S2V5 b24KPj4KPj4+Cj4+Pj4gVGhlIHJlc291cmNlcyBlbWJlZGRlZCBpbiBhIGZpcm13YXJlIGZpbGUg aXMgYW5vdGhlciBjYXBhYmlsaXR5IHRoYXQgZG9lc24ndCBhbGlnbiB3aXRoIHRoZSB3YXkgdGhl IFNPRiBmaXJtd2FyZSBpcyBnZW5lcmF0ZWQuIEkgYWxzbyBkb24ndCBrbm93IHdoZXJlIHRoZSB0 b3BvbG9neSBmaWxlIHdvdWxkIGJlIGhhbmRsZWQsIG5vciBob3cgdG8gZGVhbCB3aXRoIHN1c3Bl bmQtcmVzdW1lIHdoZXJlIHRoZSBEU1AgbmVlZHMgdG8gYmUgcmVzdGFydGVkLiBGb3IgZm9sa3Mg d2hvIG5lZWQgYW4gaW50cm9kdWN0aW9uIHRvIFJQTXNnLCB0aGUgbGluayBbMV0gaXMgdGhlIGJl c3QgSSBmb3VuZCB0byBzY29wZSBvdXQgdGhlIHdvcmsgcmVxdWlyZWQuCj4+Pj4KCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkFsc2EtZGV2ZWwgbWFpbGlu ZyBsaXN0CkFsc2EtZGV2ZWxAYWxzYS1wcm9qZWN0Lm9yZwpodHRwczovL21haWxtYW4uYWxzYS1w cm9qZWN0Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Fsc2EtZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com ([192.55.52.93]:21614 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726539AbfBYDDT (ORCPT ); Sun, 24 Feb 2019 22:03:19 -0500 Subject: Re: [Sound-open-firmware] [alsa-devel] [v4, 00/14] ASoC: Sound Open Firmware (SOF) core References: <20190213220734.10471-1-pierre-louis.bossart@linux.intel.com> <1550520188-9253-1-git-send-email-xiaoxiang@xiaomi.com> <2ad7bf66-a0a9-b97d-e1b1-48e3f9c699c5@linaro.org> <20190221043925.GF2736@vkoul-mobl> <7a67b767-5127-1844-20c5-06ef5946e85d@linux.intel.com> From: Keyon Jie Message-ID: <22872b53-338f-df35-8682-129b2db83b7e@linux.intel.com> Date: Mon, 25 Feb 2019 11:05:15 +0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-remoteproc-owner@vger.kernel.org To: xiang xiao Cc: Daniel Baluta , Kumar Gala , andriy.shevchenko@intel.com, alsa-devel@alsa-project.org, tiwai@suse.de, linux-remoteproc@vger.kernel.org, Arnaud POULIQUEN , Pierre-Louis Bossart , Bjorn Andersson , liam.r.girdwood@linux.intel.com, Vinod Koul , broonie@kernel.org, Srinivas Kandagatla , Xiang Xiao , sound-open-firmware@alsa-project.org, wendy.liang@xilinx.com, Alan Cox List-ID: On 2019/2/23 上午2:21, xiang xiao wrote: > On Fri, Feb 22, 2019 at 7:13 PM Keyon Jie wrote: >> Then this looks doable, thanks for sharing Xiang. >> >> I have several questions: >> 1. Is virtio device and vring buffer access support in remote proc(Audio >> DSP in our case) side mandatory? Which means we have to porting >> libraries like libAMP into FW(not uses Zephyr or freeRTOS yet)? > > If you go through viritio/rpmsg path, virtio/vring are required, > but if you implement the new rpmsg_endpoint_ops, both shouldn't be needed. > But both OpenAMP/RPMsg-Lite support the bare metal environment, > actually since both library define the clean and thin port layer, it's > very easy to port(I just spend 2-3 days for NuttX port). Thanks for sharing, that's good if it is possible to go without virtio support at the first step, we will do investigation based on code you shared. > >> 2. About the resource table in FW binary, is this mandatory or it could >> be empty? > > Again virtio/rpmsg path is required, but custom implementation mayn't needed. > Actually, remoteproc layer support the custom firmware format, so SOF > can migrate to remoteproc without changing firmware format. Good to know this, thanks. Thanks, ~Keyon > >> >> As you may know, we uses SOF-similar IPC framework(which is quite >> different with rpmsg/remoteproc) on Intel SoC platforms for long on tens >> of platforms of several generations(we have enabled and verified SOF on >> most of them), so the transition may take time. >> > > Yes, I understand. But Intel still could provide the current version > to the customer before the new IPC version is stable if many people > think the generic IPC is the good direction to go. > >> Thanks, >> ~Keyon >> >>> >>>> The resources embedded in a firmware file is another capability that doesn't align with the way the SOF firmware is generated. I also don't know where the topology file would be handled, nor how to deal with suspend-resume where the DSP needs to be restarted. For folks who need an introduction to RPMsg, the link [1] is the best I found to scope out the work required. >>>>