search for: ne10

Using GCC 4.9.2, decoding the opus test vector set 10 times. All tests pass. I will do longer tests later with a larger test set, but it looks good so far. I am having a lot of trouble with the Ne10 detection. Using the precompiled Ne10 binaries at http://people.linaro.org/~viswanath.puttagunta/opus/NE10_root/ Both NE10 and Ne10 capitalizations seem to be in use. Also, where should the NE10 headers be installed? In $PREFIX/include or $PREFIX/include/Ne10/? The configure.ac tests seems to assu...

Hello Thomas, I use the following configure command to link against Ne10 Eg: configure --host=arm-linux-gnueabihf --enable-intrinsics --with-NE10-libraries=${BUILD_NE10_LIB} --with-NE10-includes=${BUILD_NE10_INC}" So, in my normal testing, I explicitly specify where the NE10 header files are installed and where the NE10 libraries are installed. Looking back at c...

..., Timothy B. Terriberry <tterribe at xiph.org> wrote: > > Viswanath Puttagunta wrote: >> >> This patch series is follow up on work I posted on [1]. >> In addition to what was posted on [1], this patch series mainly >> integrates Fixed point FFT implementations in NE10 library into opus. >> You can view my opus wip code at [2]. > > > Thanks, and I apologize for being slow here. > >> - This was surprising to me because test_unit_dft passes for all >> nfft including 60, 120, 240, 480. May be there are some data >>...

Hi everyone, I am working on Ne10 project. Ne10 provides NEON optimized FFT routines that are much faster (compared to those without NEON), on most ARMv7-A and all ARMv8-A devices. How about integrate it into Opus? I am not familiar with configure script, but I find "Optinal Packages" in it. If we provides --with-ne10-f...

...ifft/mdct_backwards, fixed point fft/ifft mdct_forward/backward > etc) that are in my pipeline that depend on this patch series being > accepted. So, trying to make progress on this... On an armv7l board running Ubuntu, you've broken the build with just --enable-intrinsics (without the NE10 options): CC celt/tests/test_unit_dft.o In file included from ../celt/tests/test_unit_dft.c:47:0: ../celt/kiss_fft.c: In function 'opus_fft_free': ../celt/kiss_fft.c:507:51: warning: unused parameter 'arch' [-Wunused-parameter] void opus_fft_free(const kiss_fft_state *c...

I'm trying to get these cleaned up and landed, but I'm running into some trouble with this patch. Using commit a08b29d88e3c (July 21) of Ne10, I'm seeing test failures for 60-point FFTs: nfft=60 inverse=0,snr = -3.312408 ** poor snr: -3.312408 ** nfft=60 inverse=1,snr = -16.079597 ** poor snr: -16.079597 ** All other sizes tested appear to work fine (84 to 140 dB of SNR). This doesn't match the results you reported on this li...

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...

Hi Ralph, I have pushed patches to enable radix 3 and radix 5. Github: https://github.com/projectNe10/Ne10/releases/tag/v1.2.0 Best Regards, Phil Wang > Date: Thu, 11 Dec 2014 10:46:50 -0800 > From: Ralph Giles <giles at thaumas.net> > Subject: Re: [opus] [ARM][FFT][NEON] Integrate Ne10 into Opus? > To: opus at xiph.org > Message-ID: <5489E69A.5000305 at thaumas.net> &...

Hello Timothy, Just FYI, Phil at ARM is still looking into why mdct is failing.. will keep you posted. In the mean time, do you want me to disable NE10 for mdct_forward and re-submit the patchset so we may make progress? Regards, Vish On 30 April 2015 at 09:33, Viswanath Puttagunta <viswanath.puttagunta at linaro.org> wrote: > > On 29 April 2015 at 17:22, Timothy B. Terriberry <tterribe at xiph.org> wrote: > > > > V...

Hi all, I have already added support for scaled forward non-power-of-2 floating-point FFT: https://github.com/projectNe10/Ne10/commit/79c3d787302f8d74b9bcfe6545d487cdf1b101d9 Two flags are added to cfg structure: is_forward_scaled and is_backward_scaled. By setting is_forward_scaled to anything but zero, ne10_fft_c2c_1d_float32_neon will scale the output. So we can remove need for one buffer on stack mentioned below....

Hi Timothy, Sorry for late reply. I have upstreamed the patch to fix the regression here: https://github.com/projectNe10/Ne10/commit/ee5d856cd9cb8c4a15ace567df4239f4e788d043 I have tested it with Vish's branch: http://git.linaro.org/people/viswanath.puttagunta/opus.git/shortlog/refs/heads/rfcv3_fft_fixed) Both unit test dft and unit test mdct passed on ARM v7/v8, floating point/fixed point, with or without Ne...

...erally triggered with audio. > > Jean-Marc > > On 08/05/15 11:08 AM, Viswanath Puttagunta wrote: >> Hello Timothy, >> >> Just FYI, Phil at ARM is still looking into why mdct is failing.. will >> keep you posted. In the mean time, do you want me to disable NE10 for >> mdct_forward and re-submit the patchset so we may make progress? >> >> Regards, >> Vish >> >> On 30 April 2015 at 09:33, Viswanath Puttagunta >> <viswanath.puttagunta at linaro.org> wrote: >>> >>> On 29 April 2015 at 17:22, Ti...

...erally triggered with audio. > > Jean-Marc > > On 08/05/15 11:08 AM, Viswanath Puttagunta wrote: >> Hello Timothy, >> >> Just FYI, Phil at ARM is still looking into why mdct is failing.. will >> keep you posted. In the mean time, do you want me to disable NE10 for >> mdct_forward and re-submit the patchset so we may make progress? >> >> Regards, >> Vish >> >> On 30 April 2015 at 09:33, Viswanath Puttagunta >> <viswanath.puttagunta at linaro.org> wrote: >>> >>> On 29 April 2015 at 17:22, Ti...

Hello Timothy, Great to hear from you! Fired up my hardware today and this issue looks like a regression in Ne10 library. The commit in Ne10 [1] that I tested to be working successfully back in May 5b63074db45000f9688460990ee3f5e147d93782 which is the Patch Phil at ARM added to fix the overflow issue in nfft=60 case. After git-bisect, looks like the culprit patch in Ne10 [1] is cf33c0d51a445bbe6ad7e21c2af87...

...t/ifft mdct_forward/backward >> etc) that are in my pipeline that depend on this patch series being >> accepted. > > > So, trying to make progress on this... > > On an armv7l board running Ubuntu, you've broken the build with just > --enable-intrinsics (without the NE10 options): Yep, I should have checked with just this option to make sure I'm not causing regression. I will add to my test plan going forward. > > CC celt/tests/test_unit_dft.o > In file included from ../celt/tests/test_unit_dft.c:47:0: > ../celt/kiss_fft.c: In function '...

...wanath Puttagunta wrote: > if OPUS_ARM_NEON_INTR > CELT_ARM_NEON_INTR_OBJ = $(CELT_SOURCES_ARM_NEON_INTR:.c=.lo) \ > - %test_unit_rotation.o %test_unit_mathops.o > -$(CELT_ARM_NEON_INTR_OBJ): CFLAGS += $(OPUS_ARM_NEON_INTR_CPPFLAGS) > + $(CELT_SOURCES_ARM_NE10:.c=.lo) \ > + %test_unit_rotation.o %test_unit_mathops.o \ > + %test_unit_mdct.o %test_unit_dft.o Crazy indentation. > +$(CELT_ARM_NEON_INTR_OBJ): CFLAGS += $(OPUS_ARM_NEON_INTR_CPPFLAGS) $(NE10_CFLAGS) > endif > diff --git a/celt/arm/arm_c...

On 3 February 2015 at 01:31, Phil Wang <Phil.Wang at arm.com> wrote: > Hi all, > > I have already added support for scaled forward non-power-of-2 floating-point FFT: > https://github.com/projectNe10/Ne10/commit/79c3d787302f8d74b9bcfe6545d487cdf1b101d9 > > Two flags are added to cfg structure: is_forward_scaled and is_backward_scaled. > By setting is_forward_scaled to anything but zero, ne10_fft_c2c_1d_float32_neon will > scale the output. So we can remove need for one buffer on sta...

...n 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 follo...

...is the peak stack usage.... rest looks like fairly straight forward cleanup. Is the peak stack usage a complete blocker in current form? If it is indeed a blocker, would it be acceptable if we can reduce additional buffer requirement from 2 buffers (current) to 1, possibly by moving scaling inside ne10_fft_c2c_1d_float32_neon? (I will let Phil comment on if there is any reason not to do this.. and suggest any alternatives if necessary) I will work on fixing up code from opus side. Phil is one of developers/maintainers of opus. Please correct me if any of information below is inaccurate from a l...

...ean-Marc >>> >>> On 08/05/15 11:08 AM, Viswanath Puttagunta wrote: >>>> Hello Timothy, >>>> >>>> Just FYI, Phil at ARM is still looking into why mdct is failing.. will >>>> keep you posted. In the mean time, do you want me to disable NE10 for >>>> mdct_forward and re-submit the patchset so we may make progress? >>>> >>>> Regards, >>>> Vish >>>> >>>> On 30 April 2015 at 09:33, Viswanath Puttagunta >>>> <viswanath.puttagunta at linaro.org> wrote:...