From: Sean Paul <seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
To: freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Cc: architt-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
abhinavk-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
Sean Paul <seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
hoegsberg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org
Subject: [PATCH v3 0/8] drm/msm: Switch to atomic helpers
Date: Mon, 2 Apr 2018 11:11:24 -0400 [thread overview]
Message-ID: <20180402151136.192869-1-seanpaul@chromium.org> (raw)
I spoke with Rob on IRC about the set and he preferred the private
object be isolated in mdp5. So here's the atomic helper set rebased on
Archit's series.
One notable difference is that I've dropped the first patch from that
series which added a new private_obj lock to the core. Instead, it's
located in mdp5_kms along with the global state (same as the state_lock
which is currently there).
Thank you for your feedback thus far!
Sean
Archit Taneja (3):
drm/msm/mdp5: Add global state as a private atomic object
drm/msm/mdp5: Use the new private_obj state
drm/msm: Don't subclass drm_atomic_state anymore
Sean Paul (5):
drm/msm: Refactor complete_commit() to look more the helpers
drm/msm: Mark the crtc->state->event consumed
drm/msm: Issue queued events when disabling crtc
drm/msm: Remove msm_commit/worker, use atomic helper commit
drm/msm: Switch to atomic_helper_commit()
drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 1 +
drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 10 +
drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 105 ++++++---
drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.h | 27 +--
drivers/gpu/drm/msm/disp/mdp5/mdp5_mixer.c | 12 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_pipe.c | 20 +-
drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c | 17 +-
drivers/gpu/drm/msm/msm_atomic.c | 236 +--------------------
drivers/gpu/drm/msm/msm_drv.c | 11 +-
drivers/gpu/drm/msm/msm_drv.h | 7 +-
drivers/gpu/drm/msm/msm_kms.h | 14 --
11 files changed, 141 insertions(+), 319 deletions(-)
--
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno
next reply other threads:[~2018-04-02 15:11 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-02 15:11 Sean Paul [this message]
2018-04-02 15:11 ` [PATCH v3 1/8] drm/msm/mdp5: Add global state as a private atomic object Sean Paul
2018-04-02 15:11 ` [PATCH v3 3/8] drm/msm: Don't subclass drm_atomic_state anymore Sean Paul
2018-04-02 15:11 ` [PATCH v3 4/8] drm/msm: Refactor complete_commit() to look more the helpers Sean Paul
2018-04-02 15:11 ` [PATCH v3 7/8] drm/msm: Remove msm_commit/worker, use atomic helper commit Sean Paul
[not found] ` <20180402151136.192869-1-seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-04-02 15:11 ` [PATCH v3 2/8] drm/msm/mdp5: Use the new private_obj state Sean Paul
2018-04-02 15:11 ` [PATCH v3 5/8] drm/msm: Mark the crtc->state->event consumed Sean Paul
2018-04-02 15:11 ` [PATCH v3 6/8] drm/msm: Issue queued events when disabling crtc Sean Paul
2018-04-02 15:11 ` [PATCH v3 8/8] drm/msm: Switch to atomic_helper_commit() Sean Paul
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=20180402151136.192869-1-seanpaul@chromium.org \
--to=seanpaul-f7+t8e8rja9g9huczpvpmw@public.gmane.org \
--cc=abhinavk-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=architt-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=hoegsberg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
/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.