search for: crichton

Hi Jim, Thanks a lot for investigating. It definitely makes sense now. I'll fix the problem now. Is there any other place where you see that same (or similar) problem? Jean-Marc Jim Crichton a ?crit : > Jean-Marc, > > I dug into this further, and found that the warning occurred when PSHR32 > had a shift greater than 15. > > in fixed_generic.h, PSHR32 is defined as: > > #define PSHR32(a,shift) (SHR32((a)+((1<<((shift))>>1)),shift)) > > For 1...

Thanks Jim for looking into that, I was really starting to wonder what was going on. Let me know if you find a way to tell the compiler to stop complaining. Jean-Marc Jim Crichton a ?crit : > I looked back at my old C55 EC build, and I had the same warning in > mdf.c which Mike found. The assembly code did have a valid shift, and > this build did cancel echo. > > When I built with the SVN head, I saw the errors in kiss_fft.c also. > The assembly there al...

...e, and I have not had a chance to locate it yet). This was overrunning my heap, and enlarging the heap solved the problem. I suspect that Itay is having a similar problem. - Jim ----- Original Message ----- From: "Jean-Marc Valin" <jean-marc.valin@usherbrooke.ca> To: "Jim Crichton" <jim.crichton@comcast.net> Cc: "Itay Chamiel" <ItayC@mangodsp.com>; <speex-dev@xiph.org> Sent: Tuesday, August 15, 2006 9:26 AM Subject: Re: [Speex-dev] AEC on a TI C6x - has no effect > Jim Crichton a ?crit : >> Itay, >> >> I tested on C6x i...

Itay, I tested on C6x in May with build 11343 and 11398, and it worked fine. I just ran the same test with build 11768, and there was no cancellation, just as you saw. I will try to track this down in the next day or two. In the meantime, I suggest that you try testing with 11398. - Jim ----- Original Message ----- From: Itay Chamiel To: speex-dev@xiph.org Sent: Monday, August 14,

...nything wrong with the internal structure having a different type than the interface, as long both types are big enough to hold the possible values (in this case 0 and 1). Though, as you pointed out, testenc needs to be fixed to use spx_int32_t instead of int. I'll change that. Jean-Marc Jim Crichton a ?crit : > st->highpass_enabled is typed inconsistently. It is "int" in the > structure, but "spx_int32_t" in the SET, GET cases in > nb(sb)_encoder(decoder)_ctl. The parameter is "int" in testenc.c, and > in the TI version ti\testenc-TI-C5x.c. This b...

...ti/os_support_custom.h file, which is also attached. I remember your having some trouble reading my patches (from Tortoise SVN) before, so let me know if I need to format this differently for you to use it. Sorry if this was holding you up. - Jim ----- Original Message ----- From: "Jim Crichton" <jim.crichton@comcast.net> To: "Jean-Marc Valin" <jean-marc.valin@usherbrooke.ca> Cc: <speex-dev@xiph.org> Sent: Tuesday, November 06, 2007 1:35 PM Subject: os_support.h, libc overrides > Jean-Marc, > >> That reminds me that I've just moved lots o...

...warning, and it did change the assembled code. With the original file, the ((1<<((shift))>>1)) was optimized away. Note that fixed_debug.h has the same problem. - Jim ----- Original Message ----- From: "Jean-Marc Valin" <jean-marc.valin@usherbrooke.ca> To: "Jim Crichton" <jim.crichton@comcast.net> Cc: <speex-dev@xiph.org>; "Michael Jacobson" <Michael.Jacobson@ultratec.com> Sent: Wednesday, January 23, 2008 12:16 PM Subject: Re: [Speex-dev] Shift count warning messages > Thanks Jim for looking into that, I was really starting t...

...he C5509A, it would be greatly appreciated. Furthermore, I am looking for some technical documentation on the CELP algorithms. I would like to better understand the math being implemented. Do you know where such documentation can be found? Sincerely, Jean Quirion _____ From: Jim Crichton [mailto:jim.crichton@comcast.net] Sent: Thursday, July 19, 2007 10:54 AM To: Quirion, Jean; speex-dev@xiph.org Subject: Re: [Speex-dev] How Can I Get involved in Speex Fixed-PointDevelopment? There is a port for the TI C55X, in the TI directory of the source tree. This builds and runs directly...

...de by r, and there is the hang. It seems that the 0 in Rf[0] is the problem, but I am not sure what else to look at there. The first few 32-bit values in Rf are: 0 0 1 1 0 2 D D 5 0 8 11 19 3D 51 74 3A 8 11 8 4 Do you have any suggestions? - Jim ----- Original Message ----- From: "Jim Crichton" <jim.crichton@comcast.net> To: <speex-dev@xiph.org> Sent: Monday, May 08, 2006 12:11 PM Subject: [Speex-dev] Speex echo canceller on TI C55 DSP > Jean-Marc, > > I recently started looking at running the echo canceller on a TI C55 DSP > along with the 8kbps narrowban...

Jim Crichton a ?crit : > I played briefly with the echo canceller on the C5509A back in May > 2006. I got the same compiler warnings, and sent a message to the > list which included this: > > "I got several compiler warnings for "shift out of range" in mdf.c, > which I fixed b...

