From: Daniel Vetter <daniel@ffwll.ch>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org, sagar.a.kamble@intel.com
Subject: Re: [PATCH v3 00/12] Enabling 180 degree rotation for sprite and crtc planes
Date: Thu, 6 Feb 2014 18:13:41 +0100 [thread overview]
Message-ID: <20140206171341.GR17001@phenom.ffwll.local> (raw)
In-Reply-To: <20140206162915.GU3891@intel.com>
On Thu, Feb 06, 2014 at 06:29:15PM +0200, Ville Syrjälä wrote:
> On Thu, Feb 06, 2014 at 08:34:41PM +0530, sagar.a.kamble@intel.com wrote:
> > From: Sagar Kamble <sagar.a.kamble@intel.com>
> >
> > These patches will enable 180 degree rotation for CRTC and Sprite planes. This
> > version has following changes:
> > 1. Addressed review comments for CRTC rotation from FBC, page flip, CRTC active/
> > inactive perspective.
> > 2. Removed drm_rect_rotate amd drm_rect_rotate_inv functions as they dont take
> > care of clipped planes and 180 rotation in that case produces invalid result.
> >
> > Sagar Kamble (3):
> > drm/i915: Add 180 degree primary plane rotation support
> > drm: Set property to return invalid for unsupported arguments for
> > bitmask property
> > drm/i915: Removing rotate and inverse rotate calls from update_plane
> >
> > Ville Syrjälä (9):
> > drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h
> > drm: Add support_bits parameter to drm_property_create_bitmask()
> > drm: Add drm_mode_create_rotation_property()
> > drm/omap: Switch omapdrm over to drm_mode_create_rotation_property()
> > drm: Add drm_rect rotation functions
> > drm: Add drm_rotation_simplify()
> > drm/i915: Add 180 degree sprite rotation support
> > drm/i915: Make intel_plane_restore() return an error
> > drm/i915: Add rotation property for sprites
>
> Oh I forgot to mention it last time around, since this series touches
> more than i915, it should be cc:ed to dri-devel as well.
Ah, mail cross-over. Another process-thing: When resubmitting patches
please put a little changelog into both your cover letter and individual
patches. That way reviewers can quickly gauge what changed and concentrate
on looking at just those parts. That helps with streamline the review
process.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
prev parent reply other threads:[~2014-02-06 17:13 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-06 15:04 [PATCH v3 00/12] Enabling 180 degree rotation for sprite and crtc planes sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h sagar.a.kamble
2014-02-06 16:51 ` Daniel Vetter
2014-02-06 15:04 ` [PATCH v3 02/12] drm: Add support_bits parameter to drm_property_create_bitmask() sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 03/12] drm: Add drm_mode_create_rotation_property() sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 04/12] drm/omap: Switch omapdrm over to drm_mode_create_rotation_property() sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 05/12] drm: Add drm_rect rotation functions sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 06/12] drm: Add drm_rotation_simplify() sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 07/12] drm/i915: Add 180 degree sprite rotation support sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 08/12] drm/i915: Make intel_plane_restore() return an error sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 09/12] drm/i915: Add rotation property for sprites sagar.a.kamble
2014-02-06 15:04 ` [PATCH v3 10/12] drm/i915: Add 180 degree primary plane rotation support sagar.a.kamble
2014-02-06 15:56 ` Ville Syrjälä
2014-02-06 15:04 ` [PATCH v3 11/12] drm: Set property to return invalid for unsupported arguments for bitmask property sagar.a.kamble
2014-02-06 16:21 ` Ville Syrjälä
2014-02-06 15:04 ` [PATCH v3 12/12] drm/i915: Removing rotate and inverse rotate calls from update_plane sagar.a.kamble
2014-02-06 15:55 ` Ville Syrjälä
2014-02-06 16:29 ` [PATCH v3 00/12] Enabling 180 degree rotation for sprite and crtc planes Ville Syrjälä
2014-02-06 17:13 ` Daniel Vetter [this message]
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=20140206171341.GR17001@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=sagar.a.kamble@intel.com \
--cc=ville.syrjala@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox