Re: [PATCH v4 0/5] drm/i915/bw: Split bandwidth params into platform- and display-IP-specific structs

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

Gustavo Sousa <gustavo.sousa@intel.com> writes:

> 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 <gustavo.sousa@intel.com>

Pushed to drm-intel-next. Thanks everyone for the feedback and reviews!

--
Gustavo Sousa

> ---
> 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-68727d6fe3ec@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-e762cb8662da@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-23c53afa7db0@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 <gustavo.sousa@intel.com>