[Intel-gfx] [PATCH v2 39/50] drm/i915/dg2: Don't read DRAM info

[Srivatsa, Anusha]

> -----Original Message-----
> From: Roper, Matthew D <matthew.d.roper at intel.com>
> Sent: Tuesday, July 13, 2021 8:15 PM
> To: intel-gfx at lists.freedesktop.org
> Cc: Roper, Matthew D <matthew.d.roper at intel.com>; Srivatsa, Anusha
> <anusha.srivatsa at intel.com>
> Subject: [PATCH v2 39/50] drm/i915/dg2: Don't read DRAM info
> 
> DG2 does not use system DRAM information for BW_BUDDY programming or
> watermark workarounds, so there's no need to read this out at startup.
> 
> Cc: Anusha Srivatsa <anusha.srivatsa at intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
Reviewed-by: Anusha Srivatsa <anusha.srivatsa at intel.com>

> ---
>  drivers/gpu/drm/i915/intel_dram.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_dram.c
> b/drivers/gpu/drm/i915/intel_dram.c
> index 879b0f007be3..9675bb94b70b 100644
> --- a/drivers/gpu/drm/i915/intel_dram.c
> +++ b/drivers/gpu/drm/i915/intel_dram.c
> @@ -494,15 +494,15 @@ void intel_dram_detect(struct drm_i915_private
> *i915)
>  	struct dram_info *dram_info = &i915->dram_info;
>  	int ret;
> 
> +	if (GRAPHICS_VER(i915) < 9 || IS_DG2(i915) || !HAS_DISPLAY(i915))
> +		return;
> +
>  	/*
>  	 * Assume level 0 watermark latency adjustment is needed until
> proven
>  	 * otherwise, this w/a is not needed by bxt/glk.
>  	 */
>  	dram_info->wm_lv_0_adjust_needed = !IS_GEN9_LP(i915);
> 
> -	if (GRAPHICS_VER(i915) < 9 || !HAS_DISPLAY(i915))
> -		return;
> -
>  	if (GRAPHICS_VER(i915) >= 12)
>  		ret = gen12_get_dram_info(i915);
>  	else if (GRAPHICS_VER(i915) >= 11)
> --
> 2.25.4