mbox series

[v5,00/10] drm/msm/dpu: cleanup dpu_core_perf module

Message ID 20230730010102.350713-1-dmitry.baryshkov@linaro.org (mailing list archive)
Headers show
Series drm/msm/dpu: cleanup dpu_core_perf module | expand

Message

Dmitry Baryshkov July 30, 2023, 1 a.m. UTC
Apply several cleanups to the DPU's core_perf module.

Changes since v4:
- Dropped the 'extract bandwidth aggregation function' (Abhinav)
- Fixed commit message for the patch 9 (Abhinav)

Changes since v3:
- Dropped avg_bw type change (Abhinav)
- Removed core_perf from the commit cubject (Abhinav)

Changes since v2:
- Dropped perf tuning patches for now (Abhinav)
- Restored kms variable assignment in dpu_core_perf_crtc_release_bw
  (LKP)
- Fixed description for the last patch (Abhinav)

Changes since v1:
- Reworked overrides for the perf parameters instead of completely
  dropping them. Abhinav described why these overrides are useful.
- Moved max clock rate determination to dpu_kms.c

Dmitry Baryshkov (10):
  drm/msm/dpu: drop enum dpu_core_perf_data_bus_id
  drm/msm/dpu: bail from _dpu_core_perf_crtc_update_bus if there are no
    ICC paths
  drm/msm/dpu: drop separate dpu_core_perf_tune overrides
  drm/msm/dpu: rework indentation in dpu_core_perf
  drm/msm/dpu: drop the dpu_core_perf_crtc_update()'s stop_req param
  drm/msm/dpu: use dpu_perf_cfg in DPU core_perf code
  drm/msm/dpu: remove unused fields from struct dpu_core_perf
  drm/msm/dpu: remove extra clk_round_rate() call
  drm/msm/dpu: move max clock decision to dpu_kms.
  drm/msm/dpu: drop dpu_core_perf_destroy()

 drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 154 +++++++-----------
 drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h |  48 +-----
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c      |   6 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c       |  14 +-
 4 files changed, 77 insertions(+), 145 deletions(-)

Comments

Dmitry Baryshkov Aug. 2, 2023, 9:55 a.m. UTC | #1
On Sun, 30 Jul 2023 04:00:52 +0300, Dmitry Baryshkov wrote:
> Apply several cleanups to the DPU's core_perf module.
> 
> Changes since v4:
> - Dropped the 'extract bandwidth aggregation function' (Abhinav)
> - Fixed commit message for the patch 9 (Abhinav)
> 
> Changes since v3:
> - Dropped avg_bw type change (Abhinav)
> - Removed core_perf from the commit cubject (Abhinav)
> 
> [...]

Applied, thanks!

[01/10] drm/msm/dpu: drop enum dpu_core_perf_data_bus_id
        https://gitlab.freedesktop.org/lumag/msm/-/commit/f15de4f6245b
[02/10] drm/msm/dpu: bail from _dpu_core_perf_crtc_update_bus if there are no ICC paths
        https://gitlab.freedesktop.org/lumag/msm/-/commit/808c92df654a
[03/10] drm/msm/dpu: drop separate dpu_core_perf_tune overrides
        https://gitlab.freedesktop.org/lumag/msm/-/commit/6a4bc73915af
[04/10] drm/msm/dpu: rework indentation in dpu_core_perf
        https://gitlab.freedesktop.org/lumag/msm/-/commit/a6239e65c801
[05/10] drm/msm/dpu: drop the dpu_core_perf_crtc_update()'s stop_req param
        https://gitlab.freedesktop.org/lumag/msm/-/commit/716f0d4cac0f
[06/10] drm/msm/dpu: use dpu_perf_cfg in DPU core_perf code
        https://gitlab.freedesktop.org/lumag/msm/-/commit/779f336ed4b8
[07/10] drm/msm/dpu: remove unused fields from struct dpu_core_perf
        https://gitlab.freedesktop.org/lumag/msm/-/commit/7a735940293d
[08/10] drm/msm/dpu: remove extra clk_round_rate() call
        https://gitlab.freedesktop.org/lumag/msm/-/commit/d64d83806aa4
[09/10] drm/msm/dpu: move max clock decision to dpu_kms.
        https://gitlab.freedesktop.org/lumag/msm/-/commit/4161ec7e93d1
[10/10] drm/msm/dpu: drop dpu_core_perf_destroy()
        https://gitlab.freedesktop.org/lumag/msm/-/commit/3d5199a173d0

Best regards,