From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:3680 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932115Ab2EOOFh (ORCPT ); Tue, 15 May 2012 10:05:37 -0400 Message-ID: <4FB262AD.60107@redhat.com> Date: Tue, 15 May 2012 11:05:33 -0300 From: Mauro Carvalho Chehab MIME-Version: 1.0 To: gennarone@gmail.com CC: linux-media@vger.kernel.org, hans.verkuil@cisco.com Subject: Re: [PATCH] media_build: add fixp-arith.h in linux/include/linux References: <1337087801-31527-1-git-send-email-gennarone@gmail.com> <4FB2593B.3030402@redhat.com> <4FB25B93.6080508@gmail.com> In-Reply-To: <4FB25B93.6080508@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: 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 >> --- >>> 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= 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