Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 187992 - media-video/gspcav1 needs check for CONFIG_VIDEO_V4L1_COMPAT
Summary: media-video/gspcav1 needs check for CONFIG_VIDEO_V4L1_COMPAT
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Mike Doty (RETIRED)
URL: http://mxhaard.free.fr/spca5xx.html
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-07 11:21 UTC by Mike McQuaid
Modified: 2009-10-25 13:59 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Actual build.log with the error message (build.log,34.39 KB, text/plain)
2007-11-08 23:26 UTC, Anton Bolshakov 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike McQuaid 2007-08-07 11:21:44 UTC 
If you only have v4l2 support enabled in the kernel but not v4l1 or v4l1_compat the ebuild fails.
Currently the ebuild has CONFIG_CHECK="VIDEO_DEV", but this should also check for either CONFIG_VIDEO_V4L1 or CONFIG_VIDEO_V4L1_COMPAT or at least provide a notice to the user about this on build failure.
Comment 1 Anton Bolshakov 2007-11-08 23:26:27 UTC 
Created attachment 135534 [details]
Actual build.log with the error message

Managed to hit the the same problem.
Comment 2 Dennis Schridde 2007-11-13 13:37:30 UTC 
Run into this as well. The result were errors like "‘VIDEO_PALETTE_RGB565’ undeclared".
Comment 3 Pacho Ramos gentoo-dev 2008-01-16 19:26:23 UTC 
Is CONFIG_VIDEO_V4L1_COMPAT enough for you or are you also needing CONFIG_VIDEO_V4L1 ?

(I have both in my kernel...)
Comment 4 Dennis Schridde 2008-01-16 19:28:57 UTC 
COMPAT works here.
Comment 5 Pacho Ramos gentoo-dev 2008-01-29 12:44:22 UTC 
OK, thanks
Comment 6 Mike McQuaid 2008-02-10 23:09:31 UTC 
Bump. There is already a CONFIG_CHECK line in the ebuild. Would be good to have this for when gspcav1-20071224 is merged. 

Tried to fix it myself but a bit tricker than expected as ideally we want to allow EITHER CONFIG_VIDEO_V4L1 OR CONFIG_VIDEO_V4L1_COMPAT and I can't see a way to do this with the existing CONFIG_CHECK method.

Any ideas?
Comment 7 Pacho Ramos gentoo-dev 2008-02-25 11:54:22 UTC 
I also don't know how to do that, but reading /usr/src/linux/Documentation/feature-removal-schedule.txt:

What:   Video4Linux API 1 ioctls and video_decoder.h from Video devices.
When:   December 2008
Files:  include/linux/video_decoder.h include/linux/videodev.h
Check:  include/linux/video_decoder.h include/linux/videodev.h
Why:    V4L1 AP1 was replaced by V4L2 API during migration from 2.4 to 2.6
        series. The old API have lots of drawbacks and don't provide enough
        means to work with all video and audio standards. The newer API is
        already available on the main drivers and should be used instead.
        Newer drivers should use v4l_compat_translate_ioctl function to handle
        old calls, replacing to newer ones.
        Decoder iocts are using internally to allow video drivers to
        communicate with video decoders. This should also be improved to allow
        V4L2 calls being translated into compatible internal ioctls.
        Compatibility ioctls will be provided, for a while, via 
        v4l1-compat module. 
Who:    Mauro Carvalho Chehab <mchehab@infradead.org>

Then, maybe COMPAT check would be enough
Comment 8 Pacho Ramos gentoo-dev 2009-02-05 10:45:14 UTC 
I think that COMPAT will be better, anyway I dubt this will be fixed as this driver is deprecated in favor of gspcav2 provided by kernel >=2.6.27
Comment 9 Samuli Suominen (RETIRED) gentoo-dev 2009-10-25 13:59:01 UTC 
Removed from tree (bug 159176).