[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] media_build: add fixp-arith.h in linux/include/linux



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@xxxxxxxxx
>> ---
>>  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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux Input]     [Video for Linux]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Photos]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]     [Yosemite Backpacking]

Add to Google Powered by Linux