From mboxrd@z Thu Jan 1 00:00:00 1970 From: Damien Lespiau Subject: Re: [PATCH v2 1/1] drm/i915: Enabling 128x128 and 256x256 ARGB Cursor Support Date: Mon, 24 Mar 2014 16:30:02 +0000 Message-ID: <20140324163002.GD12622@strange.icx.intel.com> References: <20140218131333.GR3852@intel.com> <1393256503-32274-1-git-send-email-sagar.a.kamble@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTP id 54E256EAC7 for ; Mon, 24 Mar 2014 09:30:22 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1393256503-32274-1-git-send-email-sagar.a.kamble@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: sagar.a.kamble@intel.com Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, Feb 24, 2014 at 09:11:43PM +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. I've just noticed that we currently hardcode 64 pixels for the cursor WM computation. The WM code needs to be updated to take into account that the cursor plane can now have different sizes: p->cur.horiz_pixels = 64; -- Damien