https://bugs.kde.org/show_bug.cgi?id=399646

--- Comment #16 from Jakob Petsovits <jpe...@petsovits.com> ---
(In reply to Nils from comment #15)
> acpi_video0/type -> firmware
> amdgpu_bl2/type -> raw

Hm, that could be tricky to fix without potentially breaking other systems.
Reading material such as
https://www.kernel.org/doc/Documentation/ABI/stable/sysfs-class-backlight or
the more in-depth write-up at https://mjg59.livejournal.com/127103.html suggest
that it's not necessarily safe to change both at once.

Maybe we can learn something from
https://github.com/systemd/systemd/blob/main/src/backlight/backlight.c which
enumerates backlight devices to save/restore brightness at startup/shutdown
time. Based on a quick initial reading, it seems to allow both
firmware/platform devices and raw devices next to each other, but filters out
any raw backlight device that points to the same PCI device parent as another
firmware/platform backlight device. I think doing the same in powerdevil could
fix this remaining use case... while also making the code more fragile :-S

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

Reply via email to