Bug 12424

Summary: [Regression] Randr 1.2 version of radeon driver disables backlight support
Product: xorg Reporter: Erik Andren <erik.andren>
Component: Driver/RadeonAssignee: xf86-video-ati maintainers <xorg-driver-ati>
Status: VERIFIED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium CC: alexdeucher
Version: unspecifiedKeywords: regression
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Erik Andren 2007-09-14 04:06:46 UTC
Commit 8bff57a938e92dfc8729e40f7aa9ad9a96262131 introduces a regression on this Dell C640 Radeon mobility 7500 laptop stopping the backlight from functioning.
Neither the hotkeys on the keyboard or the libsmbios utilities work on further commits.
Comment 1 Alex Deucher 2007-09-14 14:42:40 UTC
I think something else is at play here as we do not change the bios scratch registers at all in the current driver.  Perhaps an issue with radeonfb (if you are using it) or acpi in your kernel?
Comment 2 Erik Andren 2007-09-14 23:49:01 UTC
I'm not using radeonfb and I identified this commit by doing a git bisect. Nothing else changed in the system.
Comment 3 Erik Andren 2007-09-15 07:55:38 UTC
I examined the offending commit a little bit more and found out that bringing back this line made the backlight come on again:

save->bios_5_scratch = orig->bios_5_scratch;
Comment 4 Erik Andren 2007-09-15 08:17:57 UTC
(In reply to comment #3)
> I examined the offending commit a little bit more and found out that bringing
> back this line made the backlight come on again:
> 
> save->bios_5_scratch = orig->bios_5_scratch;
> 

Sorry, I didn't realize that there were two instances of this expression.
I meant the one at line ~6501 when at commit 8bff57a938e92dfc8729e40f7aa9ad9a96262131
Comment 5 Alex Deucher 2007-09-15 08:39:45 UTC
fixed: 38515d402555eaa61c686d42973e59f659b07466
Comment 6 Erik Andren 2007-09-15 08:56:16 UTC
Verified, thanks a bunch!

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.