[CI 03/17] drm/i915/display: Use braces for if-ladder in intel_bw_init_hw()

[Gustavo Sousa]

Looking at the current if-ladder in intel_bw_init_hw(), we see that
Xe2_HPD contains two entries, differing only for ECC memories.  Let's
improve readability by using braces and allowing adding extra conditions
for each case.

v2:
  - Tweaked commit message, since we are not going to add the ECC case
    for Xe3p_LPD anymore.

Reviewed-by: Matt Roper <matthew.d.ro...@intel.com>
Link: 
https://patch.msgid.link/20251103-xe3p_lpd-basic-enabling-v3-3-00e87b510...@intel.com
Signed-off-by: Gustavo Sousa <gustavo.so...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_bw.c | 29 +++++++++++++------------
 1 file changed, 15 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_bw.c 
b/drivers/gpu/drm/i915/display/intel_bw.c
index f97ccc1a96a7..bf37d7a9732e 100644
--- a/drivers/gpu/drm/i915/display/intel_bw.c
+++ b/drivers/gpu/drm/i915/display/intel_bw.c
@@ -805,29 +805,30 @@ void intel_bw_init_hw(struct intel_display *display)
        if (!HAS_DISPLAY(display))
                return;
 
-       if (DISPLAY_VERx100(display) >= 3002)
+       if (DISPLAY_VERx100(display) >= 3002) {
                tgl_get_bw_info(display, dram_info, &xe3lpd_3002_sa_info);
-       else if (DISPLAY_VER(display) >= 30)
+       } 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)
-               xe2_hpd_get_bw_info(display, dram_info, &xe2_hpd_ecc_sa_info);
-       else if (DISPLAY_VERx100(display) >= 1401 && display->platform.dgfx)
-               xe2_hpd_get_bw_info(display, dram_info, &xe2_hpd_sa_info);
-       else if (DISPLAY_VER(display) >= 14)
+       } 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);
+               else
+                       xe2_hpd_get_bw_info(display, dram_info, 
&xe2_hpd_sa_info);
+       } else if (DISPLAY_VER(display) >= 14) {
                tgl_get_bw_info(display, dram_info, &mtl_sa_info);
-       else if (display->platform.dg2)
+       } else if (display->platform.dg2) {
                dg2_get_bw_info(display);
-       else if (display->platform.alderlake_p)
+       } else if (display->platform.alderlake_p) {
                tgl_get_bw_info(display, dram_info, &adlp_sa_info);
-       else if (display->platform.alderlake_s)
+       } else if (display->platform.alderlake_s) {
                tgl_get_bw_info(display, dram_info, &adls_sa_info);
-       else if (display->platform.rocketlake)
+       } else if (display->platform.rocketlake) {
                tgl_get_bw_info(display, dram_info, &rkl_sa_info);
-       else if (DISPLAY_VER(display) == 12)
+       } else if (DISPLAY_VER(display) == 12) {
                tgl_get_bw_info(display, dram_info, &tgl_sa_info);
-       else if (DISPLAY_VER(display) == 11)
+       } else if (DISPLAY_VER(display) == 11) {
                icl_get_bw_info(display, dram_info, &icl_sa_info);
+       }
 }
 
 static unsigned int intel_bw_num_active_planes(struct intel_display *display,
-- 
2.51.0