[Kde-hardware-devel] Re: Review Request: Fill in vendor and model names using Solid's udev backend when possible

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101270/#review3181
-----------------------------------------------------------


This review has been submitted with commit fdf93b29ffd000b1b45b4d49b8ad3651c85c173f by Lamarque V. Souza.

- Commit


On May 2, 2011, 2:31 p.m., Lamarque Vieira Souza wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101270/
> -----------------------------------------------------------
> 
> (Updated May 2, 2011, 2:31 p.m.)
> 
> 
> Review request for Solid, Kevin Ottens and Alex Fiestas.
> 
> 
> Summary
> -------
> 
> solid-hardware does not report my 3G modem's vendor and model names. This patch fix that:
> 
> [lamarque at evolucao ~]$ solid-hardware details /org/kde/solid/udev/sys/devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.1
> udi = '/org/kde/solid/udev/sys/devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.1'
>   parent = '/org/kde/solid/udev'  (string)
>   vendor = 'Sony Ericsson'  (string)
>   product = 'Sony Ericsson MD300'  (string)
>   description = ''  (string)
>   Block.major = 189  (0xbd)  (int)
>   Block.minor = 175  (0xaf)  (int)
>   Block.device = '/dev/bus/usb/002/048'  (string)
> 
> The patch basically tests if /sys/devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.1 exists, if affirmative it creates a UDevDevice, which makes it possible to report the available device details instead of just ignoring them completely.
> 
> 'solid-hardware list' still does not list the device. I am still not sure if I should fix that since it would give the false impression the device is supported.
> 
> The motivation for this change is Plasma NM. The patch allows me inform modem's vendor and model names
> in Mobile Connection Wizard (when creating Gsm/Cdma connections) and in PinDialog (when requesting PIN unlock code from user). It is basically usability fix.
> 
> 
> Diffs
> -----
> 
>   solid/solid/backends/udev/udevdevice.cpp 4f34382 
>   solid/solid/backends/udev/udevmanager.cpp e08fcde 
> 
> Diff: http://git.reviewboard.kde.org/r/101270/diff
> 
> 
> Testing
> -------
> 
> I have been using kdelibs-4.6.2 compiled with the patch since yesterday, not problems so far.
> 
> 
> Thanks,
> 
> Lamarque Vieira
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20110506/77696d7f/attachment.htm