From: Sean Paul <seanpaul@chromium.org>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 0/4] drm/atomic: Cleanups for adding connector atomic check function.
Date: Thu, 6 Apr 2017 08:48:36 -0400 [thread overview]
Message-ID: <20170406124836.4phve7bwjsxaflny@art_vandelay> (raw)
In-Reply-To: <1491477543-31257-1-git-send-email-maarten.lankhorst@linux.intel.com>
On Thu, Apr 06, 2017 at 01:18:59PM +0200, Maarten Lankhorst wrote:
> Some small cleanups I came across to make drm_atomic_helper_check_modeset
> more readable.
>
> This makes it a lot more clear what atomic_check does and why it's called in
> the place it is.
>
> Maarten Lankhorst (4):
> drm/atomic: Unify conflicting encoder handling.
> drm/atomic: Set all the changed flags in one place.
> drm/atomic: Move enable/connector check up in check_modeset()
> drm/atomic: Add connector atomic_check function, v2.
Hi Maarten,
For the set,
Reviewed-by: Sean Paul <seanpaul@chromium.org>
>
> drivers/gpu/drm/drm_atomic_helper.c | 84 ++++++++++++++++++++++----------
> include/drm/drm_atomic.h | 2 -
> include/drm/drm_modeset_helper_vtables.h | 34 +++++++++++++
> 3 files changed, 93 insertions(+), 27 deletions(-)
>
> --
> 2.7.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2017-04-06 12:48 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-06 11:18 [PATCH 0/4] drm/atomic: Cleanups for adding connector atomic check function Maarten Lankhorst
2017-04-06 11:19 ` [PATCH 1/4] drm/atomic: Unify conflicting encoder handling Maarten Lankhorst
2017-04-06 11:19 ` [PATCH 2/4] drm/atomic: Set all the changed flags in one place Maarten Lankhorst
2017-04-06 11:19 ` [PATCH 3/4] drm/atomic: Move enable/connector check up in check_modeset() Maarten Lankhorst
2017-04-06 11:19 ` [PATCH 4/4] drm/atomic: Add connector atomic_check function, v2 Maarten Lankhorst
2017-04-06 11:36 ` ✓ Fi.CI.BAT: success for drm/atomic: Cleanups for adding connector atomic check function Patchwork
2017-04-06 12:48 ` Sean Paul [this message]
2017-04-06 21:04 ` [PATCH 0/4] " 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=20170406124836.4phve7bwjsxaflny@art_vandelay \
--to=seanpaul@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=maarten.lankhorst@linux.intel.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.