----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110384/#review32433 -----------------------------------------------------------
Ship it! Code looks good, functionality is needed so for me this is a ship it. Wait until at least lukas replies though. - Àlex Fiestas On May 12, 2013, 12:48 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110384/ > ----------------------------------------------------------- > > (Updated May 12, 2013, 12:48 p.m.) > > > Review request for Solid. > > > Description > ------- > > This patch adds support for the "power supply" property in UPower batteries > that tells if a battery is actually powering the machine (ie. laptop battery) > or just inside a peripheral device (ie. mouse, keyboard, ..) > > In the UDev backend it reads the power supply property, in HAL, which doesn't > support this, we're assuming that primary battery and UPS battery are power > supply and others are not (which is the current way PowerDevil for example > determins whether a computer has a battery or not). Can be somewhat > problematic as not all mice report themselves as mouse battery. > > This patch will in the end allow for example Plasma's battery monitor to only > average battery percentage of batteries that are actually powering the > machine, or PowerDevil to emit a separate message when your mouse battery > goes low. > > > Diffs > ----- > > solid/solid/backends/fakehw/fakebattery.cpp 0afe765 > solid/solid/backends/fakehw/fakecomputer.xml fe7e323 > solid/solid/backends/hal/halbattery.h e2f38a0 > solid/solid/backends/hal/halbattery.cpp 4c530da > solid/solid/backends/upower/upowerbattery.h 9d52f7b > solid/solid/backends/upower/upowerbattery.cpp 1a40a2a > solid/solid/battery.h 3afac6f > solid/solid/battery.cpp 37d7321 > solid/solid/ifaces/battery.h 9987d18 > solid/solid/backends/fakehw/fakebattery.h c7e2791 > > Diff: http://git.reviewboard.kde.org/r/110384/diff/ > > > Testing > ------- > > After adjusting Plasma's powermanagement dataengine and battery monitor, I > had it show two batteries (laptop battery and bluetooth mouse) and only take > the laptop battery for battery percentage. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
