summaryrefslogtreecommitdiff
path: root/drivers/gpu/drm/exynos/exynos_drm_ipp.c
diff options
context:
space:
mode:
authorAndrzej Hajda <a.hajda@samsung.com>2014-08-28 11:07:34 +0200
committerChanho Park <chanho61.park@samsung.com>2014-11-18 12:00:46 +0900
commit3819e4d918c2cffcf473153119f3fb113e2612c4 (patch)
tree7d28dd212d125aeba8153005ca46ca7bd17be12a /drivers/gpu/drm/exynos/exynos_drm_ipp.c
parent9518f37ecc2b8feabb0cec12790f8d28ba3b3d17 (diff)
downloadlinux-3.10-3819e4d918c2cffcf473153119f3fb113e2612c4.tar.gz
linux-3.10-3819e4d918c2cffcf473153119f3fb113e2612c4.tar.bz2
linux-3.10-3819e4d918c2cffcf473153119f3fb113e2612c4.zip
drm/exynos/ipp: stop hardware before freeing memory
Memory shouldn't be freed when hardware is still running. Change-Id: I2ac15beee14bf3d952a15367bf2f4d1366445eac Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> Reviewed-by: Joonyoung Shim <jy0922.shim@samsung.com> Signed-off-by: Inki Dae <inki.dae@samsung.com>
Diffstat (limited to 'drivers/gpu/drm/exynos/exynos_drm_ipp.c')
-rw-r--r--drivers/gpu/drm/exynos/exynos_drm_ipp.c16
1 files changed, 7 insertions, 9 deletions
diff --git a/drivers/gpu/drm/exynos/exynos_drm_ipp.c b/drivers/gpu/drm/exynos/exynos_drm_ipp.c
index 68b8ae34d19..53262563265 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_ipp.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_ipp.c
@@ -1387,13 +1387,17 @@ static int ipp_stop_property(struct drm_device *drm_dev,
struct drm_exynos_ipp_cmd_node *c_node)
{
struct drm_exynos_ipp_property *property = &c_node->property;
- int ret = 0, i;
+ int i;
DRM_DEBUG_KMS("%s:prop_id[%d]\n", __func__, property->prop_id);
/* put event */
ipp_put_event(c_node, NULL);
+ /* stop operations */
+ if (ippdrv->stop)
+ ippdrv->stop(ippdrv->dev, property->cmd);
+
/* check command */
switch (property->cmd) {
case IPP_CMD_M2M:
@@ -1408,16 +1412,10 @@ static int ipp_stop_property(struct drm_device *drm_dev,
break;
default:
DRM_ERROR("invalid operations.\n");
- ret = -EINVAL;
- goto err_clear;
+ return -EINVAL;
}
-err_clear:
- /* stop operations */
- if (ippdrv->stop)
- ippdrv->stop(ippdrv->dev, property->cmd);
-
- return ret;
+ return 0;
}
void ipp_sched_cmd(struct work_struct *work)