From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v4 1/1] drm/i915: Enabling 128x128 and 256x256 ARGB Cursor Support Date: Thu, 20 Mar 2014 17:35:53 +0100 Message-ID: <20140320163553.GU30571@phenom.ffwll.local> References: <20140310100321.GA30571@phenom.ffwll.local> <1394451383-27133-1-git-send-email-sagar.a.kamble@intel.com> <1395329443.17233.10.camel@intelbox> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f49.google.com (mail-ee0-f49.google.com [74.125.83.49]) by gabe.freedesktop.org (Postfix) with ESMTP id 985F86E134 for ; Thu, 20 Mar 2014 09:35:58 -0700 (PDT) Received: by mail-ee0-f49.google.com with SMTP id c41so862021eek.22 for ; Thu, 20 Mar 2014 09:35:57 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1395329443.17233.10.camel@intelbox> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Imre Deak Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, "G, Pallavi" , sagar.a.kamble@intel.com List-Id: dri-devel@lists.freedesktop.org On Thu, Mar 20, 2014 at 05:30:43PM +0200, Imre Deak wrote: > On Mon, 2014-03-10 at 17:06 +0530, sagar.a.kamble@intel.com wrote: > > From: Sagar Kamble > > > > With this patch we allow larger cursor planes of sizes 128x128 > > and 256x256. > > > > v2: Added more precise check on size while setting cursor plane. > > > > v3: Changes related to restructuring cursor size restrictions > > and DRM_DEBUG usage. > > > > v4: Indentation related changes for setting cursor control and > > implementing DRM_CAP_CURSOR_WIDTH and DRM_CAP_CURSOR_HEIGHT > > > > Testcase: igt/kms_cursor_crc > > Cc: Daniel Vetter > > Cc: Jani Nikula > > Cc: David Airlie > > Cc: dri-devel@lists.freedesktop.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: G, Pallavi > > Signed-off-by: Sagar Kamble > > Looks ok: > Reviewed-by: Imre Deak Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759301AbaCTQgA (ORCPT ); Thu, 20 Mar 2014 12:36:00 -0400 Received: from mail-ee0-f46.google.com ([74.125.83.46]:34375 "EHLO mail-ee0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757481AbaCTQf7 (ORCPT ); Thu, 20 Mar 2014 12:35:59 -0400 Date: Thu, 20 Mar 2014 17:35:53 +0100 From: Daniel Vetter To: Imre Deak Cc: sagar.a.kamble@intel.com, Daniel Vetter , intel-gfx@lists.freedesktop.org, "G, Pallavi" , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH v4 1/1] drm/i915: Enabling 128x128 and 256x256 ARGB Cursor Support Message-ID: <20140320163553.GU30571@phenom.ffwll.local> Mail-Followup-To: Imre Deak , sagar.a.kamble@intel.com, intel-gfx@lists.freedesktop.org, "G, Pallavi" , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org References: <20140310100321.GA30571@phenom.ffwll.local> <1394451383-27133-1-git-send-email-sagar.a.kamble@intel.com> <1395329443.17233.10.camel@intelbox> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1395329443.17233.10.camel@intelbox> X-Operating-System: Linux phenom 3.12.0+ User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 20, 2014 at 05:30:43PM +0200, Imre Deak wrote: > On Mon, 2014-03-10 at 17:06 +0530, sagar.a.kamble@intel.com wrote: > > From: Sagar Kamble > > > > With this patch we allow larger cursor planes of sizes 128x128 > > and 256x256. > > > > v2: Added more precise check on size while setting cursor plane. > > > > v3: Changes related to restructuring cursor size restrictions > > and DRM_DEBUG usage. > > > > v4: Indentation related changes for setting cursor control and > > implementing DRM_CAP_CURSOR_WIDTH and DRM_CAP_CURSOR_HEIGHT > > > > Testcase: igt/kms_cursor_crc > > Cc: Daniel Vetter > > Cc: Jani Nikula > > Cc: David Airlie > > Cc: dri-devel@lists.freedesktop.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: G, Pallavi > > Signed-off-by: Sagar Kamble > > Looks ok: > Reviewed-by: Imre Deak Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch