Hi, Attached patch modifies some of the Ne10 includes in celt/arm. The original includes worked, but ended up pulling in extra Ne10 headers that were unnecessary. This allows libopus to be built with just the Ne10 DSP module installed on the user's machine. Please review. Thanks, --Michael -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.xiph.org/pipermail/opus/attachments/20170127/087d9682/attachment.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-Reduce-the-scope-of-Ne10-includes.patch Type: application/octet-stream Size: 2937 bytes Desc: not available URL: <http://lists.xiph.org/pipermail/opus/attachments/20170127/087d9682/attachment.obj>
On Fri, Jan 27, 2017 at 7:41 PM, Michael Bradshaw <mjbshaw at google.com> wrote:> Attached patch modifies some of the Ne10 includes in celt/arm. The > original includes worked, but ended up pulling in extra Ne10 headers that > were unnecessary. This allows libopus to be built with just the Ne10 DSP > module installed on the user's machine. Please review. >Hello again. Just following up to see if there are any additional changes that I should make to this patch. I've tested with both armv7 and arm64 on a machine that only has the Ne10 DSP module available. --Michael -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.xiph.org/pipermail/opus/attachments/20170215/b8b52b92/attachment.html>
Hi Michael, Sorry for accidentally dropping that. Can you send me the latest version of the patch? Thanks, Jean-Marc On 15/02/17 04:30 PM, Michael Bradshaw wrote:> On Fri, Jan 27, 2017 at 7:41 PM, Michael Bradshaw <mjbshaw at google.com > <mailto:mjbshaw at google.com>> wrote: > > Attached patch modifies some of the Ne10 includes in celt/arm. The > original includes worked, but ended up pulling in extra Ne10 headers > that were unnecessary. This allows libopus to be built with just the > Ne10 DSP module installed on the user's machine. Please review. > > > Hello again. Just following up to see if there are any additional > changes that I should make to this patch. I've tested with both armv7 > and arm64 on a machine that only has the Ne10 DSP module available. > > --Michael > > > _______________________________________________ > opus mailing list > opus at xiph.org > http://lists.xiph.org/mailman/listinfo/opus >