From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jesse Barnes <jbarnes@virtuousgeek.org>
Subject: Re: [PATCH 2/3] drm/i915: add destination color key
 support
Date: Wed, 16 Nov 2011 08:48:40 -0800
Message-ID: <20111116084840.6b1f8ec7@jbarnes-desktop>
References: <1321302133-23197-1-git-send-email-jbarnes@virtuousgeek.org>
	<1321302133-23197-2-git-send-email-jbarnes@virtuousgeek.org>
	<CAKMK7uF9F=uG3uAoNXuKCU--_j2irpyc8xpCQkFnL0SsdiLhDg@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from oproxy8-pub.bluehost.com (oproxy8-pub.bluehost.com
	[69.89.22.20])
	by gabe.freedesktop.org (Postfix) with SMTP id 5F13D9E7CA
	for <intel-gfx@lists.freedesktop.org>;
	Wed, 16 Nov 2011 08:47:52 -0800 (PST)
In-Reply-To: <CAKMK7uF9F=uG3uAoNXuKCU--_j2irpyc8xpCQkFnL0SsdiLhDg@mail.gmail.com>
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: Daniel Vetter <daniel@ffwll.ch>
Cc: intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Wed, 16 Nov 2011 17:10:53 +0100
Daniel Vetter <daniel@ffwll.ch> wrote:

> On Mon, Nov 14, 2011 at 21:22, Jesse Barnes <jbarnes@virtuousgeek.org> wr=
ote:
> > Add new ioctls for getting and setting the current destination color
> > key. =A0This allows for simple overlay display control by matching a co=
lor
> > key value in the primary plane before blending the overlay on top.
> >
> > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
> =

> Just a few comments because reviewing patches here is a royal pain:
> - Is taking dev->struct_mutex really required around the register access?

Oh did I leave them in place for these paths?  I'll drop them too.

> - You unconditionally enable the color key. Do we anticipate any other
> uses (like no color key or dst color key) or is this just the
> make-Xv-work ioctl?

I was thinking of that...  W/o a src or dest key, the overlay will
always be on top.  That's probably reasonable for simple cases, so
having a "key present" flag makes sense.

Which means setting a 0 dest key would mean "disable dest key" and go
back to the default behavior.  So the interface would stay the same
even if we added other key types in the future.

-- =

Jesse Barnes, Intel Open Source Technology Center