Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-16 Thread Mauro Carvalho Chehab
Em Tue, 16 Dec 2014 08:45:53 -0200 Mauro Carvalho Chehab escreveu: > Em Tue, 16 Dec 2014 06:41:18 + > "Prashant Laddha (prladdha)" escreveu: > > > Antti, Mauro, > > > > Thanks for your comments. > > > > On 15/12/14 7:00 pm, "Antti Palosaari" wrote: > > > > > > >On 12/15/2014 03:13 PM,

Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-16 Thread Mauro Carvalho Chehab
Em Tue, 16 Dec 2014 06:41:18 + "Prashant Laddha (prladdha)" escreveu: > Antti, Mauro, > > Thanks for your comments. > > On 15/12/14 7:00 pm, "Antti Palosaari" wrote: > > > >On 12/15/2014 03:13 PM, Mauro Carvalho Chehab wrote: > >> Em Mon, 15 Dec 2014 14:49:17 +0530 > >> Prashant Laddha

Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-15 Thread Prashant Laddha (prladdha)
Antti, Mauro, Thanks for your comments. On 15/12/14 7:00 pm, "Antti Palosaari" wrote: >On 12/15/2014 03:13 PM, Mauro Carvalho Chehab wrote: >> Em Mon, 15 Dec 2014 14:49:17 +0530 >> Prashant Laddha escreveu: >> >>> Replaced Taylor series calculation for (co)sine with a >>> look up table (LUT)

Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-15 Thread Antti Palosaari
On 12/15/2014 03:13 PM, Mauro Carvalho Chehab wrote: Em Mon, 15 Dec 2014 14:49:17 +0530 Prashant Laddha escreveu: Replaced Taylor series calculation for (co)sine with a look up table (LUT) for sine values. Kernel has already a LUT for sin/cos at: include/linux/fixp-arith.h The best

Re: [PATCH 1/6] Use LUT based implementation for (co)sine functions

2014-12-15 Thread Mauro Carvalho Chehab
Em Mon, 15 Dec 2014 14:49:17 +0530 Prashant Laddha escreveu: > Replaced Taylor series calculation for (co)sine with a > look up table (LUT) for sine values. Kernel has already a LUT for sin/cos at: include/linux/fixp-arith.h The best would be to either use it or improve its precision, i