linux-kernel - Re: [Regression] Laptop screen dark when lid closed and reopened.with 3.1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 26 Oct 2011 08:13:14 -0700
From:	Keith Packard <keithp@...thp.com>
To:	Alex Davis <alex14641@...oo.com>,
	"linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>,
	"mjg\@redhat.com" <mjg@...hat.com>,
	Kamal Mostafa <kamal@...onical.com>
Subject: Re: [Regression] Laptop screen dark when lid closed and reopened.with 3.1

On Tue, 25 Oct 2011 20:19:16 -0700 (PDT), Alex Davis <alex14641@...oo.com> wrote:

> At the request of Kamal Mostafa, an Ubuntu kernel developer, I am resubmitting this patch which fixes my backlight issue.
> It seem that there are a number of people with different machines that have this problem:

We have a patch that should land in a 3.1 stable release that should
resolve this issue; can you test that?

commit f52c619a590fa75276c07dfcaf380dee53e4ea4c
Author: Takashi Iwai <tiwai@...e.de>
Date:   Fri Oct 14 11:45:40 2011 +0200

    drm/i915/panel: Always record the backlight level again (but cleverly)
    
    The commit 47356eb67285014527a5ab87543ba1fae3d1e10a introduced a
    mechanism to record the backlight level only at disabling time, but it
    also introduced a regression.  Since intel_lvds_enable() may be called
    without disabling (e.g. intel_lvds_commit() calls it unconditionally),
    the backlight gets back to the last recorded value.  For example, this
    happens when you dim the backlight, close the lid and open the lid,
    then the backlight suddenly goes to the brightest.
    
    This patch fixes the bug by recording the backlight level always
    when changed via intel_panel_set_backlight().  And,
    intel_panel_{enable|disable}_backlight() call the internal function not
    to update the recorded level wrongly.
    
    Cc: <stable@...nel.org>
    Signed-off-by: Takashi Iwai <tiwai@...e.de>
    Reviewed-by: Keith Packard <keithp@...thp.com>
    Signed-off-by: Keith Packard <keithp@...thp.com>


-- 
keith.packard@...el.com

Content of type "application/pgp-signature" skipped