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 >
On Wed, Feb 15, 2017 at 1:40 PM, Jean-Marc Valin <jmvalin at jmvalin.ca> wrote:> Hi Michael, > > Sorry for accidentally dropping that. Can you send me the latest version > of the patch?No worries, thanks for taking a look! Attached is the patch rebased from master's HEAD. --Michael -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.xiph.org/pipermail/opus/attachments/20170215/3c32699e/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/20170215/3c32699e/attachment.obj>
Pushed to master. Jean-Marc On 15/02/17 04:44 PM, Michael Bradshaw wrote:> On Wed, Feb 15, 2017 at 1:40 PM, Jean-Marc Valin <jmvalin at jmvalin.ca > <mailto:jmvalin at jmvalin.ca>> wrote: > > Hi Michael, > > Sorry for accidentally dropping that. Can you send me the latest version > of the patch? > > > No worries, thanks for taking a look! Attached is the patch rebased from > master's HEAD. > > --Michael