D5381: Add brightness control using ddcutil lib

[Dorian Vogel]

dvogel updated this revision to Diff 14843.
dvogel added a comment.


  Absence of ddcutil on the system is now handled: 
  the ddcbrightness object is still instantiated in powerdevilupowerbackend, however, this is a dummy object, returning isSupported()=FALSE, making powerdevilupowerbackend avoid using ddcutil.
  
  The same brightness is applied to all ddc-capable displays this seems to be the most obvious choice (compared to setting only one monitor) 
  Multi-monitor not tested:
  
  - desktop + 2+ monitors: should work, as ddcutilbrightness supports multiple monitors
  - laptop + external monitor: not tested, when plugging a new monitor while the laptop LCD is already used for brightness ==> detection is just triggered when powerdevil starts, so should be fine; when booting with external monitor attached ==> ddcutil is the last possibility in ddcutilupowerbackend.cpp, the laptop LCD should always be controlled.

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D5381?vs=13586&id=14843

REVISION DETAIL
  https://phabricator.kde.org/D5381

AFFECTED FILES
  CMakeLists.txt
  cmake/FindDDCUtil.cmake
  daemon/backends/CMakeLists.txt
  daemon/backends/upower/ddcutilbrightness.cpp
  daemon/backends/upower/ddcutilbrightness.h
  daemon/backends/upower/powerdevilupowerbackend.cpp
  daemon/backends/upower/powerdevilupowerbackend.h

To: dvogel, broulik
Cc: davidedmundson, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170525/5bda84d7/attachment.html>