From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Prashant Laddha (prladdha)" Subject: Re: [RFC PATCH] fixp-arith: replace sin/cos table by a better precision one Date: Wed, 17 Dec 2014 08:17:38 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Return-path: Received: from rcdn-iport-7.cisco.com ([173.37.86.78]:43793 "EHLO rcdn-iport-7.cisco.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751692AbaLQI1O convert rfc822-to-8bit (ORCPT ); Wed, 17 Dec 2014 03:27:14 -0500 In-Reply-To: Content-Language: en-US Content-ID: <9FCEB9962CD0C44D90DB8D471AD06F43@emea.cisco.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Mauro Carvalho Chehab , Linux Media Mailing List Cc: Mauro Carvalho Chehab , Dmitry Torokhov , Hans de Goede , "linux-input@vger.kernel.org" Thanks for the patch, Mauro. Just a correction below. > >+/* cos(x) = sin(x + pi radians) */ >+ This should pi / 2. Correcting for the same below. >+#define fixp_cos32_rad(rad, twopi) \ >+ fixp_sin32_rad(rad + twopi/2, twopi) fixp_sin32_rad(rad + twopi/4, twopi) >+ I think this patch will serve the need. I will test it for vivid sir tone generation. I will rework my patches to use sin/cos functions from fixp-arith.h. >