From: Matt Atwood <[email protected]>

Bandwidth parameters for Xe3p_LPD are the same as for Xe3_LPD. Re-use
them.

Since handling for Xe3_LPD version 30.02 is more like a special case,
let's use a "== 3002" check for it inside the ">= 30" branch instead of
adding a new branch for version 35.  That allows us to re-use the ">=
30" branch for Xe3p_LPD.

v2:
  - Do not have a special case for ecc_impacting_de_bw, since there are
    no specific instructions in Bspec for this scenario. (Matt Roper)
v3:
  - Re-use the ">= 30" branch in the if-ladder. (Matt Roper)

Bspec: 68859
Signed-off-by: Matt Atwood <[email protected]>
Reviewed-by: Matt Roper <[email protected]>
Link: 
https://patch.msgid.link/[email protected]
Signed-off-by: Gustavo Sousa <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_bw.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_bw.c 
b/drivers/gpu/drm/i915/display/intel_bw.c
index bf37d7a9732e..919b25a5fbac 100644
--- a/drivers/gpu/drm/i915/display/intel_bw.c
+++ b/drivers/gpu/drm/i915/display/intel_bw.c
@@ -805,10 +805,11 @@ void intel_bw_init_hw(struct intel_display *display)
        if (!HAS_DISPLAY(display))
                return;
 
-       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);
+       if (DISPLAY_VER(display) >= 30) {
+               if (DISPLAY_VERx100(display) == 3002)
+                       tgl_get_bw_info(display, dram_info, 
&xe3lpd_3002_sa_info);
+               else
+                       tgl_get_bw_info(display, dram_info, &xe3lpd_sa_info);
        } else if (DISPLAY_VERx100(display) >= 1401 && display->platform.dgfx) {
                if (dram_info->type == INTEL_DRAM_GDDR_ECC)
                        xe2_hpd_get_bw_info(display, dram_info, 
&xe2_hpd_ecc_sa_info);
-- 
2.51.0

Reply via email to