On Thu, Jul 31, 2025 at 05:21:22PM +0300, Jani Nikula wrote:
> Add a more generic intel_mem_freq() function instead of platform
> specific ones. Expose it for future use outside of intel_dram.c.
>
Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com>
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>
> ---
> drivers/gpu/drm/i915/soc/intel_dram.c | 17 ++++++++++++-----
> drivers/gpu/drm/i915/soc/intel_dram.h | 1 +
> 2 files changed, 13 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/soc/intel_dram.c
> b/drivers/gpu/drm/i915/soc/intel_dram.c
> index 6be3618d4885..193e7f71a356 100644
> --- a/drivers/gpu/drm/i915/soc/intel_dram.c
> +++ b/drivers/gpu/drm/i915/soc/intel_dram.c
> @@ -135,16 +135,23 @@ static unsigned int vlv_mem_freq(struct
> drm_i915_private *i915)
> return 0;
> }
>
> -static void detect_mem_freq(struct drm_i915_private *i915)
> +unsigned int intel_mem_freq(struct drm_i915_private *i915)
> {
> if (IS_PINEVIEW(i915))
> - i915->mem_freq = pnv_mem_freq(i915);
> + return pnv_mem_freq(i915);
> else if (GRAPHICS_VER(i915) == 5)
> - i915->mem_freq = ilk_mem_freq(i915);
> + return ilk_mem_freq(i915);
> else if (IS_CHERRYVIEW(i915))
> - i915->mem_freq = chv_mem_freq(i915);
> + return chv_mem_freq(i915);
> else if (IS_VALLEYVIEW(i915))
> - i915->mem_freq = vlv_mem_freq(i915);
> + return vlv_mem_freq(i915);
> + else
> + return 0;
> +}
> +
> +static void detect_mem_freq(struct drm_i915_private *i915)
> +{
> + i915->mem_freq = intel_mem_freq(i915);
>
> if (IS_PINEVIEW(i915))
> i915->is_ddr3 = pnv_is_ddr3(i915);
> diff --git a/drivers/gpu/drm/i915/soc/intel_dram.h
> b/drivers/gpu/drm/i915/soc/intel_dram.h
> index 09a7a581d949..5ba75e279e84 100644
> --- a/drivers/gpu/drm/i915/soc/intel_dram.h
> +++ b/drivers/gpu/drm/i915/soc/intel_dram.h
> @@ -34,6 +34,7 @@ struct dram_info {
> void intel_dram_edram_detect(struct drm_i915_private *i915);
> int intel_dram_detect(struct drm_i915_private *i915);
> unsigned int intel_fsb_freq(struct drm_i915_private *i915);
> +unsigned int intel_mem_freq(struct drm_i915_private *i915);
> const struct dram_info *intel_dram_info(struct drm_device *drm);
>
> #endif /* __INTEL_DRAM_H__ */
> --
> 2.39.5
>
