From: Daniel Vetter <daniel@ffwll.ch>
To: Rob Clark <robdclark@gmail.com>
Cc: "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 00/16] Atomic/nuclear modeset/pageflip
Date: Wed, 19 Mar 2014 14:39:13 +0100 [thread overview]
Message-ID: <20140319133913.GA30571@phenom.ffwll.local> (raw)
In-Reply-To: <CAF6AEGvwXWWnVdoGr+snVm34Wy-JFx4cxBoQZZmK=YK98WYLig@mail.gmail.com>
On Wed, Mar 19, 2014 at 08:23:31AM -0400, Rob Clark wrote:
> On Wed, Mar 19, 2014 at 7:07 AM, Daniel Vetter <daniel@ffwll.ch> wrote:
> > On Tue, Mar 18, 2014 at 05:22:37PM -0400, Rob Clark wrote:
> > On the i915 side I think all the various pieces we need are finally coming
> > together, and I hope a lot of that goes in for 3.16. So I think we can
> > soonish jump on board with providing real proof-of-concepts based on your
> > ioctl.
> >
> > For merging the core stuff I think we should concentrate first on getting
> > Matt's universal planes stuff in asap. I've scanned through his series
> > with a cursory look and atm it has a few change-the-world flag days. I'll
> > try to do an in-depth review later on looking for ways to make the
> > transition smoother. cursor planes seem to be the main culprit in this ...
>
> it would be nice if there was a way to get things merged sooner than
> 3.18 or so.. but we do need at least the *initial* primary plane work
> first.
>
> Some of the stuff like cursor planes should really happen
> independently from atomic. And maybe at the same time we want to
> expose cursor properties, or at least somehow property-ify cursors.
> The part that really needs to be in before atomic is the
> crtc->primary->fb bit.
Hm, just wanted to dump a comment on Matt's series that imo cursor planes
should happen first before we enable universal planes. And the resulting
unification is really kinda neat for atomic imo.
Still pondering the details but I don't think there's too much fuzz
involved in exposing cursor planes. We can't do it without any driver
changes at all since there's no generic way to go from cursor fb to buffer
manager object id. But beyond that there's shouldn't be anything needed.
I'm hashing out a slightly more detailed plan atm.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
next prev parent reply other threads:[~2014-03-19 13:39 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-18 21:22 [PATCH 00/16] Atomic/nuclear modeset/pageflip Rob Clark
2014-03-18 21:22 ` [PATCH 01/16] drm: fix typo Rob Clark
2014-03-18 21:22 ` [PATCH 02/16] drm: add atomic fxns Rob Clark
2014-03-18 21:22 ` [PATCH 03/16] drm: convert crtc to ww_mutex Rob Clark
2014-03-18 21:22 ` [PATCH 04/16] drm: add object property type Rob Clark
2014-03-18 21:22 ` [PATCH 05/16] drm: add signed-range " Rob Clark
2014-03-18 21:22 ` [PATCH 06/16] drm: helpers to find mode objects Rob Clark
2014-03-18 21:22 ` [PATCH 07/16] drm: split propvals out and blob property support Rob Clark
2014-03-18 21:22 ` [PATCH 08/16] drm: Allow drm_mode_object_find() to look up an object of any type Rob Clark
2014-03-18 21:22 ` [PATCH 09/16] drm: Refactor object property check code Rob Clark
2014-03-18 21:22 ` [PATCH 10/16] drm: allow FB's in drm_mode_object_find Rob Clark
2014-03-18 21:22 ` [PATCH 11/16] drm: convert plane to properties/state Rob Clark
2014-03-18 21:22 ` [PATCH 12/16] drm: convert crtc " Rob Clark
2014-03-18 21:22 ` [PATCH 13/16] drm: push locking down into restore_fbdev_mode Rob Clark
2014-03-18 21:22 ` [PATCH 14/16] drm: Atomic modeset ioctl Rob Clark
2014-03-18 21:22 ` [PATCH 15/16] drm/msm: add atomic support Rob Clark
2014-03-18 21:22 ` [PATCH 16/16] drm: spiff out FB refcnting traces Rob Clark
2014-03-19 11:07 ` [PATCH 00/16] Atomic/nuclear modeset/pageflip Daniel Vetter
2014-03-19 12:23 ` Rob Clark
2014-03-19 13:39 ` Daniel Vetter [this message]
2014-03-20 22:34 ` Greg Hackmann
2014-03-21 0:28 ` Rob Clark
2014-03-26 21:21 ` Greg Hackmann
2014-03-26 10:08 ` Daniel Vetter
2014-03-26 21:33 ` Greg Hackmann
2014-03-26 22:33 ` Daniel Vetter
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=20140319133913.GA30571@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=robdclark@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox