From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/2] drm: add plane support v2 Date: Tue, 15 Nov 2011 13:42:40 +0200 Message-ID: <20111115114240.GK3477@intel.com> References: <1321302115-23159-1-git-send-email-jbarnes@virtuousgeek.org> <1321324843.8901.3.camel@nisroch> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 08A209EFED for ; Tue, 15 Nov 2011 03:40:22 -0800 (PST) Content-Disposition: inline In-Reply-To: <1321324843.8901.3.camel@nisroch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Ben Skeggs Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Tue, Nov 15, 2011 at 12:40:43PM +1000, Ben Skeggs wrote: > On Mon, 2011-11-14 at 12:21 -0800, Jesse Barnes wrote: > > Planes are a bit like half-CRTCs. They have a location and fb, but > > don't drive outputs directly. Add support for handling them to the core > > KMS code. > Out of curiosity, lets say you have a *really* stupid hardware overlay > that can't do scaling (or even, has limited scaling capabilities), > should we provide some way to expose this to userspace? I think yes. In fact I'd like drm_plane to replace drm_crtc as far as scanout is concerned. That's how a lot of embedded hardware is laid out already, and I think it's a lot cleaner approach than what we have currently. Stuff like borders then become a simple matter or positioning the "CRTC plane" within the larger active video area, and panel fitters could be exposed through drm_plane scaling. Se either we need to think ahead more with the GETPLANE ioctl structure, or we could add a PLANE_CAPS ioctl later to expose additional details about the hardware. -- = Ville Syrj=E4l=E4 Intel OTC