----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123262/#review78538 -----------------------------------------------------------
daemon/backends/upower/ddchelper.cpp (line 139) <https://git.reviewboard.kde.org/r/123262/#comment53730> Hmmm I do care about if this worked. People usually complains when changing brightness does not work, see https://bugs.kde.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=RESOLVED&bug_status=CLOSED&component=powermanagement&component=powermanagement-daemon&component=powermanagement-kcm&list_id=1240974&query_format=advanced&short_desc=brightness%20work&short_desc_type=allwordssubstr - Lamarque Souza On April 5, 2015, 5:31 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123262/ > ----------------------------------------------------------- > > (Updated April 5, 2015, 5:31 p.m.) > > > Review request for Solid. > > > Repository: powerdevil > > > Description > ------- > > This allows PowerDevil to use the ddccontrol utility to manipulate brightness > of external monitors through Display Data Channel. > > It becomes the last link in the fallback chain; it first tries XRandR, then > it tries the sysfs helper, and if all that fails it tries to use the > ddccontrol utility. > > dcccontrol unfortunately doesn't have machine-readable output so I have to > take apart its probing output to get the device name and id of the backlight > controls. Also it is pretty slow both initializing (kded startup, the helper > saves the values and writes directly to the address returned by the initial > probing) and setting brightness (that one doesn't block but it can take > seconds until your monitor actually changes brightness) > > > Diffs > ----- > > daemon/BackendConfig.cmake 295a8a2 > daemon/backends/upower/ddc_helper_actions.actions PRE-CREATION > daemon/backends/upower/ddchelper.h PRE-CREATION > daemon/backends/upower/ddchelper.cpp PRE-CREATION > daemon/backends/upower/powerdevilupowerbackend.h 1c4dd59 > daemon/backends/upower/powerdevilupowerbackend.cpp 87b9cc7 > > Diff: https://git.reviewboard.kde.org/r/123262/diff/ > > > Testing > ------- > > It's working pretty nicely, I can adjust brightness of my desktop monitor > through battery monitor and have it automatically dimmed after a timeout. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
