public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Joonyoung Shim <jy0922.shim@samsung.com>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: intel-gfx@lists.freedesktop.org, 박경민 <kyungmin.park@samsung.com>,
	dri-devel@lists.freedesktop.org
Subject: Re: DRM planes and new fb creation ioctl
Date: Wed, 26 Oct 2011 10:04:52 +0900	[thread overview]
Message-ID: <4EA75CB4.7090006@samsung.com> (raw)
In-Reply-To: <20111025131326.620c016a@jbarnes-x220>

10/25/2011 08:13 PM, Jesse Barnes 쓴 글:
> On Tue, 25 Oct 2011 19:47:13 +0900
> Joonyoung Shim<jy0922.shim@samsung.com>  wrote:
>> 10/25/2011 06:46 PM, Jesse Barnes 쓴 글:
>>> I've given up waiting for someone to implement support for these
>>> ioctls on another platform before they're merged, but I have
>>> received a lot of feedback on the interfaces, and it sounds like
>>> they're ok.  I've also fixed all the remaining issues I'm aware of
>>> on SNB platforms and things are working well, so I'm just going to
>>> push them out.  (Note IVB support is still missing a few bits for
>>> scaling and such; I'll fix those up when I get back home and can
>>> test on IVB again.)
>>>
>>> One change you may notice from the last set is that I've removed the
>>> 'zpos' parameter.  Plane blending and z ordering is very chipset
>>> specific (it even varies between Intel chipsets), so exposing it
>>> through a device specific ioctl is probably a better plan.
>> But i think zpos is essential parameter of plane. If plane doesn't
>> support it, drm driver cannot know user wants to use which overlay,
>> so i wonder what it meant DRM_IOCTL_MODE_SETPLANE zpos is absent .
> Setplane is just for attaching a new fb.  The order, keying, or
> whatever else your plane blender can support can be set with a device
> specific ioctl before or after the setplane call (probably before to
> avoid any flashing or bad frames).
>

OK, i see.

Thanks.

>> If use device specific ioctl, should implement device specific ioctl
>> for DRM_IOCTL_MODE_SETPLANE?
> You could if you needed, but I don't think it's strictly necessary.
>
>>>     By default, planes
>>> should just overlay the primary plane; a device specific ioctl (none
>>> available yet, but I have some planned for i915) can provide more
>>> flexibility.
>> Could you explain what is the primary plane? Is it same as the overlay
>> handled by crtc? It confuses a bit when one overlay is handled by crtc
>> and plane at the same time.
> Yeah, it is a little confusing.  When I refer to the primary, I'm
> referring to the plane bound to the CRTC.  I'm fine if someone wants to
> break that out, I think it would make sense.  I just didn't want to
> write the compat code that would be required for that scheme. :)  But
> these patches definitely don't preclude it, and I don't think these
> interfaces will need changes if we ever move to a pipe/plane split at
> the userland level.

Could you have the policy about fb of overlay? For example, user
attaches the fb of plane to the overlay already using by CRTC, then
output of overlay will be changed to the fb of plane from fb of CRTC.
And if user detaches the fb of plane, the overlay again should output
the fb of CRTC? or just be disabled?
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2011-10-26  1:04 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-25  9:46 DRM planes and new fb creation ioctl Jesse Barnes
2011-10-25  9:46 ` [PATCH 01/11] drm: add plane support Jesse Barnes
2011-10-25 10:53   ` Joonyoung Shim
2011-10-25 11:18     ` Jesse Barnes
2011-10-26  0:19       ` Joonyoung Shim
2011-10-25 11:58   ` Daniel Vetter
2011-10-25 12:26     ` Jesse Barnes
2011-10-25 13:26     ` Alan Cox
2011-10-25 13:32       ` Jesse Barnes
2011-10-25 13:42       ` Daniel Vetter
2011-10-25 14:09   ` Jesse Barnes
2011-10-25 16:43     ` Rob Clark
2011-10-25 19:41       ` Daniel Vetter
2011-10-25 20:14         ` Rob Clark
2011-10-27 14:05         ` SW Kim
2011-10-31 11:40           ` Inki Dae
2011-10-31 16:52             ` Jesse Barnes
2011-11-02  2:20               ` Inki Dae
2011-11-02 15:57                 ` Jesse Barnes
2011-10-26  5:40   ` Joonyoung Shim
2011-10-27 15:31     ` Jesse Barnes
2011-10-25  9:46 ` [PATCH 02/11] drm: add an fb creation ioctl that takes a pixel format Jesse Barnes
2011-10-25  9:46 ` [PATCH 03/11] drm/i915: rename existing overlay support to "legacy" Jesse Barnes
2011-10-25  9:46 ` [PATCH 04/11] drm/i915: add SNB video sprite support Jesse Barnes
2011-11-02  5:56   ` Inki Dae
2011-11-02 15:58     ` Jesse Barnes
2011-10-25  9:47 ` [PATCH 05/11] drm/i915: move pin & fence for plane past potential error paths Jesse Barnes
2011-10-25  9:47 ` [PATCH 06/11] drm/i915: plane teardown fixes Jesse Barnes
2011-10-25  9:47 ` [PATCH 07/11] drm/i915: enable new overlay code on IVB too Jesse Barnes
2011-10-25  9:47 ` [PATCH 08/11] drm/i915: overlay watermark hack Jesse Barnes
2011-10-25  9:47 ` [PATCH 09/11] drm/i915: fix overlay fb object handling Jesse Barnes
2011-10-25  9:47 ` [PATCH 10/11] drm/i915: clamp sprite to viewable area Jesse Barnes
2011-10-25  9:47 ` [PATCH 11/11] drm/i915: add sprite scaling support Jesse Barnes
2011-10-25 10:47 ` DRM planes and new fb creation ioctl Joonyoung Shim
2011-10-25 11:13   ` Jesse Barnes
2011-10-26  1:04     ` Joonyoung Shim [this message]
2011-10-25 11:20 ` Jesse Barnes
2011-10-25 11:22 ` [PATCH] drm/i915: add SNB video sprite support Jesse Barnes
2011-10-25 11:30   ` Jesse Barnes
2011-11-01 14:11   ` Lan, Hai
2011-11-03 18:44     ` [Intel-gfx] " 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=4EA75CB4.7090006@samsung.com \
    --to=jy0922.shim@samsung.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jbarnes@virtuousgeek.org \
    --cc=kyungmin.park@samsung.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox