From mboxrd@z Thu Jan  1 00:00:00 1970
From: Chris Wilson <chris@chris-wilson.co.uk>
Subject: Re: [PATCH 1/2] drm/i915: clear up backlight inversion
	confusion on gen4
Date: Mon, 23 Apr 2012 10:53:31 +0100
Message-ID: <1335174842_26585@CP5-2952>
References: <1335173535-14811-1-git-send-email-daniel.vetter@ffwll.ch>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from fireflyinternet.com (smtp.fireflyinternet.com [109.228.6.236])
	by gabe.freedesktop.org (Postfix) with ESMTP id 719A69F5BC
	for <intel-gfx@lists.freedesktop.org>;
	Mon, 23 Apr 2012 02:54:11 -0700 (PDT)
In-Reply-To: <1335173535-14811-1-git-send-email-daniel.vetter@ffwll.ch>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=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 <intel-gfx@lists.freedesktop.org>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>, Carsten Emde <C.Emde@osadl.org>
List-Id: intel-gfx@lists.freedesktop.org

On Mon, 23 Apr 2012 11:32:14 +0200, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> There's a bit in the docs for gen4 only that says whether the
> backlight control is inverted. And both the quirk we have and
> all bugs only concern i965gm and gm45 (and mostly Acer) afaics.
> 
> So lets drop the quirk and use the bit instead.
> 
> Also clean up the BLC register definitions a bit by correctly
> grouping the CTL and CTL2 definitions together.
> 
> This quirk was originally added in
> 
> commit 5a15ab5b93e4a3ebcd4fa6c76cf646a45e9cf806
> Author: Carsten Emde <C.Emde@osadl.org>
> Date:   Thu Mar 15 15:56:27 2012 +0100
> 
>     drm/i915: panel: invert brightness acer aspire 5734z
> 
> Cc: Carsten Emde <C.Emde@osadl.org>
> References: https://bugzilla.kernel.org/show_bug.cgi?id=31522
> References: https://bugs.freedesktop.org/show_bug.cgi?id=37986
> References: https://bugs.freedesktop.org/show_bug.cgi?id=40455
> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Oh, light dawns. That explains the polarity bit.

Hmm, on PCH this bit is moved to 29 (expect for early IBX silicon)
according to my specs.

Btw, you introduce too many tabs.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre