From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keith Packard Subject: Re: [PATCH] drm/i915: Fix inconsistent backlight level during disabled Date: Thu, 17 Nov 2011 13:00:35 -0800 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0073699853==" Return-path: Received: from keithp.com (home.keithp.com [63.227.221.253]) by gabe.freedesktop.org (Postfix) with ESMTP id D375A9E7B5 for ; Thu, 17 Nov 2011 13:00:39 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Takashi Iwai Cc: Alex Davis , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, mjg@redhat.com List-Id: intel-gfx@lists.freedesktop.org --===============0073699853== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" --=-=-= Content-Transfer-Encoding: quoted-printable On Wed, 16 Nov 2011 10:58:03 +0100, Takashi Iwai wrote: > When the brightness property is inquired while the backlight is disabled, > the driver returns a wrong value (zero) because it probes the value after > the backlight was turned off. This caused a black screen even after the > backlight is enabled again. It should return the internal backlight_level > instead, so that it won't be influenced by the backlight-enable state. Yes, this looks correct by inspection to me. I'll stick it into drm-intel-fixes. =2D-=20 keith.packard@intel.com --=-=-= Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iQIVAwUBTsV18zYtFsjWk68qAQh1NQ//b3ismcOe+N0TO/bgpa+XwFXnGxrMvbGh ps65xMaMoyQwTT84cn+lr065Bp9rGTzVStnpT0wXHEcRCyPmDJ4cHXiOcFTvt8bF JBMgn9ffi9f/3P5MW162J1NfcFGWhYtEL0KFvDrAzr8VtrrTH9suGhqXJF2ciYjA X0liAXlJQCX2cArwh0nH15PkCKQIh1iNwfBGKAbyUPf3pqdko0Qr39I7lwiCmcLe wuChDY8ueioMP7aCrwKAyDvEgbTTlMCKd6/RSRdylzO4A3BlEvHxTMwFmh0i1810 mTE96cQZyQuCPQDtpwaghgRb2ARCYtrnnhs5i5LYj7CZr6Oeo/UUzKz7jQrwmoOT zJdjqt5i9b+E2Q48JtCjfgeg4AKbtyp4wSiD7hrMlAmngAh52Yl8d81y+kwpPpaf fc4hL4f6u6kqwNlAyU0iDWv3I4lXfHyzqC4S87+0e2/sS2nGtB1pmbKy+qym2xt8 fplDVm63zNwbp0AHV1wHE/m5VVEv89HOPcWbfBDIPuPwqicoiMJ4HzkbNWqsci99 HRiEM3PeNifK7BJxV5Jr8IZ643ZdXaAgWyyxWlk3RDUPzki3Au0Dw0TocVrLYjvH /vRh/310BzoxCfewuMuFHMmJ+EeNRbY5ivtpJrjixob/oGcou+yc5zoRHiyyueAd iILGMuQNNDA= =vinB -----END PGP SIGNATURE----- --=-=-=-- --===============0073699853== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============0073699853==--