On Sun, Jun 3, 2012 at 1:08 AM, Daniel Vetter <daniel at ffwll.ch> wrote: > On Fri, Apr 27, 2012 at 10:18:08PM +0200, Carsten Emde wrote: >> Daniel, >> >> >>>Depending upon how things are set up, this might help. >> >>Yeah, thanks a lot, works great now! >> >>I removed patch 3/4 and 4/4 and pushed this one; so currently applied are: >> >>[PATCH 1/4] drm/i915: clear up backlight inversion confusion on gen4 >> >>[PATCH 2/4] drm/i915: completely revert the invert brightness quirk >> >>[PATCH] properly enable the blc controller on the right pipe >> >Hey, that was just a quick hack to check things ;-) >> Sorry, too many things in parallel. >> >> Spent some time reading the docs you mentioned - will continue to >> work on the backlight brightness problem and prepare another patch. >> I just need some more time. > > Ok, I've stitched together some real patches and actually tested them. > While doing so I've noticed that the patch you've tested essentially > disables the i915 backlight control and leaves everything to the legacy or > platform backlight control. > > New patches are at > > http://cgit.freedesktop.org/~danvet/drm/log/?h=backlight-confusion > > Note that these do not include the revert for the backlight inversion > quirk. You either have to revert that yourself for testing or disable the > backlight inversion on the kernel bootline. > > Testing feedback highly welcome. I've updated my backlight-confusion branch with new patches for gen4 (it doesn't seem to work for these machines yet). Can you please test the updated branch? Thanks, Daniel -- Daniel Vetter daniel.vetter at ffwll.ch - +41 (0) 79 365 57 48 - http://blog.ffwll.ch