drm/i915: move skl_preferred_vco_freq to display substruct

The info is related to display, and should be placed under
i915->display.

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Acked-by: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/4e9877d1641bce905cc9dd5eabe40df51ddbe9b5.1712599670.git.jani.nikula@intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
index d61aa5b7cbdb86f3a35a1b1a0592761e92e04caf..950942dc3d607822a4ea2b9e403292e3b01ed4f0 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_cdclk.c
+++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
@@ -1021,15 +1021,14 @@ static int skl_cdclk_decimal(int cdclk)
        return DIV_ROUND_CLOSEST(cdclk - 1000, 500);
 }
 
-static void skl_set_preferred_cdclk_vco(struct drm_i915_private *dev_priv,
-                                       int vco)
+static void skl_set_preferred_cdclk_vco(struct drm_i915_private *i915, int vco)
 {
-       bool changed = dev_priv->skl_preferred_vco_freq != vco;
+       bool changed = i915->display.cdclk.skl_preferred_vco_freq != vco;
 
-       dev_priv->skl_preferred_vco_freq = vco;
+       i915->display.cdclk.skl_preferred_vco_freq = vco;
 
        if (changed)
-               intel_update_max_cdclk(dev_priv);
+               intel_update_max_cdclk(i915);
 }
 
 static u32 skl_dpll0_link_rate(struct drm_i915_private *dev_priv, int vco)
@@ -1233,7 +1232,7 @@ static void skl_cdclk_init_hw(struct drm_i915_private *dev_priv)
                 * Use the current vco as our initial
                 * guess as to what the preferred vco is.
                 */
-               if (dev_priv->skl_preferred_vco_freq == 0)
+               if (dev_priv->display.cdclk.skl_preferred_vco_freq == 0)
                        skl_set_preferred_cdclk_vco(dev_priv,
                                                    dev_priv->display.cdclk.hw.vco);
                return;
@@ -1241,7 +1240,7 @@ static void skl_cdclk_init_hw(struct drm_i915_private *dev_priv)
 
        cdclk_config = dev_priv->display.cdclk.hw;
 
-       cdclk_config.vco = dev_priv->skl_preferred_vco_freq;
+       cdclk_config.vco = dev_priv->display.cdclk.skl_preferred_vco_freq;
        if (cdclk_config.vco == 0)
                cdclk_config.vco = 8100000;
        cdclk_config.cdclk = skl_calc_cdclk(0, cdclk_config.vco);
@@ -3011,7 +3010,7 @@ static int skl_dpll0_vco(struct intel_cdclk_state *cdclk_state)
 
        vco = cdclk_state->logical.vco;
        if (!vco)
-               vco = dev_priv->skl_preferred_vco_freq;
+               vco = dev_priv->display.cdclk.skl_preferred_vco_freq;
 
        for_each_new_intel_crtc_in_state(state, crtc, crtc_state, i) {
                if (!crtc_state->hw.enable)
@@ -3397,7 +3396,7 @@ void intel_update_max_cdclk(struct drm_i915_private *dev_priv)
                u32 limit = intel_de_read(dev_priv, SKL_DFSM) & SKL_DFSM_CDCLK_LIMIT_MASK;
                int max_cdclk, vco;
 
-               vco = dev_priv->skl_preferred_vco_freq;
+               vco = dev_priv->display.cdclk.skl_preferred_vco_freq;
                drm_WARN_ON(&dev_priv->drm, vco != 8100000 && vco != 8640000);
 
                /*

diff --git a/drivers/gpu/drm/i915/display/intel_display_core.h b/drivers/gpu/drm/i915/display/intel_display_core.h
index 2167dbee5eea7f2aac769b984e92989b4b357db2..b577429ee6e9a42092513bec9ff7245ae2707761 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display_core.h
+++ b/drivers/gpu/drm/i915/display/intel_display_core.h
@@ -345,6 +345,7 @@ struct intel_display {
                struct intel_global_obj obj;
 
                unsigned int max_cdclk_freq;
+               unsigned int skl_preferred_vco_freq;
        } cdclk;
 
        struct {

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index cf52d4adaa204ab7021a1a6d8127f877d0cdafd4..ba3c27c969f277ec210bff38f60d2736a2ee657c 100644 (file)
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -251,7 +251,6 @@ struct drm_i915_private {
        bool preserve_bios_swizzle;
 
        unsigned int fsb_freq, mem_freq, is_ddr3;
-       unsigned int skl_preferred_vco_freq;
 
        unsigned int max_dotclk_freq;
        unsigned int hpll_freq;

diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
index 9785eef2e5a4e6566c452e1fa8c45c447fe00b76..caa1901c126e9303c38c3e343055bcaa3ff77a08 100644 (file)
--- a/drivers/gpu/drm/xe/xe_device_types.h
+++ b/drivers/gpu/drm/xe/xe_device_types.h
@@ -498,10 +498,7 @@ struct xe_device {
        struct mutex sb_lock;
 
        /* Should be in struct intel_display */
-       u32 skl_preferred_vco_freq, max_dotclk_freq, hti_state;
-       u8 snps_phy_failed_calibration;
-       struct drm_atomic_state *modeset_restore_state;
-       struct list_head global_obj_list;
+       u32 max_dotclk_freq;
 
        union {
                /* only to allow build, not used functionally */