Skip to content
Snippets Groups Projects
Commit a94c6700 authored by Naseer Ahmed's avatar Naseer Ahmed
Browse files

msm: mdss: mdp: secure buffer is unmapped without iommu attached


For secure buffers, it is a requirement that IOMMU is attached
before unmapping can be called, otherwise the scm call to unmap the
buffer will not be successful. In the current implementation it is
possible to unmap buffer while going into suspend without IOMMU being
attached which leads to secure playback failures in subsequent sessions.
Fix this by ensuring IOMMU is attached before buffer is freed/unmapped.

Change-Id: I29cdce834356b45f05fc75ec07d1c6fa0fcf2d69
Signed-off-by: default avatarSiva Yarravarapu <sivay@codeaurora.org>
Signed-off-by: default avatarNaseer Ahmed <naseer@codeaurora.org>
parent 6faf483d
No related merge requests found
......@@ -717,9 +717,10 @@ void mdss_mdp_data_free(struct mdss_mdp_data *data)
{
int i;
mdss_iommu_ctrl(1);
for (i = 0; i < data->num_planes && data->p[i].len; i++)
mdss_mdp_put_img(&data->p[i]);
mdss_iommu_ctrl(0);
data->num_planes = 0;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment