Message ID | 1337087801-31527-1-git-send-email-gennarone@gmail.com (mailing list archive) |
---|---|
State | Rejected, archived |
Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1SUHcs-0007kV-Ed for patchwork@linuxtv.org; Tue, 15 May 2012 15:17:18 +0200 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.75/mailfrontend-2) with esmtp for <patchwork@linuxtv.org> id 1SUHcr-00074c-IU; Tue, 15 May 2012 15:17:18 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932807Ab2EONRO (ORCPT <rfc822;patchwork@linuxtv.org>); Tue, 15 May 2012 09:17:14 -0400 Received: from mail-wi0-f172.google.com ([209.85.212.172]:38520 "EHLO mail-wi0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932282Ab2EONRN (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 15 May 2012 09:17:13 -0400 Received: by wibhj8 with SMTP id hj8so2791199wib.1 for <linux-media@vger.kernel.org>; Tue, 15 May 2012 06:17:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:x-mailer; bh=pL9oPKZxov79R3qgkwRNw0kMotYJz90JkcUOzBEkldA=; b=u2EVFwlzlaRmRYP+Qr2VdqSikdsxBz2i8axCiuKIEZPCQbUH1LrBxNhZi+pykGstgb WtSZzKhoruJj6ys3BQGm1ax8xBKzIMUNhR4pIHJVJMibaPFY/4RttZAPo0QAcg41KYDu HbG1ZBCzbH6Y2YyW+629bTt2fBsruWlV1kV7LGVOE5vze2cdaZXLk2aoBu4WhHxdciaj fJkARspYNmaOQN5JnoagD7Wj5frXpNfWuZC5/p94/ingjthZnOMtnBxi9CoqLyYObKE/ vYb5VyZuHQodMu2NpYussvNVdpW4tRLYE3amoBUxyp5wSaWhOgdFcT0JuBd1G27bUctd CaSA== Received: by 10.180.107.101 with SMTP id hb5mr30292125wib.7.1337087832300; Tue, 15 May 2012 06:17:12 -0700 (PDT) Received: from localhost.localdomain (93-50-34-119.ip150.fastwebnet.it. [93.50.34.119]) by mx.google.com with ESMTPS id ga8sm6183613wib.5.2012.05.15.06.17.04 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 15 May 2012 06:17:11 -0700 (PDT) From: Gianluca Gennari <gennarone@gmail.com> To: linux-media@vger.kernel.org, mchehab@redhat.com, hans.verkuil@cisco.com Cc: Gianluca Gennari <gennarone@gmail.com> Subject: [PATCH] media_build: add fixp-arith.h in linux/include/linux Date: Tue, 15 May 2012 15:16:41 +0200 Message-Id: <1337087801-31527-1-git-send-email-gennarone@gmail.com> X-Mailer: git-send-email 1.7.0.4 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 5.6.1.2065439, Antispam-Engine: 2.7.2.376379, Antispam-Data: 2012.5.15.130317 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' FORGED_FROM_GMAIL 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODY_SIZE_3000_3999 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __C230066_P2 0, __CP_URI_IN_BODY 0, __FRAUD_BODY_WEBMAIL 0, __FRAUD_WEBMAIL 0, __FRAUD_WEBMAIL_FROM 0, __FROM_GMAIL 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __MIME_TEXT_ONLY 0, __PHISH_SPEAR_STRUCTURE_1 0, __SANE_MSGID 0, __STOCK_PHRASE_7 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS ' |
Commit Message
Gianluca Gennari
May 15, 2012, 1:16 p.m. UTC
This patch:
http://patchwork.linuxtv.org/patch/10824/
moved the file fixp-arith.h from drivers/input/ to include/linux/ .
To make this file available to old kernels, we must include it in the
media_build package.
The version included here comes from kernel 3.4-rc7.
This patch corrects the following build error:
media_build/v4l/ov534.c:38:30: error: linux/fixp-arith.h: No such file or directory
media_build/v4l/ov534.c: In function 'sethue':
media_build/v4l/ov534.c:1000: error: implicit declaration of function 'fixp_sin'
media_build/v4l/ov534.c:1001: error: implicit declaration of function 'fixp_cos'
Tested on kernel 2.6.32-41-generic-pae (Ubuntu 10.04).
Signed-off-by: Gianluca Gennari <gennarone@gmail.com
---
linux/include/linux/fixp-arith.h | 87 ++++++++++++++++++++++++++++++++++++++
1 files changed, 87 insertions(+), 0 deletions(-)
create mode 100644 linux/include/linux/fixp-arith.h
Comments
Em 15-05-2012 10:16, Gianluca Gennari escreveu: > This patch: > http://patchwork.linuxtv.org/patch/10824/ > moved the file fixp-arith.h from drivers/input/ to include/linux/ . > > To make this file available to old kernels, we must include it in the > media_build package. > > The version included here comes from kernel 3.4-rc7. > > This patch corrects the following build error: > > media_build/v4l/ov534.c:38:30: error: linux/fixp-arith.h: No such file or directory > media_build/v4l/ov534.c: In function 'sethue': > media_build/v4l/ov534.c:1000: error: implicit declaration of function 'fixp_sin' > media_build/v4l/ov534.c:1001: error: implicit declaration of function 'fixp_cos' > > Tested on kernel 2.6.32-41-generic-pae (Ubuntu 10.04). > > Signed-off-by: Gianluca Gennari <gennarone@gmail.com > --- > linux/include/linux/fixp-arith.h | 87 ++++++++++++++++++++++++++++++++++++++ It is not that simple, as make clean will remove it. I can think on a few possible solutions for it: 1) just don't compile ov534 on older kernels; 2) add a backport patch that will dynamically create it; 3) add linux/include/linux/fixp-arith.h inside the tarball with: TARFILES += include/linux/fixp-arith.h Eventually, you can also tweak with the building system, but it doesn't sound a good idea to keep this header there as-is for kernels > 3.4, as some changes on this header can be added there. From all above, (3) is the simpler one. I'll apply it. Regards, Mauro > 1 files changed, 87 insertions(+), 0 deletions(-) > create mode 100644 linux/include/linux/fixp-arith.h > > diff --git a/linux/include/linux/fixp-arith.h b/linux/include/linux/fixp-arith.h > new file mode 100644 > index 0000000..3089d73 > --- /dev/null > +++ b/linux/include/linux/fixp-arith.h > @@ -0,0 +1,87 @@ > +#ifndef _FIXP_ARITH_H > +#define _FIXP_ARITH_H > + > +/* > + * Simplistic fixed-point arithmetics. > + * Hmm, I'm probably duplicating some code :( > + * > + * Copyright (c) 2002 Johann Deneux > + */ > + > +/* > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, write to the Free Software > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA > + * > + * Should you need to contact me, the author, you can do so by > + * e-mail - mail your message to <johann.deneux@gmail.com> > + */ > + > +#include <linux/types.h> > + > +/* The type representing fixed-point values */ > +typedef s16 fixp_t; > + > +#define FRAC_N 8 > +#define FRAC_MASK ((1<<FRAC_N)-1) > + > +/* Not to be used directly. Use fixp_{cos,sin} */ > +static const fixp_t cos_table[46] = { > + 0x0100, 0x00FF, 0x00FF, 0x00FE, 0x00FD, 0x00FC, 0x00FA, 0x00F8, > + 0x00F6, 0x00F3, 0x00F0, 0x00ED, 0x00E9, 0x00E6, 0x00E2, 0x00DD, > + 0x00D9, 0x00D4, 0x00CF, 0x00C9, 0x00C4, 0x00BE, 0x00B8, 0x00B1, > + 0x00AB, 0x00A4, 0x009D, 0x0096, 0x008F, 0x0087, 0x0080, 0x0078, > + 0x0070, 0x0068, 0x005F, 0x0057, 0x004F, 0x0046, 0x003D, 0x0035, > + 0x002C, 0x0023, 0x001A, 0x0011, 0x0008, 0x0000 > +}; > + > + > +/* a: 123 -> 123.0 */ > +static inline fixp_t fixp_new(s16 a) > +{ > + return a<<FRAC_N; > +} > + > +/* a: 0xFFFF -> -1.0 > + 0x8000 -> 1.0 > + 0x0000 -> 0.0 > +*/ > +static inline fixp_t fixp_new16(s16 a) > +{ > + return ((s32)a)>>(16-FRAC_N); > +} > + > +static inline fixp_t fixp_cos(unsigned int degrees) > +{ > + int quadrant = (degrees / 90) & 3; > + unsigned int i = degrees % 90; > + > + if (quadrant == 1 || quadrant == 3) > + i = 90 - i; > + > + i >>= 1; > + > + return (quadrant == 1 || quadrant == 2)? -cos_table[i] : cos_table[i]; > +} > + > +static inline fixp_t fixp_sin(unsigned int degrees) > +{ > + return -fixp_cos(degrees + 90); > +} > + > +static inline fixp_t fixp_mult(fixp_t a, fixp_t b) > +{ > + return ((s32)(a*b))>>FRAC_N; > +} > + > +#endif -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Il 15/05/2012 15:25, Mauro Carvalho Chehab ha scritto: > Em 15-05-2012 10:16, Gianluca Gennari escreveu: >> This patch: >> http://patchwork.linuxtv.org/patch/10824/ >> moved the file fixp-arith.h from drivers/input/ to include/linux/ . >> >> To make this file available to old kernels, we must include it in the >> media_build package. >> >> The version included here comes from kernel 3.4-rc7. >> >> This patch corrects the following build error: >> >> media_build/v4l/ov534.c:38:30: error: linux/fixp-arith.h: No such file or directory >> media_build/v4l/ov534.c: In function 'sethue': >> media_build/v4l/ov534.c:1000: error: implicit declaration of function 'fixp_sin' >> media_build/v4l/ov534.c:1001: error: implicit declaration of function 'fixp_cos' >> >> Tested on kernel 2.6.32-41-generic-pae (Ubuntu 10.04). >> >> Signed-off-by: Gianluca Gennari <gennarone@gmail.com >> --- >> linux/include/linux/fixp-arith.h | 87 ++++++++++++++++++++++++++++++++++++++ > > It is not that simple, as make clean will remove it. > > I can think on a few possible solutions for it: > 1) just don't compile ov534 on older kernels; > 2) add a backport patch that will dynamically create it; > 3) add linux/include/linux/fixp-arith.h inside the tarball with: > TARFILES += include/linux/fixp-arith.h > > Eventually, you can also tweak with the building system, but it doesn't sound a good > idea to keep this header there as-is for kernels > 3.4, as some changes on this header > can be added there. > >>From all above, (3) is the simpler one. I'll apply it. > > Regards, > Mauro > It looks like this file has not been changed in the last years, so chances are it will not change in the future. So adding it in the tarball file looks as a good solution. Best regards, Gianluca -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Em 15-05-2012 10:35, Gianluca Gennari escreveu: > Il 15/05/2012 15:25, Mauro Carvalho Chehab ha scritto: >> Em 15-05-2012 10:16, Gianluca Gennari escreveu: >>> This patch: >>> http://patchwork.linuxtv.org/patch/10824/ >>> moved the file fixp-arith.h from drivers/input/ to include/linux/ . >>> >>> To make this file available to old kernels, we must include it in the >>> media_build package. >>> >>> The version included here comes from kernel 3.4-rc7. >>> >>> This patch corrects the following build error: >>> >>> media_build/v4l/ov534.c:38:30: error: linux/fixp-arith.h: No such file or directory >>> media_build/v4l/ov534.c: In function 'sethue': >>> media_build/v4l/ov534.c:1000: error: implicit declaration of function 'fixp_sin' >>> media_build/v4l/ov534.c:1001: error: implicit declaration of function 'fixp_cos' >>> >>> Tested on kernel 2.6.32-41-generic-pae (Ubuntu 10.04). >>> >>> Signed-off-by: Gianluca Gennari <gennarone@gmail.com >>> --- >>> linux/include/linux/fixp-arith.h | 87 ++++++++++++++++++++++++++++++++++++++ >> >> It is not that simple, as make clean will remove it. >> >> I can think on a few possible solutions for it: >> 1) just don't compile ov534 on older kernels; >> 2) add a backport patch that will dynamically create it; >> 3) add linux/include/linux/fixp-arith.h inside the tarball with: >> TARFILES += include/linux/fixp-arith.h >> >> Eventually, you can also tweak with the building system, but it doesn't sound a good >> idea to keep this header there as-is for kernels > 3.4, as some changes on this header >> can be added there. >> >> >From all above, (3) is the simpler one. I'll apply it. >> >> Regards, >> Mauro >> > > > It looks like this file has not been changed in the last years, so > chances are it will not change in the future. So adding it in the > tarball file looks as a good solution. It isn't, as it breaks the build system: make -C linux dir=<some git tree> breaks, and also ./build --main-git breaks, as both assume that the files under /linux/foo/ are temporary files that they can rewrite/delete/etc anytime. Regards, Mauro > > Best regards, > Gianluca -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, 15 May 2012 15:35:15 +0200 Gianluca Gennari <gennarone@gmail.com> wrote: > Il 15/05/2012 15:25, Mauro Carvalho Chehab ha scritto: > > Em 15-05-2012 10:16, Gianluca Gennari escreveu: > >> This patch: > >> http://patchwork.linuxtv.org/patch/10824/ > >> moved the file fixp-arith.h from drivers/input/ to include/linux/ . > >> > >> To make this file available to old kernels, we must include it in the > >> media_build package. > >> > >> The version included here comes from kernel 3.4-rc7. > >> > >> This patch corrects the following build error: > >> > >> media_build/v4l/ov534.c:38:30: error: linux/fixp-arith.h: No such file or directory > >> media_build/v4l/ov534.c: In function 'sethue': > >> media_build/v4l/ov534.c:1000: error: implicit declaration of function 'fixp_sin' > >> media_build/v4l/ov534.c:1001: error: implicit declaration of function 'fixp_cos' > >> > >> Tested on kernel 2.6.32-41-generic-pae (Ubuntu 10.04). > >> > >> Signed-off-by: Gianluca Gennari <gennarone@gmail.com > >> --- > >> linux/include/linux/fixp-arith.h | 87 ++++++++++++++++++++++++++++++++++++++ > > > > It is not that simple, as make clean will remove it. > > > > I can think on a few possible solutions for it: > > 1) just don't compile ov534 on older kernels; > > 2) add a backport patch that will dynamically create it; > > 3) add linux/include/linux/fixp-arith.h inside the tarball with: > > TARFILES += include/linux/fixp-arith.h > > > > Eventually, you can also tweak with the building system, but it doesn't sound a good > > idea to keep this header there as-is for kernels > 3.4, as some changes on this header > > can be added there. > > > >>From all above, (3) is the simpler one. I'll apply it. > > [...] > > > It looks like this file has not been changed in the last years, so > chances are it will not change in the future. So adding it in the > tarball file looks as a good solution. > Hi, I just wanted to mention that it has been proposed to move part of include/linux/fixp-arith.h to a .c file (maybe under lib/) in order to share some code between the users, which are now 2 (drivers/input/ff-memless.c and drivers/media/video/gspca/ov534.c). I don't know yet if I'll do it or when it will be done but the file _might_ change not too far in the future. Regards, Antonio
Em 15-05-2012 11:25, Antonio Ospite escreveu: > On Tue, 15 May 2012 15:35:15 +0200 > Gianluca Gennari <gennarone@gmail.com> wrote: > >> Il 15/05/2012 15:25, Mauro Carvalho Chehab ha scritto: >>> Em 15-05-2012 10:16, Gianluca Gennari escreveu: >>>> This patch: >>>> http://patchwork.linuxtv.org/patch/10824/ >>>> moved the file fixp-arith.h from drivers/input/ to include/linux/ . >>>> >>>> To make this file available to old kernels, we must include it in the >>>> media_build package. >>>> >>>> The version included here comes from kernel 3.4-rc7. >>>> >>>> This patch corrects the following build error: >>>> >>>> media_build/v4l/ov534.c:38:30: error: linux/fixp-arith.h: No such file or directory >>>> media_build/v4l/ov534.c: In function 'sethue': >>>> media_build/v4l/ov534.c:1000: error: implicit declaration of function 'fixp_sin' >>>> media_build/v4l/ov534.c:1001: error: implicit declaration of function 'fixp_cos' >>>> >>>> Tested on kernel 2.6.32-41-generic-pae (Ubuntu 10.04). >>>> >>>> Signed-off-by: Gianluca Gennari <gennarone@gmail.com >>>> --- >>>> linux/include/linux/fixp-arith.h | 87 ++++++++++++++++++++++++++++++++++++++ >>> >>> It is not that simple, as make clean will remove it. >>> >>> I can think on a few possible solutions for it: >>> 1) just don't compile ov534 on older kernels; >>> 2) add a backport patch that will dynamically create it; >>> 3) add linux/include/linux/fixp-arith.h inside the tarball with: >>> TARFILES += include/linux/fixp-arith.h >>> >>> Eventually, you can also tweak with the building system, but it doesn't sound a good >>> idea to keep this header there as-is for kernels > 3.4, as some changes on this header >>> can be added there. >>> >>> >From all above, (3) is the simpler one. I'll apply it. Applied, and tarball updated at linuxtv.org. The only thing left is the CONFIG_VIDEO_SMIAPP stuff. Sakari should be writing a fix for it today or tomorrow. >>> > [...] >> >> >> It looks like this file has not been changed in the last years, so >> chances are it will not change in the future. So adding it in the >> tarball file looks as a good solution. >> > > Hi, > > I just wanted to mention that it has been proposed to move part of > include/linux/fixp-arith.h to a .c file (maybe under lib/) in order to > share some code between the users, which are now 2 > (drivers/input/ff-memless.c and drivers/media/video/gspca/ov534.c). That makes sense. > I don't know yet if I'll do it or when it will be done but the file > _might_ change not too far in the future. No problem. If it changes to a *.c file, all we need to do is to add it at TARFILES. > > Regards, > Antonio > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/linux/include/linux/fixp-arith.h b/linux/include/linux/fixp-arith.h new file mode 100644 index 0000000..3089d73 --- /dev/null +++ b/linux/include/linux/fixp-arith.h @@ -0,0 +1,87 @@ +#ifndef _FIXP_ARITH_H +#define _FIXP_ARITH_H + +/* + * Simplistic fixed-point arithmetics. + * Hmm, I'm probably duplicating some code :( + * + * Copyright (c) 2002 Johann Deneux + */ + +/* + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + * + * Should you need to contact me, the author, you can do so by + * e-mail - mail your message to <johann.deneux@gmail.com> + */ + +#include <linux/types.h> + +/* The type representing fixed-point values */ +typedef s16 fixp_t; + +#define FRAC_N 8 +#define FRAC_MASK ((1<<FRAC_N)-1) + +/* Not to be used directly. Use fixp_{cos,sin} */ +static const fixp_t cos_table[46] = { + 0x0100, 0x00FF, 0x00FF, 0x00FE, 0x00FD, 0x00FC, 0x00FA, 0x00F8, + 0x00F6, 0x00F3, 0x00F0, 0x00ED, 0x00E9, 0x00E6, 0x00E2, 0x00DD, + 0x00D9, 0x00D4, 0x00CF, 0x00C9, 0x00C4, 0x00BE, 0x00B8, 0x00B1, + 0x00AB, 0x00A4, 0x009D, 0x0096, 0x008F, 0x0087, 0x0080, 0x0078, + 0x0070, 0x0068, 0x005F, 0x0057, 0x004F, 0x0046, 0x003D, 0x0035, + 0x002C, 0x0023, 0x001A, 0x0011, 0x0008, 0x0000 +}; + + +/* a: 123 -> 123.0 */ +static inline fixp_t fixp_new(s16 a) +{ + return a<<FRAC_N; +} + +/* a: 0xFFFF -> -1.0 + 0x8000 -> 1.0 + 0x0000 -> 0.0 +*/ +static inline fixp_t fixp_new16(s16 a) +{ + return ((s32)a)>>(16-FRAC_N); +} + +static inline fixp_t fixp_cos(unsigned int degrees) +{ + int quadrant = (degrees / 90) & 3; + unsigned int i = degrees % 90; + + if (quadrant == 1 || quadrant == 3) + i = 90 - i; + + i >>= 1; + + return (quadrant == 1 || quadrant == 2)? -cos_table[i] : cos_table[i]; +} + +static inline fixp_t fixp_sin(unsigned int degrees) +{ + return -fixp_cos(degrees + 90); +} + +static inline fixp_t fixp_mult(fixp_t a, fixp_t b) +{ + return ((s32)(a*b))>>FRAC_N; +} + +#endif