From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 3/5] drm/i915: i830M has watermarks like i855 Date: Fri, 10 Jan 2014 13:09:38 +0200 Message-ID: <20140110110938.GS4800@intel.com> References: <1387060712-5081-1-git-send-email-rodrigo.vivi@gmail.com> <1387060712-5081-4-git-send-email-rodrigo.vivi@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 34B78FB8A4 for ; Fri, 10 Jan 2014 03:09:42 -0800 (PST) Content-Disposition: inline In-Reply-To: <1387060712-5081-4-git-send-email-rodrigo.vivi@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Rodrigo Vivi Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, Thomas Richter List-Id: intel-gfx@lists.freedesktop.org On Sat, Dec 14, 2013 at 08:38:30PM -0200, Rodrigo Vivi wrote: > From: Daniel Vetter > = > So shuffle the checks around a bit. Also give all the structs and > functions proper prefixes: i830_ for the dual-pipe mobile platforms > and i845_ for the two single-pipe desktop platforms. > = > Note that the max fifo value isn't actually correct for the i830M, but > since we don't frob the fifo split we don't actually need it. This is > different for some gen3 devices where we need the full fifo for self > refresh mode. > = > Cc: Thomas Richter > Signed-off-by: Daniel Vetter > Signed-off-by: Rodrigo Vivi Reviewed-by: Ville Syrj=E4l=E4 > --- > drivers/gpu/drm/i915/intel_pm.c | 53 +++++++++++++++--------------------= ------ > 1 file changed, 19 insertions(+), 34 deletions(-) > = > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel= _pm.c > index bcb8470..465304a 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -824,7 +824,7 @@ static int i9xx_get_fifo_size(struct drm_device *dev,= int plane) > return size; > } > = > -static int i85x_get_fifo_size(struct drm_device *dev, int plane) > +static int i830_get_fifo_size(struct drm_device *dev, int plane) > { > struct drm_i915_private *dev_priv =3D dev->dev_private; > uint32_t dsparb =3D I915_READ(DSPARB); > @@ -857,21 +857,6 @@ static int i845_get_fifo_size(struct drm_device *dev= , int plane) > return size; > } > = > -static int i830_get_fifo_size(struct drm_device *dev, int plane) > -{ > - struct drm_i915_private *dev_priv =3D dev->dev_private; > - uint32_t dsparb =3D I915_READ(DSPARB); > - int size; > - > - size =3D dsparb & 0x7f; > - size >>=3D 1; /* Convert to cachelines */ > - > - DRM_DEBUG_KMS("FIFO size - (0x%08x) %s: %d\n", dsparb, > - plane ? "B" : "A", size); > - > - return size; > -} > - > /* Pineview has different values for various configs */ > static const struct intel_watermark_params pineview_display_wm =3D { > PINEVIEW_DISPLAY_FIFO, > @@ -950,14 +935,14 @@ static const struct intel_watermark_params i915_wm_= info =3D { > 2, > I915_FIFO_LINE_SIZE > }; > -static const struct intel_watermark_params i855_wm_info =3D { > +static const struct intel_watermark_params i830_wm_info =3D { > I855GM_FIFO_SIZE, > I915_MAX_WM, > 1, > 2, > I830_FIFO_LINE_SIZE > }; > -static const struct intel_watermark_params i830_wm_info =3D { > +static const struct intel_watermark_params i845_wm_info =3D { > I830_FIFO_SIZE, > I915_MAX_WM, > 1, > @@ -1574,7 +1559,7 @@ static void i9xx_update_wm(struct drm_crtc *unused_= crtc) > else if (!IS_GEN2(dev)) > wm_info =3D &i915_wm_info; > else > - wm_info =3D &i855_wm_info; > + wm_info =3D &i830_wm_info; > = > fifo_size =3D dev_priv->display.get_fifo_size(dev, 0); > crtc =3D intel_get_crtc_for_plane(dev, 0); > @@ -1681,7 +1666,7 @@ static void i9xx_update_wm(struct drm_crtc *unused_= crtc) > } > } > = > -static void i830_update_wm(struct drm_crtc *unused_crtc) > +static void i845_update_wm(struct drm_crtc *unused_crtc) > { > struct drm_device *dev =3D unused_crtc->dev; > struct drm_i915_private *dev_priv =3D dev->dev_private; > @@ -1696,7 +1681,7 @@ static void i830_update_wm(struct drm_crtc *unused_= crtc) > = > adjusted_mode =3D &to_intel_crtc(crtc)->config.adjusted_mode; > planea_wm =3D intel_calculate_wm(adjusted_mode->crtc_clock, > - &i830_wm_info, > + &i845_wm_info, > dev_priv->display.get_fifo_size(dev, 0), > 4, latency_ns); > fwater_lo =3D I915_READ(FW_BLC) & ~0xfff; > @@ -6180,21 +6165,21 @@ void intel_init_pm(struct drm_device *dev) > dev_priv->display.update_wm =3D i9xx_update_wm; > dev_priv->display.get_fifo_size =3D i9xx_get_fifo_size; > dev_priv->display.init_clock_gating =3D gen3_init_clock_gating; > - } else if (IS_I865G(dev)) { > - dev_priv->display.update_wm =3D i830_update_wm; > - dev_priv->display.init_clock_gating =3D i85x_init_clock_gating; > - dev_priv->display.get_fifo_size =3D i830_get_fifo_size; > - } else if (IS_I85X(dev)) { > - dev_priv->display.update_wm =3D i9xx_update_wm; > - dev_priv->display.get_fifo_size =3D i85x_get_fifo_size; > - dev_priv->display.init_clock_gating =3D i85x_init_clock_gating; > - } else { > - dev_priv->display.update_wm =3D i830_update_wm; > - dev_priv->display.init_clock_gating =3D i830_init_clock_gating; > - if (IS_845G(dev)) > + } else if (IS_GEN2(dev)) { > + if (INTEL_INFO(dev)->num_pipes =3D=3D 1) { > + dev_priv->display.update_wm =3D i845_update_wm; > dev_priv->display.get_fifo_size =3D i845_get_fifo_size; > - else > + } else { > + dev_priv->display.update_wm =3D i9xx_update_wm; > dev_priv->display.get_fifo_size =3D i830_get_fifo_size; > + } > + > + if (IS_I85X(dev) || IS_I865G(dev)) > + dev_priv->display.init_clock_gating =3D i85x_init_clock_gating; > + else > + dev_priv->display.init_clock_gating =3D i830_init_clock_gating; > + } else { > + DRM_ERROR("unexpected fall-through in intel_init_pm\n"); > } > } > = > -- = > 1.8.3.1 > = > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- = Ville Syrj=E4l=E4 Intel OTC