[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-31 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||5.27.6

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-31 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
   |ma/powerdevil/-/commit/ddca |ma/powerdevil/-/commit/caf2
   |baf93712b953e3ec327416e0332 |6c7d2842b3e81e370f414e85960
   |d7e44fc40   |9d2fa607b

--- Comment #11 from Nate Graham  ---
Git commit caf26c7d2842b3e81e370f414e859609d2fa607b by Nate Graham, on behalf
of Quang Ngô.
Committed on 31/05/2023 at 17:16.
Pushed by ngraham into branch 'master'.

Fix ddcutil for laptop user

This will prioritize usage of backlight helper over ddcutil.
If backlight helper fail, fall back to ddcutil.
Expected result:
 * Laptop users can now change the brightness of the internal
display. External display will use its own controls.
 * Does not affect desktop users.


(cherry picked from commit ddcabaf93712b953e3ec327416e0332d7e44fc40)

M  +71   -76   daemon/backends/upower/powerdevilupowerbackend.cpp

https://invent.kde.org/plasma/powerdevil/-/commit/caf26c7d2842b3e81e370f414e859609d2fa607b

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-31 Thread Albert Vaca
https://bugs.kde.org/show_bug.cgi?id=469895

Albert Vaca  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/plas
   ||ma/powerdevil/-/commit/ddca
   ||baf93712b953e3ec327416e0332
   ||d7e44fc40
 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #10 from Albert Vaca  ---
Git commit ddcabaf93712b953e3ec327416e0332d7e44fc40 by Albert Vaca Cintora, on
behalf of Quang Ngô.
Committed on 31/05/2023 at 11:24.
Pushed by albertvaka into branch 'Plasma/5.27'.

Fix ddcutil for laptop user

This will prioritize usage of backlight helper over ddcutil.
If backlight helper fail, fall back to ddcutil.
Expected result:
 * Laptop users can now change the brightness of the internal
display. External display will use its own controls.
 * Does not affect desktop users.

M  +71   -76   daemon/backends/upower/powerdevilupowerbackend.cpp

https://invent.kde.org/plasma/powerdevil/-/commit/ddcabaf93712b953e3ec327416e0332d7e44fc40

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-25 Thread postix
https://bugs.kde.org/show_bug.cgi?id=469895

postix  changed:

   What|Removed |Added

 CC||pos...@posteo.eu

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
   Keywords|regression  |
 Status|REPORTED|CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

 CC||hockeymi...@hockeymikey.com

--- Comment #9 from Nate Graham  ---
*** Bug 449263 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Resolution|DUPLICATE   |---
 Status|RESOLVED|REPORTED

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-24 Thread Albert Vaca
https://bugs.kde.org/show_bug.cgi?id=469895

--- Comment #8 from Albert Vaca  ---
Related issue about supporting multiple brightness backends:
https://invent.kde.org/plasma/powerdevil/-/issues/19

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-24 Thread Albert Vaca
https://bugs.kde.org/show_bug.cgi?id=469895

--- Comment #7 from Albert Vaca  ---
I'm the one who enabled compiling with libddc by default. I think it's a great
feature to be able to control your monitor brightness from Plasma instead of
having to use the wonky built-in buttons, and would be nice to polish this
feature before Plasma 6. Maybe it can be one of the "cool new things" for
marketing. 

I also told Antonio, the Arch packager, that it was ready to enable in Plasma 5
even if it was not on by default. 

I didn't know that powerdevil could only use one backend though, and that
enabling libddc would mean the laptop brightness control would be effectively
disabled.

Thanks quagngo for the fix where the laptop brightness backend takes
precedence, I'll give a review now. The best fix however would be for both
backends to be active at the same time, and for Plasma to show two sliders. It
would be awesome if we can get that for Plasma 6.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-23 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #6 from Nate Graham  ---


*** This bug has been marked as a duplicate of bug 449263 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-23 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

 Status|REPORTED|CONFIRMED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-21 Thread quangngo
https://bugs.kde.org/show_bug.cgi?id=469895

--- Comment #5 from quangngo  ---
I create a merge request here
https://invent.kde.org/plasma/powerdevil/-/merge_requests/178. Can someone test
this.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-19 Thread Sergey Kozharinov
https://bugs.kde.org/show_bug.cgi?id=469895

Sergey Kozharinov  changed:

   What|Removed |Added

 CC||m...@sergeykozharinov.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-18 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

--- Comment #4 from Nate Graham  ---
It was enabled by default here in KDE in
https://invent.kde.org/plasma/powerdevil/-/commit/e4a5e5d3e017ef45979933e14e47408bb3a59dda

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-18 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=469895

--- Comment #3 from ratijas  ---
They keep telling "that this is no longer considered experimental and is
enabled by default for Plasma 6". I don't know who said that, and I'd like to
have a word with that person. Also, passing an otherwise disabled by default
configuration flag certainly does not strike me as "enabled by default".

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-18 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

 Resolution|DOWNSTREAM  |---
 Status|RESOLVED|REPORTED

--- Comment #2 from Nate Graham  ---
It's not a downstream issue for a distro to enable our feature and that feature
is broken. Distros can certainly disable it again, but it's on us to fix it.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-18 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=469895

ratijas  changed:

   What|Removed |Added

URL||https://bugs.archlinux.org/
   ||task/78536
 Resolution|--- |DOWNSTREAM
 Status|REPORTED|RESOLVED

--- Comment #1 from ratijas  ---
Packaging problem, already reported as https://bugs.archlinux.org/task/78536

-- 
You are receiving this mail because:
You are watching all bug changes.

[Powerdevil] [Bug 469895] Cannot change laptop display brightness when connected to external monitor with ddcutil support

2023-05-17 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=469895

Nate Graham  changed:

   What|Removed |Added

   Keywords||regression
   Severity|normal  |major
 CC||albertv...@gmail.com,
   ||aleix...@kde.org,
   ||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.