Bug 17188 - V4L version information insufficient
Summary: V4L version information insufficient
Status: NEW
Alias: None
Product: hal
Classification: Unclassified
Component: misc (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: David Zeuthen (not reading bugmail)
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-18 07:11 UTC by Sven Herzberg
Modified: 2008-09-15 02:51 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Proposed Patch (4.60 KB, patch)
2008-08-22 15:28 UTC, Sven Herzberg
Details | Splinter Review
Proposed Patch (4.60 KB, patch)
2008-08-22 20:33 UTC, Sven Herzberg
Details | Splinter Review

Description Sven Herzberg 2008-08-18 07:11:15 UTC
Currently the video4linux.version string is described as "MAX([supported versions])", which is not good. And here's one concrete example followed by a hypothetic example.

1. Camorama
Camorama only supports video4linux version 1 (right now, of course version 2 support is planned). As of today, there's no chance to find out if a webcam supports video4linux 1 (because, as soon as it supports version 2, the supported version will get increased to "2").

2. Upcoming Changes
So, right now, one could argue, "implement v4l2 and be happy"; but then, there's no guarantee that there won't be a video4linux 3 API and then the same issues would pop up again.

So here's my proposal: Keep the key as it is (but deprecate it's usage, tell people it will disappear in the future); and add "video4linux.versions" which returns an array of int (so I can check if that array contains "1" and be happy).
Comment 1 Sven Herzberg 2008-08-22 20:33:01 UTC
Created attachment 18473 [details] [review]
Proposed Patch

Here's a patch I propose. It changes the hald probe tool and updates the specification.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.