[Intel-gfx] [PATCH v4 02/10] drm/i915: prefer 3-letter acronym for pineview
Matt Roper
matthew.d.roper at intel.com
Tue Dec 24 00:23:20 UTC 2019
On Mon, Dec 23, 2019 at 04:05:20PM -0800, Lucas De Marchi wrote:
> We are currently using a mix of platform name and acronym to name the
> functions. Let's prefer the acronym as it should be clear what platform
> it's about and it's shorter, so it doesn't go over 80 columns in a few
> cases. This converts pineview to pnv where appropriate.
>
> v2: Add missing conversions in intel_pm.c (Matt Roper). While at it, fix
> missing blank lines between structs that would otherwise trigger
> checkpatch errors (Lucas)
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
> Acked-by: Jani Nikula <jani.nikula at linux.intel.com>
> Acked-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
Reviewed-by: Matt Roper <matthew.d.roper at intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_display.c | 8 ++--
> drivers/gpu/drm/i915/intel_pm.c | 41 ++++++++++++--------
> 2 files changed, 29 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index 652b8800d585..930362124808 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -370,7 +370,7 @@ static const struct intel_limit intel_limits_g4x_dual_channel_lvds = {
> },
> };
>
> -static const struct intel_limit intel_limits_pineview_sdvo = {
> +static const struct intel_limit pnv_limits_sdvo = {
> .dot = { .min = 20000, .max = 400000},
> .vco = { .min = 1700000, .max = 3500000 },
> /* Pineview's Ncounter is a ring counter */
> @@ -385,7 +385,7 @@ static const struct intel_limit intel_limits_pineview_sdvo = {
> .p2_slow = 10, .p2_fast = 5 },
> };
>
> -static const struct intel_limit intel_limits_pineview_lvds = {
> +static const struct intel_limit pnv_limits_lvds = {
> .dot = { .min = 20000, .max = 400000 },
> .vco = { .min = 1700000, .max = 3500000 },
> .n = { .min = 3, .max = 6 },
> @@ -8779,9 +8779,9 @@ static int pnv_crtc_compute_clock(struct intel_crtc *crtc,
> DRM_DEBUG_KMS("using SSC reference clock of %d kHz\n", refclk);
> }
>
> - limit = &intel_limits_pineview_lvds;
> + limit = &pnv_limits_lvds;
> } else {
> - limit = &intel_limits_pineview_sdvo;
> + limit = &pnv_limits_sdvo;
> }
>
> if (!crtc_state->clock_set &&
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index 31ec82337e4f..b55e9d2d2a83 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -140,7 +140,7 @@ static void glk_init_clock_gating(struct drm_i915_private *dev_priv)
>
> }
>
> -static void i915_pineview_get_mem_freq(struct drm_i915_private *dev_priv)
> +static void pnv_get_mem_freq(struct drm_i915_private *dev_priv)
> {
> u32 tmp;
>
> @@ -549,34 +549,38 @@ static int i845_get_fifo_size(struct drm_i915_private *dev_priv,
> }
>
> /* Pineview has different values for various configs */
> -static const struct intel_watermark_params pineview_display_wm = {
> +static const struct intel_watermark_params pnv_display_wm = {
> .fifo_size = PINEVIEW_DISPLAY_FIFO,
> .max_wm = PINEVIEW_MAX_WM,
> .default_wm = PINEVIEW_DFT_WM,
> .guard_size = PINEVIEW_GUARD_WM,
> .cacheline_size = PINEVIEW_FIFO_LINE_SIZE,
> };
> -static const struct intel_watermark_params pineview_display_hplloff_wm = {
> +
> +static const struct intel_watermark_params pnv_display_hplloff_wm = {
> .fifo_size = PINEVIEW_DISPLAY_FIFO,
> .max_wm = PINEVIEW_MAX_WM,
> .default_wm = PINEVIEW_DFT_HPLLOFF_WM,
> .guard_size = PINEVIEW_GUARD_WM,
> .cacheline_size = PINEVIEW_FIFO_LINE_SIZE,
> };
> -static const struct intel_watermark_params pineview_cursor_wm = {
> +
> +static const struct intel_watermark_params pnv_cursor_wm = {
> .fifo_size = PINEVIEW_CURSOR_FIFO,
> .max_wm = PINEVIEW_CURSOR_MAX_WM,
> .default_wm = PINEVIEW_CURSOR_DFT_WM,
> .guard_size = PINEVIEW_CURSOR_GUARD_WM,
> .cacheline_size = PINEVIEW_FIFO_LINE_SIZE,
> };
> -static const struct intel_watermark_params pineview_cursor_hplloff_wm = {
> +
> +static const struct intel_watermark_params pnv_cursor_hplloff_wm = {
> .fifo_size = PINEVIEW_CURSOR_FIFO,
> .max_wm = PINEVIEW_CURSOR_MAX_WM,
> .default_wm = PINEVIEW_CURSOR_DFT_WM,
> .guard_size = PINEVIEW_CURSOR_GUARD_WM,
> .cacheline_size = PINEVIEW_FIFO_LINE_SIZE,
> };
> +
> static const struct intel_watermark_params i965_cursor_wm_info = {
> .fifo_size = I965_CURSOR_FIFO,
> .max_wm = I965_CURSOR_MAX_WM,
> @@ -584,6 +588,7 @@ static const struct intel_watermark_params i965_cursor_wm_info = {
> .guard_size = 2,
> .cacheline_size = I915_FIFO_LINE_SIZE,
> };
> +
> static const struct intel_watermark_params i945_wm_info = {
> .fifo_size = I945_FIFO_SIZE,
> .max_wm = I915_MAX_WM,
> @@ -591,6 +596,7 @@ static const struct intel_watermark_params i945_wm_info = {
> .guard_size = 2,
> .cacheline_size = I915_FIFO_LINE_SIZE,
> };
> +
> static const struct intel_watermark_params i915_wm_info = {
> .fifo_size = I915_FIFO_SIZE,
> .max_wm = I915_MAX_WM,
> @@ -598,6 +604,7 @@ static const struct intel_watermark_params i915_wm_info = {
> .guard_size = 2,
> .cacheline_size = I915_FIFO_LINE_SIZE,
> };
> +
> static const struct intel_watermark_params i830_a_wm_info = {
> .fifo_size = I855GM_FIFO_SIZE,
> .max_wm = I915_MAX_WM,
> @@ -605,6 +612,7 @@ static const struct intel_watermark_params i830_a_wm_info = {
> .guard_size = 2,
> .cacheline_size = I830_FIFO_LINE_SIZE,
> };
> +
> static const struct intel_watermark_params i830_bc_wm_info = {
> .fifo_size = I855GM_FIFO_SIZE,
> .max_wm = I915_MAX_WM/2,
> @@ -612,6 +620,7 @@ static const struct intel_watermark_params i830_bc_wm_info = {
> .guard_size = 2,
> .cacheline_size = I830_FIFO_LINE_SIZE,
> };
> +
> static const struct intel_watermark_params i845_wm_info = {
> .fifo_size = I830_FIFO_SIZE,
> .max_wm = I915_MAX_WM,
> @@ -848,7 +857,7 @@ static struct intel_crtc *single_enabled_crtc(struct drm_i915_private *dev_priv)
> return enabled;
> }
>
> -static void pineview_update_wm(struct intel_crtc *unused_crtc)
> +static void pnv_update_wm(struct intel_crtc *unused_crtc)
> {
> struct drm_i915_private *dev_priv = to_i915(unused_crtc->base.dev);
> struct intel_crtc *crtc;
> @@ -876,8 +885,8 @@ static void pineview_update_wm(struct intel_crtc *unused_crtc)
> int clock = adjusted_mode->crtc_clock;
>
> /* Display SR */
> - wm = intel_calculate_wm(clock, &pineview_display_wm,
> - pineview_display_wm.fifo_size,
> + wm = intel_calculate_wm(clock, &pnv_display_wm,
> + pnv_display_wm.fifo_size,
> cpp, latency->display_sr);
> reg = I915_READ(DSPFW1);
> reg &= ~DSPFW_SR_MASK;
> @@ -886,8 +895,8 @@ static void pineview_update_wm(struct intel_crtc *unused_crtc)
> DRM_DEBUG_KMS("DSPFW1 register is %x\n", reg);
>
> /* cursor SR */
> - wm = intel_calculate_wm(clock, &pineview_cursor_wm,
> - pineview_display_wm.fifo_size,
> + wm = intel_calculate_wm(clock, &pnv_cursor_wm,
> + pnv_display_wm.fifo_size,
> 4, latency->cursor_sr);
> reg = I915_READ(DSPFW3);
> reg &= ~DSPFW_CURSOR_SR_MASK;
> @@ -895,8 +904,8 @@ static void pineview_update_wm(struct intel_crtc *unused_crtc)
> I915_WRITE(DSPFW3, reg);
>
> /* Display HPLL off SR */
> - wm = intel_calculate_wm(clock, &pineview_display_hplloff_wm,
> - pineview_display_hplloff_wm.fifo_size,
> + wm = intel_calculate_wm(clock, &pnv_display_hplloff_wm,
> + pnv_display_hplloff_wm.fifo_size,
> cpp, latency->display_hpll_disable);
> reg = I915_READ(DSPFW3);
> reg &= ~DSPFW_HPLL_SR_MASK;
> @@ -904,8 +913,8 @@ static void pineview_update_wm(struct intel_crtc *unused_crtc)
> I915_WRITE(DSPFW3, reg);
>
> /* cursor HPLL off SR */
> - wm = intel_calculate_wm(clock, &pineview_cursor_hplloff_wm,
> - pineview_display_hplloff_wm.fifo_size,
> + wm = intel_calculate_wm(clock, &pnv_cursor_hplloff_wm,
> + pnv_display_hplloff_wm.fifo_size,
> 4, latency->cursor_hpll_disable);
> reg = I915_READ(DSPFW3);
> reg &= ~DSPFW_HPLL_CURSOR_MASK;
> @@ -7180,7 +7189,7 @@ void intel_init_pm(struct drm_i915_private *dev_priv)
> {
> /* For cxsr */
> if (IS_PINEVIEW(dev_priv))
> - i915_pineview_get_mem_freq(dev_priv);
> + pnv_get_mem_freq(dev_priv);
> else if (IS_GEN(dev_priv, 5))
> i915_ironlake_get_mem_freq(dev_priv);
>
> @@ -7238,7 +7247,7 @@ void intel_init_pm(struct drm_i915_private *dev_priv)
> intel_set_memory_cxsr(dev_priv, false);
> dev_priv->display.update_wm = NULL;
> } else
> - dev_priv->display.update_wm = pineview_update_wm;
> + dev_priv->display.update_wm = pnv_update_wm;
> } else if (IS_GEN(dev_priv, 4)) {
> dev_priv->display.update_wm = i965_update_wm;
> } else if (IS_GEN(dev_priv, 3)) {
> --
> 2.24.0
>
--
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795
More information about the Intel-gfx
mailing list