Re: [PATCH 6/8] drm/i915/dram: Fix some spelling aroynd the 16Gb DIMM w/a

[Jani Nikula <jani.nikula@linux.intel.com>]

On Tue, 02 Sep 2025, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Use consistent spelling when talking about the 16Gb DIMM w/a.
>
> Even currently language is a bit off as the w/a is actually
> about DIMMs with 16Gb DRAM devices on them, not the total capacity
> of the whole DIMM. But this language does more or less match how
> Bspec talks about this stuff.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Personally, I usually prefer a space between the value and the unit,
i.e. "16 Gb" but that's neither here nor there.

Reviewed-by: Jani Nikula <jani.nikula@intel.com>

> ---
>  drivers/gpu/drm/i915/display/skl_watermark.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c b/drivers/gpu/drm/i915/display/skl_watermark.c
> index ae3ce0d65cfc..0b9c86042e78 100644
> --- a/drivers/gpu/drm/i915/display/skl_watermark.c
> +++ b/drivers/gpu/drm/i915/display/skl_watermark.c
> @@ -3209,9 +3209,9 @@ adjust_wm_latency(struct intel_display *display,
>  	}
>  
>  	/*
> -	 * WA Level-0 adjustment for 16GB DIMMs: SKL+
> +	 * WA Level-0 adjustment for 16Gb DIMMs: SKL+
>  	 * If we could not get dimm info enable this WA to prevent from
> -	 * any underrun. If not able to get Dimm info assume 16GB dimm
> +	 * any underrun. If not able to get DIMM info assume 16Gb DIMM
>  	 * to avoid any underrun.
>  	 */
>  	if (!display->platform.dg2 && dram_info->has_16gb_dimms)

-- 
Jani Nikula, Intel