Commit e7dc33f3 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Store rawclk_freq in dev_priv

Generalize rawclk handling by storing it in dev_priv.

Presumably our hrawclk readout works at least for CTG and ELK
since we've been using it for DP AUX on those platforms. There
are no real docs anymore after configdb vanished, so the only
reference is the public CTG GMCH spec. What bits are listed in
that doc match our code. The ELK GMCH spec have no relevant
details unfortunately.

The PNV situation is less clear. Starting from
commit aa17cdb4 ("drm/i915: initialize backlight max from VBT")
we assume that the CTG/ELK hrawclk readout works for PNV as well.
At least the results *seem* reasonable for one PNV machine (Lenovo
Ideapad S10-3t). Sadly the PNV GMCH spec doesn't have the goods on
the relevant register either.

So let's keep assuming it works for PNV,ELK,CTG and read it out on
those platforms. G33 also has hrawclk according to some notes
in BSpec, but we don't actually need it for anything, so let's not
even try to read it out there.

v2: Rebase due to IS_VALLYVIEW vs. IS_CHERRYVIEW split
    Use KHz() all over, and kill off a few useless temp variables
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1456932138-14004-2-git-send-email-ville.syrjala@linux.intel.comReviewed-by: default avatarJani Nikula <jani.nikula@intel.com>
parent 5ba00178
...@@ -1836,6 +1836,7 @@ struct drm_i915_private { ...@@ -1836,6 +1836,7 @@ struct drm_i915_private {
unsigned int skl_boot_cdclk; unsigned int skl_boot_cdclk;
unsigned int cdclk_freq, max_cdclk_freq, atomic_cdclk_freq; unsigned int cdclk_freq, max_cdclk_freq, atomic_cdclk_freq;
unsigned int max_dotclk_freq; unsigned int max_dotclk_freq;
unsigned int rawclk_freq;
unsigned int hpll_freq; unsigned int hpll_freq;
unsigned int czclk_freq; unsigned int czclk_freq;
......
...@@ -169,49 +169,61 @@ static int vlv_get_cck_clock_hpll(struct drm_i915_private *dev_priv, ...@@ -169,49 +169,61 @@ static int vlv_get_cck_clock_hpll(struct drm_i915_private *dev_priv,
return DIV_ROUND_CLOSEST(dev_priv->hpll_freq << 1, divider + 1); return DIV_ROUND_CLOSEST(dev_priv->hpll_freq << 1, divider + 1);
} }
int static int
intel_pch_rawclk(struct drm_device *dev) intel_pch_rawclk(struct drm_i915_private *dev_priv)
{ {
struct drm_i915_private *dev_priv = dev->dev_private; return (I915_READ(PCH_RAWCLK_FREQ) & RAWCLK_FREQ_MASK) * 1000;
}
WARN_ON(!HAS_PCH_SPLIT(dev));
return I915_READ(PCH_RAWCLK_FREQ) & RAWCLK_FREQ_MASK; static int
intel_vlv_hrawclk(struct drm_i915_private *dev_priv)
{
return 200000;
} }
/* hrawclock is 1/4 the FSB frequency */ static int
int intel_hrawclk(struct drm_device *dev) intel_g4x_hrawclk(struct drm_i915_private *dev_priv)
{ {
struct drm_i915_private *dev_priv = dev->dev_private;
uint32_t clkcfg; uint32_t clkcfg;
/* There is no CLKCFG reg in Valleyview. VLV hrawclk is 200 MHz */ /* hrawclock is 1/4 the FSB frequency */
if (IS_VALLEYVIEW(dev) || IS_CHERRYVIEW(dev))
return 200;
clkcfg = I915_READ(CLKCFG); clkcfg = I915_READ(CLKCFG);
switch (clkcfg & CLKCFG_FSB_MASK) { switch (clkcfg & CLKCFG_FSB_MASK) {
case CLKCFG_FSB_400: case CLKCFG_FSB_400:
return 100; return 100000;
case CLKCFG_FSB_533: case CLKCFG_FSB_533:
return 133; return 133333;
case CLKCFG_FSB_667: case CLKCFG_FSB_667:
return 166; return 166667;
case CLKCFG_FSB_800: case CLKCFG_FSB_800:
return 200; return 200000;
case CLKCFG_FSB_1067: case CLKCFG_FSB_1067:
return 266; return 266667;
case CLKCFG_FSB_1333: case CLKCFG_FSB_1333:
return 333; return 333333;
/* these two are just a guess; one of them might be right */ /* these two are just a guess; one of them might be right */
case CLKCFG_FSB_1600: case CLKCFG_FSB_1600:
case CLKCFG_FSB_1600_ALT: case CLKCFG_FSB_1600_ALT:
return 400; return 400000;
default: default:
return 133; return 133333;
} }
} }
static void intel_update_rawclk(struct drm_i915_private *dev_priv)
{
if (HAS_PCH_SPLIT(dev_priv))
dev_priv->rawclk_freq = intel_pch_rawclk(dev_priv);
else if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv))
dev_priv->rawclk_freq = intel_vlv_hrawclk(dev_priv);
else if (IS_G4X(dev_priv) || IS_PINEVIEW(dev_priv))
dev_priv->rawclk_freq = intel_g4x_hrawclk(dev_priv);
else
return; /* no rawclk on other platforms, or no need to know it */
DRM_DEBUG_DRIVER("rawclk rate: %d kHz\n", dev_priv->rawclk_freq);
}
static void intel_update_czclk(struct drm_i915_private *dev_priv) static void intel_update_czclk(struct drm_i915_private *dev_priv)
{ {
if (!(IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv))) if (!(IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)))
...@@ -15617,6 +15629,7 @@ void intel_modeset_init(struct drm_device *dev) ...@@ -15617,6 +15629,7 @@ void intel_modeset_init(struct drm_device *dev)
} }
intel_update_czclk(dev_priv); intel_update_czclk(dev_priv);
intel_update_rawclk(dev_priv);
intel_update_cdclk(dev); intel_update_cdclk(dev);
intel_shared_dpll_init(dev); intel_shared_dpll_init(dev);
......
...@@ -674,13 +674,13 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp, bool has_aux_irq) ...@@ -674,13 +674,13 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp, bool has_aux_irq)
static uint32_t i9xx_get_aux_clock_divider(struct intel_dp *intel_dp, int index) static uint32_t i9xx_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
{ {
struct intel_digital_port *intel_dig_port = dp_to_dig_port(intel_dp); struct intel_digital_port *intel_dig_port = dp_to_dig_port(intel_dp);
struct drm_device *dev = intel_dig_port->base.base.dev; struct drm_i915_private *dev_priv = to_i915(intel_dig_port->base.base.dev);
/* /*
* The clock divider is based off the hrawclk, and would like to run at * The clock divider is based off the hrawclk, and would like to run at
* 2MHz. So, take the hrawclk value and divide by 2 and use that * 2MHz. So, take the hrawclk value and divide by 2 and use that
*/ */
return index ? 0 : DIV_ROUND_CLOSEST(intel_hrawclk(dev), 2); return index ? 0 : DIV_ROUND_CLOSEST(dev_priv->rawclk_freq, 2000);
} }
static uint32_t ilk_get_aux_clock_divider(struct intel_dp *intel_dp, int index) static uint32_t ilk_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
...@@ -692,12 +692,10 @@ static uint32_t ilk_get_aux_clock_divider(struct intel_dp *intel_dp, int index) ...@@ -692,12 +692,10 @@ static uint32_t ilk_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
if (index) if (index)
return 0; return 0;
if (intel_dig_port->port == PORT_A) { if (intel_dig_port->port == PORT_A)
return DIV_ROUND_CLOSEST(dev_priv->cdclk_freq, 2000); return DIV_ROUND_CLOSEST(dev_priv->cdclk_freq, 2000);
else
} else { return DIV_ROUND_CLOSEST(dev_priv->rawclk_freq, 2000);
return DIV_ROUND_CLOSEST(intel_pch_rawclk(dev), 2);
}
} }
static uint32_t hsw_get_aux_clock_divider(struct intel_dp *intel_dp, int index) static uint32_t hsw_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
...@@ -718,7 +716,7 @@ static uint32_t hsw_get_aux_clock_divider(struct intel_dp *intel_dp, int index) ...@@ -718,7 +716,7 @@ static uint32_t hsw_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
default: return 0; default: return 0;
} }
} else { } else {
return index ? 0 : DIV_ROUND_CLOSEST(intel_pch_rawclk(dev), 2); return index ? 0 : DIV_ROUND_CLOSEST(dev_priv->rawclk_freq, 2000);
} }
} }
...@@ -5268,7 +5266,7 @@ intel_dp_init_panel_power_sequencer_registers(struct drm_device *dev, ...@@ -5268,7 +5266,7 @@ intel_dp_init_panel_power_sequencer_registers(struct drm_device *dev,
{ {
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
u32 pp_on, pp_off, pp_div, port_sel = 0; u32 pp_on, pp_off, pp_div, port_sel = 0;
int div = HAS_PCH_SPLIT(dev) ? intel_pch_rawclk(dev) : intel_hrawclk(dev); int div = dev_priv->rawclk_freq / 1000;
i915_reg_t pp_on_reg, pp_off_reg, pp_div_reg, pp_ctrl_reg; i915_reg_t pp_on_reg, pp_off_reg, pp_div_reg, pp_ctrl_reg;
enum port port = dp_to_dig_port(intel_dp)->port; enum port port = dp_to_dig_port(intel_dp)->port;
const struct edp_power_seq *seq = &intel_dp->pps_delays; const struct edp_power_seq *seq = &intel_dp->pps_delays;
......
...@@ -1119,8 +1119,6 @@ void i915_audio_component_cleanup(struct drm_i915_private *dev_priv); ...@@ -1119,8 +1119,6 @@ void i915_audio_component_cleanup(struct drm_i915_private *dev_priv);
extern const struct drm_plane_funcs intel_plane_funcs; extern const struct drm_plane_funcs intel_plane_funcs;
unsigned int intel_rotation_info_size(const struct intel_rotation_info *rot_info); unsigned int intel_rotation_info_size(const struct intel_rotation_info *rot_info);
bool intel_has_pending_fb_unpin(struct drm_device *dev); bool intel_has_pending_fb_unpin(struct drm_device *dev);
int intel_pch_rawclk(struct drm_device *dev);
int intel_hrawclk(struct drm_device *dev);
void intel_mark_busy(struct drm_device *dev); void intel_mark_busy(struct drm_device *dev);
void intel_mark_idle(struct drm_device *dev); void intel_mark_idle(struct drm_device *dev);
void intel_crtc_restore_mode(struct drm_crtc *crtc); void intel_crtc_restore_mode(struct drm_crtc *crtc);
......
...@@ -1251,16 +1251,14 @@ static u32 bxt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) ...@@ -1251,16 +1251,14 @@ static u32 bxt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
static u32 spt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) static u32 spt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
{ {
struct drm_i915_private *dev_priv = to_i915(connector->base.dev); struct drm_i915_private *dev_priv = to_i915(connector->base.dev);
u32 mul, clock; u32 mul;
if (I915_READ(SOUTH_CHICKEN1) & SPT_PWM_GRANULARITY) if (I915_READ(SOUTH_CHICKEN1) & SPT_PWM_GRANULARITY)
mul = 128; mul = 128;
else else
mul = 16; mul = 16;
clock = MHz(24); return MHz(24) / (pwm_freq_hz * mul);
return clock / (pwm_freq_hz * mul);
} }
/* /*
...@@ -1292,10 +1290,9 @@ static u32 lpt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) ...@@ -1292,10 +1290,9 @@ static u32 lpt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
*/ */
static u32 pch_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) static u32 pch_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
{ {
struct drm_device *dev = connector->base.dev; struct drm_i915_private *dev_priv = to_i915(connector->base.dev);
int clock = MHz(intel_pch_rawclk(dev));
return clock / (pwm_freq_hz * 128); return KHz(dev_priv->rawclk_freq) / (pwm_freq_hz * 128);
} }
/* /*
...@@ -1308,14 +1305,13 @@ static u32 pch_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) ...@@ -1308,14 +1305,13 @@ static u32 pch_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
*/ */
static u32 i9xx_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) static u32 i9xx_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
{ {
struct drm_device *dev = connector->base.dev; struct drm_i915_private *dev_priv = to_i915(connector->base.dev);
struct drm_i915_private *dev_priv = dev->dev_private;
int clock; int clock;
if (IS_PINEVIEW(dev)) if (IS_PINEVIEW(dev_priv))
clock = MHz(intel_hrawclk(dev)); clock = KHz(dev_priv->rawclk_freq);
else else
clock = 1000 * dev_priv->cdclk_freq; clock = KHz(dev_priv->cdclk_freq);
return clock / (pwm_freq_hz * 32); return clock / (pwm_freq_hz * 32);
} }
...@@ -1332,9 +1328,9 @@ static u32 i965_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) ...@@ -1332,9 +1328,9 @@ static u32 i965_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
int clock; int clock;
if (IS_G4X(dev_priv)) if (IS_G4X(dev_priv))
clock = MHz(intel_hrawclk(dev)); clock = KHz(dev_priv->rawclk_freq);
else else
clock = 1000 * dev_priv->cdclk_freq; clock = KHz(dev_priv->cdclk_freq);
return clock / (pwm_freq_hz * 128); return clock / (pwm_freq_hz * 128);
} }
...@@ -1346,19 +1342,21 @@ static u32 i965_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) ...@@ -1346,19 +1342,21 @@ static u32 i965_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
*/ */
static u32 vlv_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz) static u32 vlv_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
{ {
struct drm_device *dev = connector->base.dev; struct drm_i915_private *dev_priv = to_i915(connector->base.dev);
struct drm_i915_private *dev_priv = dev->dev_private; int mul, clock;
int clock;
if ((I915_READ(CBR1_VLV) & CBR_PWM_CLOCK_MUX_SELECT) == 0) { if ((I915_READ(CBR1_VLV) & CBR_PWM_CLOCK_MUX_SELECT) == 0) {
if (IS_CHERRYVIEW(dev)) if (IS_CHERRYVIEW(dev_priv))
return KHz(19200) / (pwm_freq_hz * 16); clock = KHz(19200);
else else
return MHz(25) / (pwm_freq_hz * 16); clock = MHz(25);
mul = 16;
} else { } else {
clock = intel_hrawclk(dev); clock = KHz(dev_priv->rawclk_freq);
return MHz(clock) / (pwm_freq_hz * 128); mul = 128;
} }
return clock / (pwm_freq_hz * mul);
} }
static u32 get_backlight_max_vbt(struct intel_connector *connector) static u32 get_backlight_max_vbt(struct intel_connector *connector)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment