forked from luck/tmp_suning_uos_patched
drm/vc4: hdmi: Fix PM reference leak in vc4_hdmi_encoder_pre_crtc_co()
[ Upstream commit 5e4322a8b266bc9f5ee7ea4895f661c01dbd7cb3 ] pm_runtime_get_sync will increment pm usage counter even it failed. Forgetting to putting operation will result in reference leak here. Fix it by replacing it with pm_runtime_resume_and_get to keep usage counter balanced. Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Zou Wei <zou_wei@huawei.com> Signed-off-by: Maxime Ripard <maxime@cerno.tech> Link: https://patchwork.freedesktop.org/patch/msgid/1621840854-105978-1-git-send-email-zou_wei@huawei.com Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
48c96d5bac
commit
f06ea024c1
|
@ -627,7 +627,7 @@ static void vc4_hdmi_encoder_pre_crtc_configure(struct drm_encoder *encoder)
|
||||||
unsigned long pixel_rate, hsm_rate;
|
unsigned long pixel_rate, hsm_rate;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
ret = pm_runtime_get_sync(&vc4_hdmi->pdev->dev);
|
ret = pm_runtime_resume_and_get(&vc4_hdmi->pdev->dev);
|
||||||
if (ret < 0) {
|
if (ret < 0) {
|
||||||
DRM_ERROR("Failed to retain power domain: %d\n", ret);
|
DRM_ERROR("Failed to retain power domain: %d\n", ret);
|
||||||
return;
|
return;
|
||||||
|
|
Loading…
Reference in New Issue
Block a user