...49) 923 0074 Publisher and Developer Services, http://www.poweredbygamespy.com NEW: Online Matters - PubServices blog: http://www.poweredbygamespy.com/blog -----Original Message----- From: Jean-Marc Valin [mailto:jean-marc.valin@usherbrooke.ca] Sent: Wednesday, October 24, 2007 7:12 AM To: Jim Crichton Cc: Saad Nader; speex-dev@xiph.org Subject: Re: [Speex-dev] Speex with PS3 SPE support Hi Jim, Jim Crichton wrote: > Please correct me if I am wrong, Jean-Marc, but I do not think that > any patches are getting applied to 1.0.5 anyway. Also, if you expect > a patch to be applied, you...

...5 June. The problem is in lsp.c, function lsp_to_lpc(). The line (lsp.c line 461 in build 11700): xin = 1<<(QIMP-1); /* 0.5 in QIMP format */ evaluates to zero. The following change corrects the problem: xin = SHL32(EXTEND32(1), (QIMP-1)); /* 0.5 in QIMP format */ Regards, Jim Crichton -------------- next part -------------- An HTML attachment was scrubbed... URL: http://lists.xiph.org/pipermail/speex-dev/attachments/20060724/03c37193/attachment.htm

...ds the configuration for the filter, sized as spx_int32_t. I also included the modified files (I know that you have had some trouble with patches that I have sent before). - Jim ----- Original Message ----- From: "Jean-Marc Valin" <jean-marc.valin@usherbrooke.ca> To: "Jim Crichton" <jim.crichton@comcast.net> Cc: <speex-dev@xiph.org> Sent: Tuesday, August 22, 2006 8:12 PM Subject: Re: [Speex-dev] Please test upcoming release > Hi Jim, > > Actually, I don't see anything wrong with the internal structure having > a different type than the inte...

...so, please post the c5X-specific files to the list (.cmd, .pjt, ...) so they'll be archived. Last thing, I see you defined spx_word64_t as long long for the C5X, isn't there a way to tell the compiler that the 40-bit accumulator is fine? Jean-Marc Le jeudi 18 ao?t 2005 ? 16:18 -0400, Jim Crichton a ?crit : > Jean-Marc, > > I have attached a small patch with modifications to arch.h, bits.c, and > misc.c. This contains the few mods remaining to support the various fixed > point TI DSPs after the work that you did at the end of May (thank you for > this). > > arch...

...P FIR Filter: 4*floor((nh-1)/2)+14)*(ceil(nr/4)) + 8 SP Inner product: nx/2 + 25 But unless the whole algorithm gets down near 10MIPS, I'm going to have to go to the 64xx fixed point. Jerry J. Trantow Applied Signal Processing, Inc. jtrantow@ieee.org -----Original Message----- From: Jim Crichton [mailto:jim.crichton@comcast.net] Sent: Thursday, January 19, 2006 10:33 AM To: Jerry Trantow Cc: speex-dev@xiph.org Subject: Re: [Speex-dev] TI 6xxx platform performance Jerry, I think that just removing the FIXED_POINT define should be sufficient, though you mind want to turn off MANUAL_ALLOC...

...g it easier to tune memory usage. Thanks to Jamey Hicks for the original C5x patch last year, which took care of most of the compiler-specific issues (mainly related to the 16-bit char size on the C5x DSPs). I hope that these builds prove useful to some prospective Speex users. Regards, Jim Crichton -------------- next part -------------- A non-text attachment was scrubbed... Name: speex-1-1-10 TI DSP build.zip Type: application/x-zip-compressed Size: 22144 bytes Desc: not available Url : http://lists.xiph.org/pipermail/speex-dev/attachments/20051017/53059da7/speex-1-1-10TIDSPbuild-0001.bin

For a streaming application like VOIP, you collect 20 ms of samples, feed this through the encoder, stick it in an RTP packet, and send if over the network. On the receive side you feed packets through a jitter buffer to the decoder, and then copy the output audio to your output device. Speex runs efficiently on most compilers, so the real-time requirement is not a big deal, as long as you

...idle loop but it's close enough so that it should be able to work the same right? the only deviation I do from the testenc is I don't use the callbacks, set the complexity, or disable VBR and I have my own bit buffer so I don't rewind or reset the bits. -Mike >>> "Jim Crichton" <jim.crichton@comcast.net> 07/24/07 3:19 PM >>> Mike, Generally "Invalid mode encounterd" == "frames are misaligned" You should be getting 20 bytes from the encoder each time, and passing 20 bytes to the decoder each time. Is it correct that you have m...

Hi, My name is Jean Quirion and I am a DSP engineer. Currently I am working on a project where it is desired to implement a VoIP solution over a GSM GPRS link. I would like to use Speex as the vocoder for this application. This application would require the Speex encoder/decoder and possibly the pre-processor to run on a low power fixed-point DSP such as a TI C55x. Thus, I am interested in

...often not true on DSPs (at least on Blackfin), so it is possible to re-write the algorithm to take that into account. There is an example (in C) for filter_mem2() and compute_impulse_response() at the end of filters_bfin.h. Hope this helps. Jean-Marc Le mercredi 17 ao?t 2005 ? 16:41 -0400, Jim Crichton a ?crit : > The 42 MIP number was from my post of 24 May, and it was for C55x and > Speex 1.1.8. I tested 1.1.10 yesterday, and the number went down very > slightly; it was just over 41 MIPs peak measured in 20 ms blocks on > MALE.WAV, for encoder/decoder loop, 8kbps narrowband with mi...