Re: [kernel/f21] Make sure acpi brightness_switch is disabled (like forever in Fedora)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Josh,

On 07/28/2014 07:04 PM, Josh Boyer wrote:
> commit e6fe382d1d53d4cdf9b544729dc823d4eab0217c
> Author: Josh Boyer <jwboyer@xxxxxxxxxxxxxxxxx>
> Date:   Mon Jul 28 13:03:01 2014 -0400
> 
>     Make sure acpi brightness_switch is disabled (like forever in Fedora)
>     
>     Upstream reverted the change to turn the ACPI brightness_switch_enabled
>     parameter off by default.  Revert the revert so we go back to the state
>     Fedora has traditionally been in.

Ack, I was planning on doing this myself but you beat me to it, thanks for
taking care of this.

Note that 3.17 will have this patch:

https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit/?h=linux-next&id=8ab58e8e7e097bae5fe39cbc67eb93a91f7134b7

Which fixes the 2 steps being taken for one keypress problem while
keeping the acpi brightness_switch behavior enabled, so that people who
have an acpi-video controlled backlight and a userspace which does not
do backlight control (e.g. windowmaker).

So for 3.17 we should IMHO drop the revert-revert and stick with
upstream behavior.

Alternatively we could apply that patch now instead of the revert-revert.

Regards,

Hans
_______________________________________________
kernel mailing list
kernel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/kernel





[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux