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

Bug#600405: xserver-xorg-video-intel: changing to a non-native mode dims backlight



On Wed, Nov 17, 2010 at 18:25:32 +0000, Simon McVittie wrote:

> tags 600405 + fixed-upstream
> thanks
> 
> On Sun, 17 Oct 2010 at 13:51:56 +0100, Simon McVittie wrote:
> > It turns out this can be worked around by reverting part of the "+shadow"
> > series. On the upstream bug, Chris Wilson thinks the correct fix is to improve
> > backlight setting in the kernel, though.
> 
> There is now a different upstream fix for this in master
> (commit 33c08882c0d from Keith Packard) which also works for me. I don't know
> whether you'd prefer to use that one, or the reversion I suggested.
> 
> http://git.debian.org/?p=users/smcv/xserver-xorg-video-intel.git
> (the upstream fix applied as a patch instead) certainly works with my current
> kernel, which is 2.6.32-27 for amd64.
> 
I'd committed the revert on monday:
http://git.debian.org/?p=pkg-xorg/driver/xserver-xorg-video-intel.git;a=commitdiff;h=2e37d6f4183f695386ded784720870a9598d3825
I don't know of a bug number the initial patch was fixing, so I'd go
with the revert instead of the cherry-pick.

We were waiting for 2.13.0 to move to testing (which has now happened), I
guess we'll upload soonish.  The next upload will also probably disable
shadow on 865G (which is apparently less broken than 845 and 855), and
have some version of
http://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?id=455f2939a661764ebb8d1747d44e16a0a8937808
(I had a simpler patch to do this which just called
pci_device_has_kernel_driver() from intel_pci_probe(); since we're
autoloading i915 from udev we don't need to modprobe it from X.)

Let me know if you'd like to work on getting that ready so I can cross
it out of my todo list ;)

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature


Reply to: