From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 1/2] drm/i915: Don't set the 8to6 dither flag when not scaling Date: Sun, 13 Apr 2014 11:52:47 +0200 Message-ID: <20140413094941.GA8068@phenom.ffwll.local> References: <1390813231-15476-1-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f53.google.com (mail-ee0-f53.google.com [74.125.83.53]) by gabe.freedesktop.org (Postfix) with ESMTP id DFDF06E07F for ; Sun, 13 Apr 2014 02:52:57 -0700 (PDT) Received: by mail-ee0-f53.google.com with SMTP id b57so5498801eek.40 for ; Sun, 13 Apr 2014 02:52:55 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1390813231-15476-1-git-send-email-daniel.vetter@ffwll.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Intel Graphics Development Cc: Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org On Mon, Jan 27, 2014 at 10:00:30AM +0100, Daniel Vetter wrote: > Apparently we really only need this when the pfit is enabled, at least > I couldn't dicern any difference here. Furthermore the hacks we have > to reconstruct this bit is a bit glaring, and probably only works > because we can't move the lvds port to any other pipe than pipe B on > gen2/3. > > So let's just rip this out. > > Signed-off-by: Daniel Vetter I've just written this exact patch again, so figured I might as well merge it. r-b still appreciated ... -Daniel > --- > drivers/gpu/drm/i915/intel_lvds.c | 7 ------- > drivers/gpu/drm/i915/intel_panel.c | 8 ++++---- > 2 files changed, 4 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c > index 8bcb93a2a9f6..8a3ebe940a74 100644 > --- a/drivers/gpu/drm/i915/intel_lvds.c > +++ b/drivers/gpu/drm/i915/intel_lvds.c > @@ -111,13 +111,6 @@ static void intel_lvds_get_config(struct intel_encoder *encoder, > > pipe_config->adjusted_mode.flags |= flags; > > - /* gen2/3 store dither state in pfit control, needs to match */ > - if (INTEL_INFO(dev)->gen < 4) { > - tmp = I915_READ(PFIT_CONTROL); > - > - pipe_config->gmch_pfit.control |= tmp & PANEL_8TO6_DITHER_ENABLE; > - } > - > dotclock = pipe_config->port_clock; > > if (HAS_PCH_SPLIT(dev_priv->dev)) > diff --git a/drivers/gpu/drm/i915/intel_panel.c b/drivers/gpu/drm/i915/intel_panel.c > index 9f83ab06fb5e..41116753f1c0 100644 > --- a/drivers/gpu/drm/i915/intel_panel.c > +++ b/drivers/gpu/drm/i915/intel_panel.c > @@ -308,16 +308,16 @@ void intel_gmch_panel_fitting(struct intel_crtc *intel_crtc, > pfit_control |= ((intel_crtc->pipe << PFIT_PIPE_SHIFT) | > PFIT_FILTER_FUZZY); > > + /* Make sure pre-965 set dither correctly for 18bpp panels. */ > + if (INTEL_INFO(dev)->gen < 4 && pipe_config->pipe_bpp == 18) > + pfit_control |= PANEL_8TO6_DITHER_ENABLE; > + > out: > if ((pfit_control & PFIT_ENABLE) == 0) { > pfit_control = 0; > pfit_pgm_ratios = 0; > } > > - /* Make sure pre-965 set dither correctly for 18bpp panels. */ > - if (INTEL_INFO(dev)->gen < 4 && pipe_config->pipe_bpp == 18) > - pfit_control |= PANEL_8TO6_DITHER_ENABLE; > - > pipe_config->gmch_pfit.control = pfit_control; > pipe_config->gmch_pfit.pgm_ratios = pfit_pgm_ratios; > pipe_config->gmch_pfit.lvds_border_bits = border; > -- > 1.8.4.rc3 > -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch