|Summary:||[Regression] Randr 1.2 version of radeon driver disables backlight support|
|Product:||xorg||Reporter:||Erik Andren <erik.andren>|
|Component:||Driver/Radeon||Assignee:||xf86-video-ati maintainers <xorg-driver-ati>|
|Status:||VERIFIED FIXED||QA Contact:||Xorg Project Team <xorg-team>|
|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
Comment 6 Erik Andren 2007-09-15 08:56:16 UTC
Verified, thanks a bunch!