Bug 427518 - ACPI video driver should validate brightness level before setting it via _BCM
Summary: ACPI video driver should validate brightness level before setting it via _BCM
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: 8
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Kernel Maintainer List
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FCMETA_ACPI
TreeView+ depends on / blocked
 
Reported: 2008-01-04 11:45 UTC by Julian Sikorski
Modified: 2008-02-11 22:39 UTC (History)
4 users (show)

Fixed In Version: 2.6.23.15-137.fc8
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-02-11 22:39:25 UTC
Type: ---


Attachments (Terms of Use)
the patch in question, in case someone wants to test (2.85 KB, text/plain)
2008-01-09 19:36 UTC, Chuck Ebbert
no flags Details


Links
System ID Priority Status Summary Last Updated
Linux Kernel 9277 None None None Never
Linux Kernel 9614 None None None Never

Description Julian Sikorski 2008-01-04 11:45:03 UTC
Description of problem:
In the current situation, sliding the brightness bar in gnome-power-manager
causes a lot of screen flickering, since it goes to 100 % for invalid values.
Upstream bug report contains a working fix, which applies cleanly against
kernel-2.6.24-0.133.rc6.git8.fc9. It would be nice if the fix could be
backported to F8 kernel as well.

Comment 1 Jon Stanley 2008-01-04 15:55:21 UTC
Adding ACPI metadata.

Comment 2 Julian Sikorski 2008-01-09 12:39:12 UTC
Patch has now been merged to acpi test branch.

Comment 3 Chuck Ebbert 2008-01-09 19:36:33 UTC
Created attachment 291191 [details]
the patch in question, in case someone wants to test

Comment 4 Julian Sikorski 2008-01-12 09:35:15 UTC
So, are there any chances this patch hits Fedora kernel before it gets merged
upstream?

Comment 5 Julian Sikorski 2008-01-12 09:42:47 UTC
CCing upstream submitter.

Comment 6 Danny Baumann 2008-01-12 12:26:47 UTC
The patch in comment #3 works fine for me, and I'd also appreciate it being
merged to the Fedora kernels. g-p-m backlight management is just unusable as it
is now for ACPI video driver users.

Comment 7 Julian Sikorski 2008-02-02 16:03:11 UTC
The patch has been updated, new one is even better. It would be very cool if
this patch could be applied to Fedora kernel along with the one mentioned in
kernel bug #9614. These two greatly improve the backlight experience.

Comment 8 Chuck Ebbert 2008-02-06 00:50:08 UTC
Fixed in 2.6.23.14-132

Comment 9 Julian Sikorski 2008-02-06 09:27:46 UTC
The backlight issues are fixed, thank you. Not sure if this bug should be
closed, or stay open until the fixed kernel hits the repositories.

Comment 10 Fedora Update System 2008-02-11 03:34:14 UTC
kernel-2.6.23.15-137.fc8 has been submitted as an update for Fedora 8

Comment 11 Fedora Update System 2008-02-11 22:39:00 UTC
kernel-2.6.23.15-137.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.