From: Ville Syrjälä Date: Tue, 2 Sep 2025 13:31:12 +0000 (+0300) Subject: drm/i915/dram: Don't call skl_get_dram_info()/skl_get_dram_type() on icl X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=a6d31e18c01211b041d416661b9ccfd75475e01b;p=users%2Fhch%2Fmisc.git drm/i915/dram: Don't call skl_get_dram_info()/skl_get_dram_type() on icl Currently the icl codepaths first determine the memory type from the memory controller registers (via skl_get_dram_info()->skl_get_dram_type()) and then overwrite the results with icl_pcode_read_mem_global_info(). Get rid of the pointless (and potentially incorrect) skl_get_dram_type() stuff. Signed-off-by: Ville Syrjälä Link: https://patchwork.freedesktop.org/patch/msgid/20250902133113.18778-8-ville.syrjala@linux.intel.com Reviewed-by: Jani Nikula --- diff --git a/drivers/gpu/drm/i915/soc/intel_dram.c b/drivers/gpu/drm/i915/soc/intel_dram.c index bf3ba874f8c5..38b7dd20b18d 100644 --- a/drivers/gpu/drm/i915/soc/intel_dram.c +++ b/drivers/gpu/drm/i915/soc/intel_dram.c @@ -669,8 +669,9 @@ static int icl_pcode_read_mem_global_info(struct drm_i915_private *dev_priv, static int gen11_get_dram_info(struct drm_i915_private *i915, struct dram_info *dram_info) { - int ret = skl_get_dram_info(i915, dram_info); + int ret; + ret = skl_dram_get_channels_info(i915, dram_info); if (ret) return ret;