Bug 12424 - [Regression] Randr 1.2 version of radeon driver disables backlight support
Summary: [Regression] Randr 1.2 version of radeon driver disables backlight support
Status: VERIFIED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/Radeon (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: xf86-video-ati maintainers
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2007-09-14 04:06 UTC by Erik Andren
Modified: 2007-09-15 08:56 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

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.