On Sat, Dec 14, 2013 at 08:38:30PM -0200, Rodrigo Vivi wrote: > From: Daniel Vetter <daniel.vetter@xxxxxxxx> > > 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 <richter@xxxxxxxxxxxxxxxxxxxx> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > 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 = dev->dev_private; > uint32_t dsparb = 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 = dev->dev_private; > - uint32_t dsparb = I915_READ(DSPARB); > - int size; > - > - size = dsparb & 0x7f; > - size >>= 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 = { > PINEVIEW_DISPLAY_FIFO, > @@ -950,14 +935,14 @@ static const struct intel_watermark_params i915_wm_info = { > 2, > I915_FIFO_LINE_SIZE > }; > -static const struct intel_watermark_params i855_wm_info = { > +static const struct intel_watermark_params i830_wm_info = { > I855GM_FIFO_SIZE, > I915_MAX_WM, > 1, > 2, > I830_FIFO_LINE_SIZE > }; > -static const struct intel_watermark_params i830_wm_info = { > +static const struct intel_watermark_params i845_wm_info = { > 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 = &i915_wm_info; > else > - wm_info = &i855_wm_info; > + wm_info = &i830_wm_info; > > fifo_size = dev_priv->display.get_fifo_size(dev, 0); > crtc = 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 = unused_crtc->dev; > struct drm_i915_private *dev_priv = dev->dev_private; > @@ -1696,7 +1681,7 @@ static void i830_update_wm(struct drm_crtc *unused_crtc) > > adjusted_mode = &to_intel_crtc(crtc)->config.adjusted_mode; > planea_wm = 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 = I915_READ(FW_BLC) & ~0xfff; > @@ -6180,21 +6165,21 @@ void intel_init_pm(struct drm_device *dev) > dev_priv->display.update_wm = i9xx_update_wm; > dev_priv->display.get_fifo_size = i9xx_get_fifo_size; > dev_priv->display.init_clock_gating = gen3_init_clock_gating; > - } else if (IS_I865G(dev)) { > - dev_priv->display.update_wm = i830_update_wm; > - dev_priv->display.init_clock_gating = i85x_init_clock_gating; > - dev_priv->display.get_fifo_size = i830_get_fifo_size; > - } else if (IS_I85X(dev)) { > - dev_priv->display.update_wm = i9xx_update_wm; > - dev_priv->display.get_fifo_size = i85x_get_fifo_size; > - dev_priv->display.init_clock_gating = i85x_init_clock_gating; > - } else { > - dev_priv->display.update_wm = i830_update_wm; > - dev_priv->display.init_clock_gating = i830_init_clock_gating; > - if (IS_845G(dev)) > + } else if (IS_GEN2(dev)) { > + if (INTEL_INFO(dev)->num_pipes == 1) { > + dev_priv->display.update_wm = i845_update_wm; > dev_priv->display.get_fifo_size = i845_get_fifo_size; > - else > + } else { > + dev_priv->display.update_wm = i9xx_update_wm; > dev_priv->display.get_fifo_size = i830_get_fifo_size; > + } > + > + if (IS_I85X(dev) || IS_I865G(dev)) > + dev_priv->display.init_clock_gating = i85x_init_clock_gating; > + else > + dev_priv->display.init_clock_gating = 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@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx