From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Jindal, Sonika" <sonika.jindal@intel.com>
Subject: Re: [PATCH 0/6] Add 180 degree primary and sprite
	rotation
Date: Tue, 15 Jul 2014 16:25:57 +0530
Message-ID: <53C508BD.30301@intel.com>
References: <1405413629-4266-1-git-send-email-sonika.jindal@intel.com>
 <20140715105220.GD4919@strange.ger.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"; Format="flowed"
Content-Transfer-Encoding: quoted-printable
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mga09.intel.com (mga09.intel.com [134.134.136.24])
 by gabe.freedesktop.org (Postfix) with ESMTP id 945366E56C
 for <intel-gfx@lists.freedesktop.org>; Tue, 15 Jul 2014 03:55:59 -0700 (PDT)
In-Reply-To: <20140715105220.GD4919@strange.ger.corp.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>
Errors-To: intel-gfx-bounces@lists.freedesktop.org
Sender: "Intel-gfx" <intel-gfx-bounces@lists.freedesktop.org>
To: Damien Lespiau <damien.lespiau@intel.com>
Cc: intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org



On 7/15/2014 4:22 PM, Damien Lespiau wrote:
> On Tue, Jul 15, 2014 at 02:10:23PM +0530, sonika.jindal@intel.com wrote:
>> From: Sonika Jindal <sonika.jindal@intel.com>
>>
>> This patchset provides support for 0/180 degree hardare rotaion for prim=
ary and
>> sprite planes. The rotation property is now made global and is part of
>> drm_mode_config. It is attached to different planes.
>>
>> Sonika Jindal (3):
>>    drm: Add rotation_property to mode_config
>>    drm/i915: Add 180 degree primary plane rotation support
>>    drm: Resetting rotation property
>>
>> Ville Syrj=E4l=E4 (3):
>>    drm/i915: Add 180 degree sprite rotation support
>>    drm/i915: Make intel_plane_restore() return an error
>>    drm/i915: Add rotation property for sprites
>
> Also, you forgot to bump the series version, it's really nice to have it
> to follow the progress (and later, to have tools do that for us).
>
Since this was a new patch set without the previous patches I thought it =

is not needed. Sure I will keep this in mind for the next time.

> I noticed you likely used --subject-prefix=3D"v3" in the previous one,
> that's not quite how one is supposed to do it. The series versioning
> goes into the cover letter subject, and that's it.
>
I was not aware of this. I thought every patch has to have the version =

number. Thanks for pointing it out.
> Cheers,
>