From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: Drop unused lut tables from intel_plane Date: Tue, 3 Jun 2014 09:48:59 +0200 Message-ID: <20140603074859.GK19050@phenom.ffwll.local> References: <1401729126-15529-1-git-send-email-matthew.d.roper@intel.com> <20140602180652.GA32209@strange.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f182.google.com (mail-wi0-f182.google.com [209.85.212.182]) by gabe.freedesktop.org (Postfix) with ESMTP id E2EFB6E6BB for ; Tue, 3 Jun 2014 00:49:06 -0700 (PDT) Received: by mail-wi0-f182.google.com with SMTP id r20so6033151wiv.9 for ; Tue, 03 Jun 2014 00:49:03 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140602180652.GA32209@strange.amr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Damien Lespiau Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, Jun 02, 2014 at 07:06:52PM +0100, Damien Lespiau wrote: > On Mon, Jun 02, 2014 at 10:12:06AM -0700, Matt Roper wrote: > > Signed-off-by: Matt Roper > > The commit message is a bit terse and could do with some digging. > Something like: > > Those LUT where defined in the original sprite patch introducing intel_plane, > but were never used. > > commit b840d907fcf6d5d5ef91af4518b3dab3a5da0f75 > Author: Jesse Barnes > Date: Tue Dec 13 13:19:38 2011 -0800 > > drm/i915: add SNB and IVB video sprite support v6 Added. > Reviewed-by: Damien Lespiau Queued for -next, thanks for the patch. -Daniel > > > --- > > drivers/gpu/drm/i915/intel_drv.h | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h > > index 76de420..fea8e05 100644 > > --- a/drivers/gpu/drm/i915/intel_drv.h > > +++ b/drivers/gpu/drm/i915/intel_drv.h > > @@ -428,7 +428,6 @@ struct intel_plane { > > struct drm_i915_gem_object *obj; > > bool can_scale; > > int max_downscale; > > - u32 lut_r[1024], lut_g[1024], lut_b[1024]; > > int crtc_x, crtc_y; > > unsigned int crtc_w, crtc_h; > > uint32_t src_x, src_y; > > -- > > 1.8.5.1 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch