By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1588 - tdehwlibrary does not support DeviceKit Power management (suspend does not work in CentOS 6 / R14)
Summary: tdehwlibrary does not support DeviceKit Power management (suspend does not wo...
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdelibs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All Linux
: P5 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks:
 
Reported: 2013-07-24 04:09 CDT by Francois Andriot
Modified: 2013-08-15 13:27 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
tdelibs 14.0.0 : add DeviceKit Power support in TDE HW library (5.44 KB, patch)
2013-07-24 04:11 CDT, Francois Andriot 		Details | Diff
tdelibs 14.0.0 : add DeviceKit Power support in TDE HW library (2) (5.58 KB, patch)
2013-08-12 12:57 CDT, Francois Andriot 		Details | Diff
tdelibs 14.0.0 : fix FTBFS because missing "extern C" in libudev.h (881 bytes, patch)
2013-08-12 12:57 CDT, Francois Andriot 		Details | Diff
Show Obsolete (1) 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 Francois Andriot 2013-07-24 04:09:28 CDT 
Hello, running CentOS 6, the distribution use "DeviceKit Power", which is the ancestor of Upower. 

See: http://upower.freedesktop.org/

The current Upower implementation in TDE does not support the old DeviceKit Power, so the suspend/hibernate features in CentOS 6 do not work anymore. They were working with HAL daemon.
Comment 1 Francois Andriot 2013-07-24 04:11:04 CDT 
Created attachment 1377 [details]
tdelibs 14.0.0 : add DeviceKit Power support in TDE HW library

The attached patch adds DeviceKit Power option to tdelibs.
The main difference (for us) between DeviceKit Power and uPower is the DBUS objects name.
Comment 2 Francois Andriot 2013-08-12 12:57:25 CDT 
Created attachment 1460 [details]
tdelibs 14.0.0 : add DeviceKit Power support in TDE HW library (2)
Comment 3 Francois Andriot 2013-08-12 12:57:52 CDT 
Created attachment 1461 [details]
tdelibs 14.0.0 : fix FTBFS because missing "extern C" in libudev.h
Comment 4 Slávek Banko 2013-08-12 13:18:54 CDT 
Although it is only a change in dbus paths, I recommend either have separate backends, or add a calls so that uPower can support both  paths - similar to the hal supports can_supend and can_suspend_to_ram, can_hibernate and can_suspend_to_disk.

What do you think?
Comment 5 Francois Andriot 2013-08-12 13:50:09 CDT 
Hello, you can do as you prefer, I have no opinion about which method is better.
Comment 6 Slávek Banko 2013-08-15 13:27:08 CDT 
Given that on some distributions was devicekit-power never used, I chose to add a separate backend. This allow to these dbus call might not ever be used on these distributions. And at the same time this allows for the distributions, where it was used both - devicekit-power and also upower, enable both backends simultaneously.

Fixed in GIT hash 18b25099.