Bug 381114

Summary: Can't control brightness
Product: [Plasma] Powerdevil Reporter: sunwebrw
Component: generalAssignee: Bhushan Shah <bhush94>
Status: RESOLVED FIXED    
Severity: normal CC: bhush94, kilian.cavalotti.work, simonandric5
Priority: NOR    
Version: 5.10.1   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description sunwebrw 2017-06-12 04:59:58 UTC
Notebook. Trying to change brightness with plasma shell applet or automaticly doesn't work. It works with cli though: echo 7 | sudo tee /sys/class/backlight/acpi_video0/brightness

It did work before, unfortunately i don't remember what was the last working version.
Comment 1 Bhushan Shah 2017-06-12 05:43:32 UTC
Can you please run,

ls -l /sys/class/backlight/ and post output here?

Thanks
Comment 2 sunwebrw 2017-06-12 10:42:23 UTC
ls -l /sys/class/backlight/
total 0
lrwxrwxrwx 1 root root 0 чер 12 10:49 acpi_video0 -> ../../devices/pci0000:00/0000:00:02.0/backlight/acpi_video0
lrwxrwxrwx 1 root root 0 чер 12 10:49 intel_backlight -> ../../devices/pci0000:00/0000:00:02.0/drm/card0/card0-LVDS-1/intel_backlight
Comment 3 Bhushan Shah 2017-06-12 10:46:01 UTC
Can you please run,

cat /sys/class/backlight/intel_backlight/type

and

cat /sys/class/backlight/acpi_video0/type
Comment 4 sunwebrw 2017-06-12 11:21:04 UTC
cat /sys/class/backlight/intel_backlight/type
raw

cat /sys/class/backlight/acpi_video0/type
firmware
Comment 5 kilian.cavalotti.work 2017-06-12 16:14:10 UTC
Just want to confirm seeing the same issue here. All backlight controls have disappeared from the "Power Management" section of System Settings.


It has also been reported here: https://forum.manjaro.org/t/kde-can-not-control-screen-backlight-anymore/25654
Comment 6 Bhushan Shah 2017-06-14 13:03:35 UTC
Git commit 97f3900049aa657be5f4c124c9f992120f124944 by Bhushan Shah.
Committed on 14/06/2017 at 12:55.
Pushed by bshah into branch 'master'.

Revert "skip the disabled backlight device"

This reverts commits 86c3548d103ff5966366788220796de629be9230 and
5c0d35ca6caf7bb0a0e9897a1a868e8406f3b112.

Not all raw interfaces have device/enabled under the backlight sysfs
interface. Some examples are nvidia-bl and acpi_video0, they are raw
interfaces but doesn't have device/enabled properties.

Overall we can just return true even if file doesn't exist but then this
code serves no purpose.
Related: bug 381199

Ackd-by: Kai Uwe Broulik <kde@privat.broulik.de>
CCMAIL: AceLan Kao <acelan@acelan.idv.tw>

M  +1    -19   daemon/backends/upower/backlighthelper.cpp
M  +0    -5    daemon/backends/upower/backlighthelper.h

https://commits.kde.org/powerdevil/97f3900049aa657be5f4c124c9f992120f124944
Comment 7 Bhushan Shah 2017-06-14 13:05:02 UTC
Git commit 5c57cf64b5e5c880b1a5f3a0177293f6958e1b9a by Bhushan Shah.
Committed on 14/06/2017 at 13:04.
Pushed by bshah into branch 'Plasma/5.10'.

Revert "skip the disabled backlight device"

This reverts commits 86c3548d103ff5966366788220796de629be9230 and
5c0d35ca6caf7bb0a0e9897a1a868e8406f3b112.

Not all raw interfaces have device/enabled under the backlight sysfs
interface. Some examples are nvidia-bl and acpi_video0, they are raw
interfaces but doesn't have device/enabled properties.

Overall we can just return true even if file doesn't exist but then this
code serves no purpose.
Related: bug 381199

Ackd-by: Kai Uwe Broulik <kde@privat.broulik.de>
CCMAIL: AceLan Kao <acelan@acelan.idv.tw>

M  +1    -19   daemon/backends/upower/backlighthelper.cpp
M  +0    -5    daemon/backends/upower/backlighthelper.h

https://commits.kde.org/powerdevil/5c57cf64b5e5c880b1a5f3a0177293f6958e1b9a
Comment 8 sunwebrw 2017-06-15 09:00:25 UTC
Thanks.