Some of the parameters of used in display bandwidth calculations are tied to the platform and are orthogonal to the display IP. After talking with the hardware team, we now have the information (and Bspec has been updated) that the members deprogbwlimit and derating of struct intel_sa_info are such platform-specific ones.
With that, we are now able to make the driver code more aligned with the hardware by splitting structs intel_sa_info into two different structs: one that is platform-specific and another that is display-IP-specific. That change also allows us to simplify how we select the parameters for the calculation. Signed-off-by: Gustavo Sousa <[email protected]> --- Changes in v4: - Minor updates to fix issues captured by CI; see changelog in individual patches for details. - Link to v3: https://patch.msgid.link/20260514-separate-platform-from-diplay-ip-specific-bw-params-v3-0-68727d6fe...@intel.com Changes in v3: - Incorporated review feedback; see each individual patch for details. - Link to v2: https://patch.msgid.link/20260511-separate-platform-from-diplay-ip-specific-bw-params-v2-0-e762cb866...@intel.com Changes in v2: - Incorporated review feedback; see each individual patch for details. - Link to v1: https://patch.msgid.link/20260408-separate-platform-from-diplay-ip-specific-bw-params-v1-0-23c53afa7...@intel.com --- Gustavo Sousa (5): drm/i915/bw: Don't call intel_dram_info() too early drm/i915/bw: Extract platform-specific parameters drm/i915/bw: Deduplicate intel_sa_info instances drm/i915/bw: Rename struct intel_sa_info to intel_display_bw_params drm/i915/bw: Extract get_display_bw_params() drivers/gpu/drm/i915/display/intel_bw.c | 205 ++++++++++++++++++++------------ 1 file changed, 128 insertions(+), 77 deletions(-) --- base-commit: f05be6b9858836632ce6b4839e1bda3a470278b9 change-id: 20260408-separate-platform-from-diplay-ip-specific-bw-params-65bfba0603be Best regards, -- Gustavo Sousa <[email protected]>
