From: Fabien DESSENNE <fabien.dessenne@st.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Vincent ABRIOU <vincent.abriou@st.com>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 0/5] drm/sti: do not sync legacy IOCTL on vblank if not ATOMIC
Date: Thu, 5 Jan 2017 15:23:24 +0000 [thread overview]
Message-ID: <e1475226-5c2b-bb28-e2aa-94d43d16c49b@st.com> (raw)
In-Reply-To: <20170104091813.qu6agrcx6kncumia@phenom.ffwll.local>
Hi Daniel
I come to the conclusion that (only) Atomic Weston will solve all of my
troubles, so let's forget these patches (and work for atomic weston).
By the way, is the expected behavior (Vblank - sync'ed or not) of
drmModeSetPlane() described anywhere? The last time I browsed the
related documentation I could not find the answer. Maybe something that
needs to be clarified ?
I will also re-send patch 1 & 5 out of this abandoned series since they
make sense independently of this (a)sync stuff.
BR.
Fabien
On 04/01/17 10:18, Daniel Vetter wrote:
> On Tue, Jan 03, 2017 at 05:56:47PM +0100, Fabien Dessenne wrote:
>> These patches allow a legacy framework (eg non-atomic Weston) to call
>> several SETPLANE within the same Vsync cycle.
>> - [PATCH 1/5] drm/sti: use atomic_helper for commit
>> - [PATCH 2/5] drm/sti: add drm_file to sti_private
>> - [PATCH 3/5] drm/sti: do not sync SETPLANE on vblank if not ATOMIC
>> - [PATCH 4/5] drm/sti: do not sync SETPROPERTY on vblank if not ATOMIC
>> - [PATCH 5/5] drm/sti: do not check hw scaling if mode is not set
> Upstream weston never really enabled plane support, why exactly do you
> need this? Also, if this really is required, I think we should implement
> something like this (aka async plane flips) in general for everyone. sti
> is by far not the only driver playing around with these games.
> -Daniel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-01-05 15:23 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-03 16:56 [PATCH 0/5] drm/sti: do not sync legacy IOCTL on vblank if not ATOMIC Fabien Dessenne
2017-01-03 16:56 ` [PATCH 1/5] drm/sti: use atomic_helper for commit Fabien Dessenne
2017-01-03 16:56 ` [PATCH 2/5] drm/sti: add drm_file to sti_private Fabien Dessenne
2017-01-03 16:56 ` [PATCH 3/5] drm/sti: do not sync SETPLANE on vblank if not ATOMIC Fabien Dessenne
2017-01-03 16:56 ` [PATCH 4/5] drm/sti: do not sync SETPROPERTY " Fabien Dessenne
2017-01-03 16:56 ` [PATCH 5/5] drm/sti: do not check hw scaling if mode is not set Fabien Dessenne
2017-01-04 9:18 ` [PATCH 0/5] drm/sti: do not sync legacy IOCTL on vblank if not ATOMIC Daniel Vetter
2017-01-05 15:23 ` Fabien DESSENNE [this message]
2017-01-09 10:03 ` 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=e1475226-5c2b-bb28-e2aa-94d43d16c49b@st.com \
--to=fabien.dessenne@st.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=vincent.abriou@st.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