| Message ID | 20250209-dpu-perf-rework-v5-0-87e936cf3004@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm/msm/dpu: rework debugfs interface of dpu_core_perf | expand |
From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> On Sun, 09 Feb 2025 05:21:10 +0200, Dmitry Baryshkov wrote: > Bring back a set of patches extracted from [1] per Abhinav's suggestion. > > Rework debugging overrides for the bandwidth and clock settings. Instead > of specifying the 'mode' and some values, allow one to set the affected > value directly. > > [1] https://patchwork.freedesktop.org/series/119552/#rev2 > > [...] Applied, thanks! [1/8] drm/msm/dpu: extract bandwidth aggregation function https://gitlab.freedesktop.org/lumag/msm/-/commit/b9aedd32a817 [2/8] drm/msm/dpu: remove duplicate code calculating sum of bandwidths https://gitlab.freedesktop.org/lumag/msm/-/commit/795aef6f3653 [3/8] drm/msm/dpu: change ib values to u32 https://gitlab.freedesktop.org/lumag/msm/-/commit/c54b61102f46 [4/8] drm/msm/dpu: make fix_core_ab_vote consistent with fix_core_ib_vote https://gitlab.freedesktop.org/lumag/msm/-/commit/879a7f8c8c2d [5/8] drm/msm/dpu: also use KBps for bw_ctl output https://gitlab.freedesktop.org/lumag/msm/-/commit/f6c782ea7de5 [6/8] drm/msm/dpu: rename average bandwidth-related debugfs files https://gitlab.freedesktop.org/lumag/msm/-/commit/40fc01273f07 [7/8] drm/msm/dpu: drop core_clk_rate overrides from _dpu_core_perf_calc_crtc https://gitlab.freedesktop.org/lumag/msm/-/commit/e7e2495595e0 [8/8] drm/msm/dpu: handle perf mode in _dpu_core_perf_crtc_update_bus() https://gitlab.freedesktop.org/lumag/msm/-/commit/6d3175a72cc0 Best regards,
Bring back a set of patches extracted from [1] per Abhinav's suggestion. Rework debugging overrides for the bandwidth and clock settings. Instead of specifying the 'mode' and some values, allow one to set the affected value directly. [1] https://patchwork.freedesktop.org/series/119552/#rev2 Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- Changes in v5: - Fixed bps -> KBps in the comments (Abhinav) - Split the core_clk_rate changes into a separate commit (Abhinav) - Expanded commit message to note that debugfs bandwidth settings are not a subject to catalog thresholds (Abhinav) - Temporarily dropped the patches actually reworking the debugfs modes, they will be reposted separately. - Link to v4: https://lore.kernel.org/r/20250106-dpu-perf-rework-v4-0-00b248349476@linaro.org Changes in v4: - Dropped core_perf: from patch subject (Abhinav) - Fixed indentation of _dpu_core_perf_crtc_update_bus() (Abhinav) - Expanded commit messages to reflect different questions (Abhinav) - Reworked existing files to specify max allowed average bandwidth (Abhinav) - Fixed u32 vs u64 and KBps vs Bps values in debugfs interface - Link to v3: https://lore.kernel.org/r/20240314-dpu-perf-rework-v3-0-79fa4e065574@linaro.org --- Dmitry Baryshkov (8): drm/msm/dpu: extract bandwidth aggregation function drm/msm/dpu: remove duplicate code calculating sum of bandwidths drm/msm/dpu: change ib values to u32 drm/msm/dpu: make fix_core_ab_vote consistent with fix_core_ib_vote drm/msm/dpu: also use KBps for bw_ctl output drm/msm/dpu: rename average bandwidth-related debugfs files drm/msm/dpu: drop core_clk_rate overrides from _dpu_core_perf_calc_crtc drm/msm/dpu: handle perf mode in _dpu_core_perf_crtc_update_bus() drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 140 ++++++++++++-------------- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h | 10 +- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 5 +- 3 files changed, 71 insertions(+), 84 deletions(-) --- base-commit: ed58d103e6da15a442ff87567898768dc3a66987 change-id: 20240314-dpu-perf-rework-97fca999eb46 Best regards,