From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [PATCH v3 00/12] Enabling 180 degree rotation for
 sprite and crtc planes
Date: Thu, 6 Feb 2014 18:13:41 +0100
Message-ID: <20140206171341.GR17001@phenom.ffwll.local>
References: <1391699093-12625-1-git-send-email-sagar.a.kamble@intel.com>
	<20140206162915.GU3891@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mail-ea0-f180.google.com (mail-ea0-f180.google.com
	[209.85.215.180])
	by gabe.freedesktop.org (Postfix) with ESMTP id C77D3FA64D
	for <intel-gfx@lists.freedesktop.org>;
	Thu,  6 Feb 2014 09:13:47 -0800 (PST)
Received: by mail-ea0-f180.google.com with SMTP id o10so1049189eaj.11
	for <intel-gfx@lists.freedesktop.org>;
	Thu, 06 Feb 2014 09:13:45 -0800 (PST)
Content-Disposition: inline
In-Reply-To: <20140206162915.GU3891@intel.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces@lists.freedesktop.org
Errors-To: intel-gfx-bounces@lists.freedesktop.org
To: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org, sagar.a.kamble@intel.com
List-Id: intel-gfx@lists.freedesktop.org

On Thu, Feb 06, 2014 at 06:29:15PM +0200, Ville Syrj=E4l=E4 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 plane=
s. This
> > version has following changes:
> > 1. Addressed review comments for CRTC rotation from FBC, page flip, CRT=
C active/
> > inactive perspective.
> > 2. Removed drm_rect_rotate amd drm_rect_rotate_inv functions as they do=
nt take
> > care of clipped planes and 180 rotation in that case produces invalid r=
esult.
> > =

> > 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=E4l=E4 (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