[Intel-gfx] [PATCH v3 1/2] drm/i915: Make sandybridge_pcode_read() deal with the second data register

Sripada, Radhakrishna radhakrishna.sripada at intel.com
Wed May 8 20:49:37 UTC 2019 

On Fri, 2019-05-03 at 22:08 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> The pcode mailbox has two data registers. So far we've only ever used
> the one, but that's about to change. Expose the second data register
> to
> the callers of sandybridge_pcode_read().
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
LGTM
Reviewed-by: Radhakrishna Sripada <radhakrishna.sripada at intel.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c   |  4 ++--
>  drivers/gpu/drm/i915/intel_pm.c       | 12 +++++++-----
>  drivers/gpu/drm/i915/intel_sideband.c | 15 +++++++++------
>  drivers/gpu/drm/i915/intel_sideband.h |  3 ++-
>  4 files changed, 20 insertions(+), 14 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
> b/drivers/gpu/drm/i915/i915_debugfs.c
> index 14cd83e9ea8b..203088f6f269 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -1494,7 +1494,7 @@ static int gen6_drpc_info(struct seq_file *m)
>  
>  	if (INTEL_GEN(dev_priv) <= 7)
>  		sandybridge_pcode_read(dev_priv,
> GEN6_PCODE_READ_RC6VIDS,
> -				       &rc6vids);
> +				       &rc6vids, NULL);
>  
>  	seq_printf(m, "RC1e Enabled: %s\n",
>  		   yesno(rcctl1 & GEN6_RC_CTL_RC1e_ENABLE));
> @@ -1777,7 +1777,7 @@ static int i915_ring_freq_table(struct seq_file
> *m, void *unused)
>  		ia_freq = gpu_freq;
>  		sandybridge_pcode_read(dev_priv,
>  				       GEN6_PCODE_READ_MIN_FREQ_TABLE,
> -				       &ia_freq);
> +				       &ia_freq, NULL);
>  		seq_printf(m, "%d\t\t%d\t\t\t\t%d\n",
>  			   intel_gpu_freq(dev_priv, (gpu_freq *
>  						     (IS_GEN9_BC(dev_pr
> iv) ||
> diff --git a/drivers/gpu/drm/i915/intel_pm.c
> b/drivers/gpu/drm/i915/intel_pm.c
> index ef9fc77f8162..b043a96e123c 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -2822,7 +2822,7 @@ static void intel_read_wm_latency(struct
> drm_i915_private *dev_priv,
>  		val = 0; /* data0 to be programmed to 0 for first set
> */
>  		ret = sandybridge_pcode_read(dev_priv,
>  					     GEN9_PCODE_READ_MEM_LATENC
> Y,
> -					     &val);
> +					     &val, NULL);
>  
>  		if (ret) {
>  			DRM_ERROR("SKL Mailbox read error = %d\n",
> ret);
> @@ -2841,7 +2841,7 @@ static void intel_read_wm_latency(struct
> drm_i915_private *dev_priv,
>  		val = 1; /* data0 to be programmed to 1 for second set
> */
>  		ret = sandybridge_pcode_read(dev_priv,
>  					     GEN9_PCODE_READ_MEM_LATENC
> Y,
> -					     &val);
> +					     &val, NULL);
>  		if (ret) {
>  			DRM_ERROR("SKL Mailbox read error = %d\n",
> ret);
>  			return;
> @@ -7061,7 +7061,7 @@ static void gen6_init_rps_frequencies(struct
> drm_i915_private *dev_priv)
>  
>  		if (sandybridge_pcode_read(dev_priv,
>  					   HSW_PCODE_DYNAMIC_DUTY_CYCLE
> _CONTROL,
> -					   &ddcc_status) == 0)
> +					   &ddcc_status, NULL) == 0)
>  			rps->efficient_freq =
>  				clamp_t(u8,
>  					((ddcc_status >> 8) & 0xff),
> @@ -7408,7 +7408,8 @@ static void gen6_enable_rc6(struct
> drm_i915_private *dev_priv)
>  		   GEN6_RC_CTL_HW_ENABLE);
>  
>  	rc6vids = 0;
> -	ret = sandybridge_pcode_read(dev_priv, GEN6_PCODE_READ_RC6VIDS,
> &rc6vids);
> +	ret = sandybridge_pcode_read(dev_priv, GEN6_PCODE_READ_RC6VIDS,
> +				     &rc6vids, NULL);
>  	if (IS_GEN(dev_priv, 6) && ret) {
>  		DRM_DEBUG_DRIVER("Couldn't check for BIOS
> workaround\n");
>  	} else if (IS_GEN(dev_priv, 6) && (GEN6_DECODE_RC6_VID(rc6vids
> & 0xff) < 450)) {
> @@ -8555,7 +8556,8 @@ void intel_init_gt_powersave(struct
> drm_i915_private *dev_priv)
>  	    IS_IVYBRIDGE(dev_priv) || IS_HASWELL(dev_priv)) {
>  		u32 params = 0;
>  
> -		sandybridge_pcode_read(dev_priv, GEN6_READ_OC_PARAMS,
> &params);
> +		sandybridge_pcode_read(dev_priv, GEN6_READ_OC_PARAMS,
> +				       &params, NULL);
>  		if (params & BIT(31)) { /* OC supported */
>  			DRM_DEBUG_DRIVER("Overclocking supported, max:
> %dMHz, overclock: %dMHz\n",
>  					 (rps->max_freq & 0xff) * 50,
> diff --git a/drivers/gpu/drm/i915/intel_sideband.c
> b/drivers/gpu/drm/i915/intel_sideband.c
> index 87b5a14c7ca8..a115625e980c 100644
> --- a/drivers/gpu/drm/i915/intel_sideband.c
> +++ b/drivers/gpu/drm/i915/intel_sideband.c
> @@ -374,7 +374,7 @@ static inline int gen7_check_mailbox_status(u32
> mbox)
>  }
>  
>  static int __sandybridge_pcode_rw(struct drm_i915_private *i915,
> -				  u32 mbox, u32 *val,
> +				  u32 mbox, u32 *val, u32 *val1,
>  				  int fast_timeout_us,
>  				  int slow_timeout_ms,
>  				  bool is_read)
> @@ -393,7 +393,7 @@ static int __sandybridge_pcode_rw(struct
> drm_i915_private *i915,
>  		return -EAGAIN;
>  
>  	intel_uncore_write_fw(uncore, GEN6_PCODE_DATA, *val);
> -	intel_uncore_write_fw(uncore, GEN6_PCODE_DATA1, 0);
> +	intel_uncore_write_fw(uncore, GEN6_PCODE_DATA1, val1 ? *val1 :
> 0);
>  	intel_uncore_write_fw(uncore,
>  			      GEN6_PCODE_MAILBOX, GEN6_PCODE_READY |
> mbox);
>  
> @@ -407,6 +407,8 @@ static int __sandybridge_pcode_rw(struct
> drm_i915_private *i915,
>  
>  	if (is_read)
>  		*val = intel_uncore_read_fw(uncore, GEN6_PCODE_DATA);
> +	if (is_read && val1)
> +		*val1 = intel_uncore_read_fw(uncore, GEN6_PCODE_DATA1);
>  
>  	if (INTEL_GEN(i915) > 6)
>  		return gen7_check_mailbox_status(mbox);
> @@ -414,12 +416,13 @@ static int __sandybridge_pcode_rw(struct
> drm_i915_private *i915,
>  		return gen6_check_mailbox_status(mbox);
>  }
>  
> -int sandybridge_pcode_read(struct drm_i915_private *i915, u32 mbox,
> u32 *val)
> +int sandybridge_pcode_read(struct drm_i915_private *i915, u32 mbox,
> +			   u32 *val, u32 *val1)
>  {
>  	int err;
>  
>  	mutex_lock(&i915->sb_lock);
> -	err = __sandybridge_pcode_rw(i915, mbox, val,
> +	err = __sandybridge_pcode_rw(i915, mbox, val, val1,
>  				     500, 0,
>  				     true);
>  	mutex_unlock(&i915->sb_lock);
> @@ -440,7 +443,7 @@ int sandybridge_pcode_write_timeout(struct
> drm_i915_private *i915,
>  	int err;
>  
>  	mutex_lock(&i915->sb_lock);
> -	err = __sandybridge_pcode_rw(i915, mbox, &val,
> +	err = __sandybridge_pcode_rw(i915, mbox, &val, NULL,
>  				     fast_timeout_us, slow_timeout_ms,
>  				     false);
>  	mutex_unlock(&i915->sb_lock);
> @@ -457,7 +460,7 @@ static bool skl_pcode_try_request(struct
> drm_i915_private *i915, u32 mbox,
>  				  u32 request, u32 reply_mask, u32
> reply,
>  				  u32 *status)
>  {
> -	*status = __sandybridge_pcode_rw(i915, mbox, &request,
> +	*status = __sandybridge_pcode_rw(i915, mbox, &request, NULL,
>  					 500, 0,
>  					 true);
>  
> diff --git a/drivers/gpu/drm/i915/intel_sideband.h
> b/drivers/gpu/drm/i915/intel_sideband.h
> index a0907e2c4992..7fb95745a444 100644
> --- a/drivers/gpu/drm/i915/intel_sideband.h
> +++ b/drivers/gpu/drm/i915/intel_sideband.h
> @@ -127,7 +127,8 @@ u32 intel_sbi_read(struct drm_i915_private *i915,
> u16 reg,
>  void intel_sbi_write(struct drm_i915_private *i915, u16 reg, u32
> value,
>  		     enum intel_sbi_destination destination);
>  
> -int sandybridge_pcode_read(struct drm_i915_private *i915, u32 mbox,
> u32 *val);
> +int sandybridge_pcode_read(struct drm_i915_private *i915, u32 mbox,
> +			   u32 *val, u32 *val1);
>  int sandybridge_pcode_write_timeout(struct drm_i915_private *i915,
> u32 mbox,
>  				    u32 val, int fast_timeout_us,
>  				    int slow_timeout_ms);

More information about the Intel-gfx mailing list