drm/msm: adreno: add support for DDR bandwidth scaling via GMU
The Adreno GMU Management Unit (GMU) can also vote for DDR Bandwidth
along the Frequency and Power Domain level, but by default we leave the
OPP core scale the interconnect ddr path.
While scaling the interconnect path was sufficient, newer GPUs
like the A750 requires specific vote parameters and bandwidth to
achieve full functionnality.
In order to get the vote values to be used by the GPU Management
Unit (GMU), we need to parse all the possible OPP Bandwidths and
create a vote value to be send to the appropriate Bus Control
Modules (BCMs) declared in the GPU info struct.
The added dev_pm_opp_get_bw() is used in this case.
The vote array will then be used to dynamically generate the GMU
bw_table sent during the GMU power-up.
Those entries will then be used by passing the appropriate
bandwidth level when voting for a GPU frequency.
This will make sure all resources are equally voted for a
same OPP, whatever decision is done by the GMU, it will
ensure all resources votes are synchronized.
Ran full vulkan-cts-1.3.7.3-0-gd71a36db16d98313c431829432a136dbda692a08 with mesa 25.0.0+git3ecf2a0518 on:
- QRD8550
- QRD8650
- HDK8650
Patchset is based on current msm-next including preemption support.
Any feedback is welcome.
To: Akhil P Oommen <quic_akhilpo@quicinc.com>
To: Viresh Kumar <vireshk@kernel.org>
To: Nishanth Menon <nm@ti.com>
To: Stephen Boyd <sboyd@kernel.org>
To: Rafael J. Wysocki <rafael@kernel.org>
To: Rob Clark <robdclark@gmail.com>
To: Sean Paul <sean@poorly.run>
To: Konrad Dybcio <konradybcio@kernel.org>
To: Abhinav Kumar <quic_abhinavk@quicinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Marijn Suijten <marijn.suijten@somainline.org>
To: David Airlie <airlied@gmail.com>
To: Simona Vetter <simona@ffwll.ch>
To: Bjorn Andersson <andersson@kernel.org>
To: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzk+dt@kernel.org>
To: Conor Dooley <conor+dt@kernel.org>
Cc: Connor Abbott <cwabbott0@gmail.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
Changes in v2:
- opp: rename to dev_pm_opp_get_bw, fix commit message and kerneldoc
- remove quirks that are features and move them to a dedicated .features bitfield
- get icc bcm kerneldoc, and simplify/cleanup a6xx_gmu_rpmh_bw_votes_init()
- no more copies of data
- take calculations from icc-rpmh/bcm-voter
- move into a single cleaner function
- fix a6xx_gmu_set_freq() but not calling dev_pm_opp_set_opp() if !bw_index
- also vote for maximum bus bandwidth usage (AB)
- overall fix typos in commit messages
- Link to v1: https://lore.kernel.org/r/20241113-topic-sm8x50-gpu-bw-vote-v1-0-3b8d39737a9b@linaro.org
---
Neil Armstrong (11):
opp: core: implement dev_pm_opp_get_bw
drm/msm: adreno: rename quirks that are features
drm/msm: adreno: move features bits in a separate variable
drm/msm: adreno: add GMU_BW_VOTE feature flag
drm/msm: adreno: add plumbing to generate bandwidth vote table for GMU
drm/msm: adreno: dynamically generate GMU bw table
drm/msm: adreno: find bandwidth index of OPP and set it along freq index
drm/msm: adreno: request for maximum bus bandwidth usage
drm/msm: adreno: enable GMU bandwidth for A740 and A750
arm64: qcom: dts: sm8550: add interconnect and opp-peak-kBps for GPU
arm64: qcom: dts: sm8650: add interconnect and opp-peak-kBps for GPU
arch/arm64/boot/dts/qcom/sm8550.dtsi | 11 ++
arch/arm64/boot/dts/qcom/sm8650.dtsi | 14 +++
drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 84 ++++++++-----
drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 181 ++++++++++++++++++++++++++++-
drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 16 ++-
drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 4 +-
drivers/gpu/drm/msm/adreno/a6xx_gpu.h | 1 +
drivers/gpu/drm/msm/adreno/a6xx_hfi.c | 45 ++++++-
drivers/gpu/drm/msm/adreno/a6xx_hfi.h | 5 +
drivers/gpu/drm/msm/adreno/adreno_device.c | 2 +-
drivers/gpu/drm/msm/adreno/adreno_gpu.h | 9 +-
drivers/opp/core.c | 25 ++++
include/linux/pm_opp.h | 7 ++
13 files changed, 358 insertions(+), 46 deletions(-)
---
base-commit: 86313a9cd152330c634b25d826a281c6a002eb77
change-id: 20241113-topic-sm8x50-gpu-bw-vote-f5e022fe7a47
Best regards,