Re: [PATCH v2 4/9] drm/i915/xe3lpd: Update bandwidth parameters for display version 30.02

[Gustavo Sousa <gustavo.sousa@intel.com>]

When sending a new version of a series, I think it would be good to have
all patches of the sames series tagged with the same version number
(i.e. [PATCH v<n>]), even if not all of them have updates.

By marking only some patches with the version, tools get confused. For
example, I'm unable to easily apply the full series to my local tree
with b4 shazam, as it tells me the series is incomplete.

More importantly, I'm not exactly sure how Patchwork (which I believe is
what ends up triggering CI) behaves in these situations.

--
Gustavo Sousa

Quoting Dnyaneshwar Bhadane (2025-06-06 16:50:59-03:00)
>From: Matt Atwood <matthew.s.atwood@intel.com>
>
>Bandwidth parameters for WCL have been updated with respect to
>previous display releases. Encode them into xe3lpd_3002_sa_info and use
>that new struct.
>
>-v2: Resolve conflict on applying patche.
>
>Bspec: 68859
>Signed-off-by: Matt Atwood <matthew.s.atwood@intel.com>
>Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>
>Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
>---
> drivers/gpu/drm/i915/display/intel_bw.c | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/i915/display/intel_bw.c b/drivers/gpu/drm/i915/display/intel_bw.c
>index 97aef729f7d4..6175081d41a8 100644
>--- a/drivers/gpu/drm/i915/display/intel_bw.c
>+++ b/drivers/gpu/drm/i915/display/intel_bw.c
>@@ -418,6 +418,13 @@ static const struct intel_sa_info xe3lpd_sa_info = {
>         .derating = 10,
> };
> 
>+static const struct intel_sa_info xe3lpd_3002_sa_info = {
>+        .deburst = 32,
>+        .deprogbwlimit = 22, /* GB/s */
>+        .displayrtids = 256,
>+        .derating = 10,
>+};
>+
> static int icl_get_bw_info(struct intel_display *display,
>                            const struct dram_info *dram_info,
>                            const struct intel_sa_info *sa)
>@@ -769,7 +776,9 @@ void intel_bw_init_hw(struct intel_display *display)
>         if (!HAS_DISPLAY(display))
>                 return;
> 
>-        if (DISPLAY_VER(display) >= 30)
>+        if (DISPLAY_VERx100(display) >= 3002)
>+                tgl_get_bw_info(display, dram_info, &xe3lpd_3002_sa_info);
>+        else if (DISPLAY_VER(display) >= 30)
>                 tgl_get_bw_info(display, dram_info, &xe3lpd_sa_info);
>         else if (DISPLAY_VERx100(display) >= 1401 && display->platform.dgfx &&
>                  dram_info->type == INTEL_DRAM_GDDR_ECC)
>-- 
>2.34.1
>