<table><tr><td style="">dvogel updated this revision to Diff 14843.<br />dvogel added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D5381" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Absence of ddcutil on the system is now handled: <br />
the ddcbrightness object is still instantiated in powerdevilupowerbackend, however, this is a dummy object, returning isSupported()=FALSE, making powerdevilupowerbackend avoid using ddcutil.</p>

<p>The same brightness is applied to all ddc-capable displays this seems to be the most obvious choice (compared to setting only one monitor) <br />
Multi-monitor not tested:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">desktop + 2+ monitors: should work, as ddcutilbrightness supports multiple monitors</li>
<li class="remarkup-list-item">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.</li>
</ul></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D5381?vs=13586&id=14843" rel="noreferrer">https://phabricator.kde.org/D5381?vs=13586&id=14843</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5381" rel="noreferrer">https://phabricator.kde.org/D5381</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
cmake/FindDDCUtil.cmake<br />
daemon/backends/CMakeLists.txt<br />
daemon/backends/upower/ddcutilbrightness.cpp<br />
daemon/backends/upower/ddcutilbrightness.h<br />
daemon/backends/upower/powerdevilupowerbackend.cpp<br />
daemon/backends/upower/powerdevilupowerbackend.h</div></div></div><br /><div><strong>To: </strong>dvogel, broulik<br /><strong>Cc: </strong>davidedmundson, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>