diff options
| author | Dinghao Liu <dinghao.liu@zju.edu.cn> | 2020-05-21 08:27:45 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2020-06-23 13:18:10 +0200 |
| commit | 7dcfb327f8fd7418124fc6d63bdce91b5844f50a (patch) | |
| tree | 5e077dd42203f6660ea4968f950437d466831e51 /drivers/staging/media/tegra-vde/vde.c | |
| parent | 0c8bc934ee304e43daa04492bced568dd0bde594 (diff) | |
| download | linux-7dcfb327f8fd7418124fc6d63bdce91b5844f50a.tar.bz2 | |
media: staging: tegra-vde: fix runtime pm imbalance on error
pm_runtime_get_sync() increments the runtime PM usage counter even
the call returns an error code. Thus a pairing decrement is needed
on the error handling path to keep the counter balanced.
Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/staging/media/tegra-vde/vde.c')
| -rw-r--r-- | drivers/staging/media/tegra-vde/vde.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/staging/media/tegra-vde/vde.c b/drivers/staging/media/tegra-vde/vde.c index d3e63512a765..3fdf2cd0b99e 100644 --- a/drivers/staging/media/tegra-vde/vde.c +++ b/drivers/staging/media/tegra-vde/vde.c @@ -777,7 +777,7 @@ static int tegra_vde_ioctl_decode_h264(struct tegra_vde *vde, ret = pm_runtime_get_sync(dev); if (ret < 0) - goto unlock; + goto put_runtime_pm; /* * We rely on the VDE registers reset value, otherwise VDE @@ -843,8 +843,6 @@ static int tegra_vde_ioctl_decode_h264(struct tegra_vde *vde, put_runtime_pm: pm_runtime_mark_last_busy(dev); pm_runtime_put_autosuspend(dev); - -unlock: mutex_unlock(&vde->lock); release_dpb_frames: |