[PATCH 3/3] drm/i915: Revert backlight cargo-culting

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

 



On Fri, Mar 22, 2013 at 11:13 AM, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> On Fri, Mar 22, 2013 at 11:00 AM, Takashi Iwai <tiwai at suse.de> wrote:
>> At Fri, 22 Mar 2013 10:53:41 +0100,
>> Daniel Vetter wrote:
>>>
>>> This reverts the following commits:
>>>
>>> cf0a6584aa6d382f802 drm/i915: write backlight harder
>>> 770c12312ad617172b1 drm/i915: Fix blank panel at reopening lid
>>>
>>> We've come full-circle in this mess and now broke the originally fixed
>>> machines again with the new trick. So remove it all and start over.
>>>
>>> References: http://www.mail-archive.com/intel-gfx at lists.freedesktop.org/msg18788.html
>>> Cc: Jani Nikula <jani.nikula at intel.com>
>>> Cc: Takashi Iwai <tiwai at suse.de>
>>> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
>>
>> Well, although I fully agree that it's better to clear all voodoo once
>> and fix from scratch, my concern is that it'll cause mess for stable
>> kernel.  This path won't fix anything by itself, so it's not suitable
>> for stable alone.  If any, it should be together with the real fix
>> patch.
>>
>> I guess you'll submit the proper fix patch(es) to stable later on?
>
> Oops, I've missed that 770c12312ad617172 itself is a regression fix,
> but lacks the relevant commit citations. I'll resend with more
> aggressive reverting.

Actually the only other patch I've found is 6db65cbb941f, but no one
reported a regression against that one yet. So I think we can leave
that in for now. So I still think that this is the patch I want for
3.9&stable, since oldest regression wins.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux