From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: fix up PCH backlight #define mixup Date: Wed, 11 Jul 2012 10:23:44 +0200 Message-ID: <20120711082344.GA5204@phenom.ffwll.local> References: <1341959466-1564-1-git-send-email-daniel.vetter@ffwll.ch> <20120710223659.GD5108@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-we0-f177.google.com (mail-we0-f177.google.com [74.125.82.177]) by gabe.freedesktop.org (Postfix) with ESMTP id 3572E9E754 for ; Wed, 11 Jul 2012 01:23:50 -0700 (PDT) Received: by weyr3 with SMTP id r3so656167wey.36 for ; Wed, 11 Jul 2012 01:23:49 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20120710223659.GD5108@phenom.ffwll.local> 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: Intel Graphics Development Cc: Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org On Wed, Jul 11, 2012 at 12:36:59AM +0200, Daniel Vetter wrote: > On Wed, Jul 11, 2012 at 12:31:06AM +0200, Daniel Vetter wrote: > > I so totally suck. > > > > This can cause a black screen if (for whatever reason) the bios > > hasn't set this bit itself. > > > > This regression has been introduced in > > > > commit 7cf4160148136deb31ee5f2802857dd935a38529 > > Author: Daniel Vetter > > Date: Tue Jun 5 10:07:09 2012 +0200 > > > > drm/i915: clear up backlight #define confusion on gen4+ > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=51463 > > Signed-Off-by: Daniel Vetter > > From irc: > > Tested-by: Kenneth Graunke I've queued this one for next, but with the bugzilla line removed - that bug isn't fixed by this patch unfortunately. -Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48