From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/1] drm/i915: Enabling plane rotation control through sysfs Date: Fri, 24 Jan 2014 16:24:46 +0200 Message-ID: <20140124142446.GF9454@intel.com> References: <1390571323-4885-1-git-send-email-sagar.a.kamble@intel.com> <20140124141410.GE9454@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 92514FA39D for ; Fri, 24 Jan 2014 06:24:49 -0800 (PST) Content-Disposition: inline In-Reply-To: <20140124141410.GE9454@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: sagar.a.kamble@intel.com Cc: intel-gfx@lists.freedesktop.org, Uma Shankar List-Id: intel-gfx@lists.freedesktop.org On Fri, Jan 24, 2014 at 04:14:10PM +0200, Ville Syrj=E4l=E4 wrote: > On Fri, Jan 24, 2014 at 07:18:43PM +0530, sagar.a.kamble@intel.com wrote: > > From: Sagar Kamble > > = > > This patch enables 180 degree rotation for primary and sprite planes > > through sysfs interface. > = > NAK. > = > See here for the right apporach: > http://lists.freedesktop.org/archives/intel-gfx/2013-September/033951.html And BTW if you're really interested in getting this stuff in, we (Daniel actually :) need tests in i-g-t. We have the display CRC stuff availalbe, so fully automated tests are possible for this stuff. So this is definitely one area where we're lacking manpower. Some ideas for simple sprite tests: - make sure it appears on the right spot on the screen - make sure it doesn't appear when off screen - make sure it clips correctly when partially off screen - repeat everything w/ 180 degree rotation One idea for a color key test: - clear primary to solid color (or some fancier image) - clear sprite to another color (or some fancier image) - fill a small rect in the color key on the primary - place the sprite on top of the color keyed are but make it slightly bigger than the area, and make sure only the color keyed area shows the sprite So this would all involve rendering the reference images w/ software, and then reproducing the expected result using the sprite, and making sure it all matches up. Finally come up with more tests as more properties get added (z order, blending, etc.). -- = Ville Syrj=E4l=E4 Intel OTC