From: Archit Taneja <architt@codeaurora.org>
To: robdclark@gmail.com
Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
dhinakaran.pandiyan@intel.com
Subject: [RFC 0/3] drm/msm: Avoid subclassing of drm_atomic_state
Date: Thu, 21 Dec 2017 11:44:22 +0530 [thread overview]
Message-ID: <20171221061425.5511-1-architt@codeaurora.org> (raw)
It's been recommended that we use drm_private_objs embedded in
drm_atomic_state to hold shared resources instead of subclassing
drm_atomic_state.
This will also help us in getting one step closer to using the
atomic commit helpers instead of the msm_atomic_commit() funcs
in msm_atomic.c
I've taken the drm_private_obj usage in drm_dp_mst_topology as
reference. I've put this as RFC because I want others to see if
the private_obj stuff is used correctly.
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
drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 104 +++++++++++++++++++++---------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.h | 29 +++++----
drivers/gpu/drm/msm/mdp/mdp5/mdp5_mixer.c | 12 ++--
drivers/gpu/drm/msm/mdp/mdp5/mdp5_pipe.c | 20 +++---
drivers/gpu/drm/msm/mdp/mdp5/mdp5_smp.c | 17 +++--
drivers/gpu/drm/msm/msm_atomic.c | 31 ---------
drivers/gpu/drm/msm/msm_drv.c | 3 -
drivers/gpu/drm/msm/msm_kms.h | 14 ----
8 files changed, 121 insertions(+), 109 deletions(-)
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next reply other threads:[~2017-12-21 6:14 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-21 6:14 Archit Taneja [this message]
2017-12-21 6:14 ` [RFC 1/3] drm/msm/mdp5: Add global state as a private atomic object Archit Taneja
2017-12-21 9:56 ` Daniel Vetter
2018-01-03 11:32 ` Archit Taneja
2018-01-03 21:34 ` Rob Clark
2018-02-20 20:05 ` Rob Clark
2017-12-21 6:14 ` [RFC 2/3] drm/msm/mdp5: Use the new private_obj state Archit Taneja
2017-12-21 6:14 ` [RFC 3/3] drm/msm: Don't subclass drm_atomic_state anymore Archit Taneja
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=20171221061425.5511-1-architt@codeaurora.org \
--to=architt@codeaurora.org \
--cc=dhinakaran.pandiyan@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-arm-msm@vger.kernel.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 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.