All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Ben Skeggs <skeggsb@gmail.com>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 1/2] drm: add plane support v2
Date: Tue, 15 Nov 2011 13:42:40 +0200	[thread overview]
Message-ID: <20111115114240.GK3477@intel.com> (raw)
In-Reply-To: <1321324843.8901.3.camel@nisroch>

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älä
Intel OTC

  reply	other threads:[~2011-11-15 11:40 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-14 20:21 [PATCH 1/2] drm: add plane support v2 Jesse Barnes
2011-11-14 20:21 ` [PATCH 2/2] drm: add an fb creation ioctl that takes a pixel format v4 Jesse Barnes
2011-11-14 21:16   ` Ville Syrjälä
2011-11-14 21:22     ` Jesse Barnes
2011-11-15 12:57       ` Ville Syrjälä
2011-11-15 16:16         ` Jesse Barnes
2011-11-15 20:30           ` Ville Syrjälä
2011-11-15 20:55             ` Jesse Barnes
2011-11-14 21:36     ` Rob Clark
2011-11-14 21:24   ` Ville Syrjälä
2011-11-14 21:35     ` Jesse Barnes
2011-11-14 22:37       ` Ville Syrjälä
2011-11-14 22:53         ` Jesse Barnes
2011-11-15  2:40 ` [PATCH 1/2] drm: add plane support v2 Ben Skeggs
2011-11-15 11:42   ` Ville Syrjälä [this message]
2011-11-15 16:13     ` Jesse Barnes
  -- strict thread matches above, loose matches on Subject: below --
2011-11-14 19:47 Jesse Barnes
2011-11-14 20:13 ` Jesse Barnes

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20111115114240.GK3477@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=skeggsb@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.