List supported burn speeds for CD writers

Danny Kukawka danny.kukawka at web.de
Sun Jan 15 11:34:05 PST 2006


On Friday 13 January 2006 11:34, Danny Kukawka wrote:
> On Friday 13 January 2006 07:16, Ryan Lortie wrote:
> > I've written a patch that publishes the list of burn speeds that a drive
> > supports as the 'write_speeds' property attached to CD burners.  It
> > lists the (unique) supported speeds as a descending comma-separated
> > list.  For example:
> >
> >   storage.cdrom.write_speeds = '7056,5645,4234,2822,2117'  (string)
> >
> > I had to do it this way since hal doesn't support the concept of lists
> > of integers and the libhal interface doesn't even allow you to use lists
> > of strings (in any case, I believe a comma-separated string to be better
> > than a list of strings - far less overhead).
>
> This is not right. We have already list of strings: strlist. You can use
> them instead of a comma.seperated string.

I attached a new version of the patch with strlist instead of comma-separated 
string. Also updated the spec for this new key.

@David: we should integrate this in 0.5.6

Cheers,

Danny

 doc/spec/hal-spec.xml.in                 |    6 +
 hald/linux2/probing/linux_dvd_rw_utils.c |   94 ++++++++++++++++++++-
 hald/linux2/probing/linux_dvd_rw_utils.h |    2
 hald/linux2/probing/probe-storage.c      |   16 +++
 4 files changed, 114 insertions(+), 4 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-add_cd_write_speeds_v2.diff
Type: text/x-diff
Size: 6184 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20060115/7373ecc8/hal-add_cd_write_speeds_v2.bin


More information about the hal mailing list