Video4Linux device capabilities

[Guillem Jover]

Hi David,

On Tue, 2007-05-01 at 21:56:15 -0400, ext David Zeuthen wrote:
> On Tue, 2007-04-24 at 00:47 +0300, Guillem Jover wrote:
> > I was checking the v4l support as the n770 has a radio and webcam, and
> > the n800 has a webcam, and saw that some time ago [0] the capabilities
> > of the v4l devices were exposed through some keys under the multimedia
> > namespace, but that was removed [1] after some cleanup, the only reason
> > I could find in the changelog was that the functionality was going to
> > be reimplemented in the new code, and then it was forgotten?
> > 
> > If that's it, I could ressurrect the code by Kay and put the v4l
> > capabilities namespacesd in the info.capabilities key, after
> > discussion of which cap names would be appropriate.
> 
> This works fine for me though
> 
>  http://people.freedesktop.org/~david/hal-v4l.png

Right, but I was talking about the capabilities as in:

udi = '/org/freedesktop/Hal/devices/platform_omap24xxcam_video4linux'
  info.capabilities = {'video4linux', 'video4linux.video_capture'} (string list)
  info.category = 'video4linux'  (string)
  info.parent = '/org/freedesktop/Hal/devices/platform_omap24xxcam'  (string)
  info.product = 'omap24xxcam'  (string)
  info.udi = '/org/freedesktop/Hal/devices/platform_omap24xxcam_video4linux'  (string)
  linux.device_file = '/dev/video0'  (string)
  linux.hotplug_type = 2  (0x2)  (int)
  linux.subsystem = 'video4linux'  (string)
  linux.sysfs_path = '/sys/class/video4linux/video0'  (string)
  video4linux.device = '/dev/video0'  (string)
  video4linux.version = '2'  (string)

or:

udi = '/org/freedesktop/Hal/devices/platform_i2c_omap_2_video4linux'
  info.capabilities = {'video4linux', 'video4linux.tuner'} (string list)
  info.category = 'video4linux'  (string)
  info.parent = '/org/freedesktop/Hal/devices/platform_i2c_omap_2'  (string)
  info.product = 'Multimedia Device'  (string)
  info.udi = '/org/freedesktop/Hal/devices/platform_i2c_omap_2_video4linux'  (string)
  linux.device_file = '/dev/radio0'  (string)
  linux.hotplug_type = 2  (0x2)  (int)
  linux.subsystem = 'video4linux'  (string)
  linux.sysfs_path = '/sys/class/video4linux/radio0'  (string)
  video4linux.device = '/dev/radio0'  (string)
  video4linux.version = '2'  (string)

With the new 'video4linux.video_capture' and 'video4linux.tuner' caps
and 'video4linux.version' key.

I'm attaching the patch I've cooked, if it seems fine to you, I'll repost,
as a proper git format-patch and proper dbus error code checks.

regards,
guillem
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal_v4l_caps.patch
Type: text/x-diff
Size: 7296 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20070510/7236fa5f/attachment.patch