----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122283/#review75556 -----------------------------------------------------------
Sorry for the delay, I haven't had the means of testing it, but now I managed to get the X server use the intel backlight interface instead. Could you rebase this patch to current master? It didn't apply. - Kai Uwe Broulik On Jan. 27, 2015, 11:10 nachm., Nikita Skovoroda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122283/ > ----------------------------------------------------------- > > (Updated Jan. 27, 2015, 11:10 nachm.) > > > Review request for Solid and Kai Uwe Broulik. > > > Repository: powerdevil > > > Description > ------- > > Currently (10 steps preferred): > http://oserv.org/tests/kde/powerdevil/brightness_after.php > > After the patch (20 steps preferred): > http://oserv.org/tests/kde/powerdevil/brightness_after_2.php > > See https://todo.kde.org/?controller=task&action=show&task_id=829 for > reasoning. > See https://git.reviewboard.kde.org/r/119597/ for the steps logic explanation. > > > Diffs > ----- > > daemon/powerdevilscreenbrightnesslogic.cpp 7948c67 > > Diff: https://git.reviewboard.kde.org/r/122283/diff/ > > > Testing > ------- > > Tested the new logic using a script to simulate different maxValue values > (see the links above). > Did not test on the actual notebook, but it should work (current logic > works), only the numbers are changed. > > > Thanks, > > Nikita Skovoroda > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
