From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Roper Subject: Re: [PATCH] drm/i915: Intel-specific primary plane handling (v4) Date: Tue, 22 Apr 2014 10:32:39 -0700 Message-ID: <20140422173239.GR1063@intel.com> References: <1397249099-11235-1-git-send-email-matthew.d.roper@intel.com> <1397250822-17280-1-git-send-email-matthew.d.roper@intel.com> <20140422124737.GV18465@intel.com> <20140422151830.GN1063@intel.com> <20140422171422.GV10722@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id EAC3C6E926 for ; Tue, 22 Apr 2014 10:33:54 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140422171422.GV10722@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Apr 22, 2014 at 07:14:22PM +0200, Daniel Vetter wrote: > On Tue, Apr 22, 2014 at 08:18:30AM -0700, Matt Roper wrote: > > On Tue, Apr 22, 2014 at 03:47:37PM +0300, Ville Syrj=E4l=E4 wrote: > > > On Fri, Apr 11, 2014 at 02:13:42PM -0700, Matt Roper wrote: > > ... > > > > + int ret; > > > > + > > > > + /* > > > > + * At the moment we use the same set of setplane restrictions as = the > > > > + * DRM primary plane helper, so go ahead and just call the helper= if > > > > + * the primary plane is already enabled. We only need to take sp= ecial > > > > + * action if the primary plane is disabled (something i915 can do= but > > > > + * the generic helper can't). > > > > + */ > > > > + if (intel_crtc->primary_enabled) > > > > + return drm_primary_helper_update(plane, crtc, fb, > > > > + crtc_x, crtc_y, > > > > + crtc_w, crtc_h, > > > > + src_x, src_y, > > > > + src_w, src_h); > > > = > > > Why would we want to call that if we have a custom implementation > > > anyway? > > = > > This was something Daniel requested on a previous patch iteration; even > > though we're stuck duplicating most of the checks here for the !enabled > > case, he still wanted to see us call into the helper for the enabled > > case (although this will have to change in the future if/when we want to > > start relaxing some of the tests that the helper does, such as plane > > scaling). > = > To clarify my request: I was unhappy with all the duplicated tests we have > and would like some way to share them with the plane helper code. If > there's no sane way to do that, then I'm ok with duplication. > = > I'm not sure any more what was the issue with extracting the tests from > the plane helper into a new function and reusing them with i915 though. > -Daniel Ah, okay. I think I may have misunderstood what you were asking for the previous time around. Extracting the tests from the helper into a new function should be doable; I'll include that in my next iteration. Sorry for the confusion. Matt -- = Matt Roper Graphics Software Engineer IoTG Platform Enabling & Development Intel Corporation (916) 356-2795