From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jesse Barnes <jbarnes@virtuousgeek.org>
Subject: Re: [Intel-gfx] [PATCH 1/4] drm/i915: Wait for LVDS panel power
 sequence
Date: Mon, 8 Aug 2011 11:50:39 -0700
Message-ID: <20110808115039.75384477@jbarnes-desktop>
References: <1312653248-3487-1-git-send-email-keithp@keithp.com>
	<1312653248-3487-2-git-send-email-keithp@keithp.com>
	<20110808092719.4f62983d@jbarnes-desktop>
	<yunty9ru4qh.fsf@aiko.keithp.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Return-path: <linux-kernel-owner@vger.kernel.org>
In-Reply-To: <yunty9ru4qh.fsf@aiko.keithp.com>
Sender: linux-kernel-owner@vger.kernel.org
To: Keith Packard <keithp@keithp.com>
Cc: Dave Airlie <airlied@redhat.com>, intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Mon, 08 Aug 2011 11:40:06 -0700
Keith Packard <keithp@keithp.com> wrote:

> On Mon, 8 Aug 2011 09:27:19 -0700, Jesse Barnes <jbarnes@virtuousgeek.org> wrote:
> 
> > ...to catch places like this where the wrong register gets used. :)
> 
> Ouch! There are only two places we *should* have these loops, one when
> turning it off, another when turning it on. There's a couple of loops
> which just need to be removed.
> 

Yeah looks better.

Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>

-- 
Jesse Barnes, Intel Open Source Technology Center