Skip to content

Pr tech mm drm 1787227724 1 - #1727

Open
quicmahap wants to merge 3 commits into
qualcomm-linux:tech/mm/drmfrom
quicmahap:pr-tech-mm-drm-1787227724-1
Open

Pr tech mm drm 1787227724 1#1727
quicmahap wants to merge 3 commits into
qualcomm-linux:tech/mm/drmfrom
quicmahap:pr-tech-mm-drm-1787227724-1

Conversation

@quicmahap

Copy link
Copy Markdown

No description provided.

dev_pm_opp_set_rate(0) removes the vote specified in required-opps but
does not actually park the clock, making it run without the necessary
power backing. Prevent that from happening when
_dpu_core_perf_get_core_clk_rate() returns 0.

Fixes: 25fdd59 ("drm/msm: Add SDM845 DPU support")
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20260728-topic-dpu_power-v1-1-e7783b859a70@oss.qualcomm.comSigned-off-by: Mahadevan P <mahadevan.p@oss.qualcomm.com>
dev_pm_opp_set_rate(0) removes the vote specified in required-opps but
does not actually park the clock, making it run without the necessary
power backing. Drop the explicit calls to it.

Fixes: c943b49 ("drm/msm/dp: add displayPort driver support")
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20260728-topic-dpu_power-v1-2-e7783b859a70@oss.qualcomm.com
[mahadevan: adapted to this branch - the downstream DP MST series consolidated
 msm_dp_ctrl_off_link_stream() and msm_dp_ctrl_off() into msm_dp_ctrl_off_link(),
 so the vote drop is removed from 3 call sites instead of the 4 upstream. The
 upstream hunk targeting msm_dp_ctrl_off_link_stream() is therefore dropped.]
Signed-off-by: Mahadevan P <mahadevan.p@oss.qualcomm.com>
dev_pm_opp_set_rate(0) removes the vote specified in required-opps but
does not actually park the clock, making it run without the necessary
power backing. Drop the explicit call to it.

Every call site of ops->link_clk_disable() is followed by
pm_runtime_put(), so the power vote will be rescinded if deemed safe.

Fixes: 32d3e0f ("drm/msm: dsi: Use OPP API to set clk/perf state")
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Link: https://lore.kernel.org/r/20260728-topic-dpu_power-v1-3-e7783b859a70@oss.qualcomm.comSigned-off-by: Mahadevan P <mahadevan.p@oss.qualcomm.com>
@qcomlnxci
qcomlnxci requested review from a team, Rajesh Kemisetti (quic-rajeshk) and riteshk-quic and removed request for a team August 20, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants