diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-01-08 19:08:10 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-01-08 19:08:10 -0500 |
commit | ca5c8a4c2aba577b4cd2f4c3c72a768a80830294 (patch) | |
tree | 78e48d1a5cdd25f4e20c3e5c84000e122c9f25e1 /drivers/gpu/drm/exynos/exynos_drm_ipp.c | |
parent | ed2c8911684ac780d051d251ad0cd9d797dd029c (diff) | |
parent | e8e89622ed361c46bf90ba4828e685a8b603f7e5 (diff) |
Merge branch 'drm-fixes' of git://people.freedesktop.org/~airlied/linux
Pull drm update from Dave Airlie:
"Exynos and Radeon mostly, with a dma-buf and ttm fix thrown in.
It's a bit big but its mostly exynos license fix ups and I'd rather
not hold those up since its legally stuff.
Radeon has a couple of fixes from dma engine work, TTM is just a
locking fix, and dma-buf fix has been hanging around and I finally got
a chance to review it."
* 'drm-fixes' of git://people.freedesktop.org/~airlied/linux: (30 commits)
drm/ttm: fix fence locking in ttm_buffer_object_transfer
drm/prime: drop reference on imported dma-buf come from gem
drm/radeon: add quirk for d3 delay during switcheroo poweron for apple macbooks
drm/exynos: move finish page flip to a common place
drm/exynos: fimd: modify condition in fimd resume
drm/radeon: fix DMA CS parser for r6xx linear copy packet
drm/radeon: split r6xx and r7xx copy_dma functions
drm/exynos: Use devm_clk_get in exynos_drm_gsc.c
drm/exynos: Remove redundant NULL check in exynos_drm_gsc.c
drm/exynos: Remove explicit freeing using devm_* APIs in exynos_drm_gsc.c
drm/exynos: Use devm_clk_get in exynos_drm_rotator.c
drm/exynos: Remove redundant NULL check in exynos_drm_rotator.c
drm/exynos: Remove unnecessary devm_* freeing APIs in exynos_drm_rotator.c
drm/exynos: Use devm_clk_get in exynos_drm_fimc.c
drm/exynos: Remove redundant NULL check
drm/exynos: Remove explicit freeing using devm_* APIs in exynos_drm_fimc.c
drm/exynos: Use devm_kzalloc in exynos_drm_ipp.c
drm/exynos: fix gem buffer allocation type checking
drm/exynos: remove needless parenthesis.
drm/exynos: fix incorrect interrupt induced by m2m operation.
...
Diffstat (limited to 'drivers/gpu/drm/exynos/exynos_drm_ipp.c')
-rw-r--r-- | drivers/gpu/drm/exynos/exynos_drm_ipp.c | 22 |
1 files changed, 6 insertions, 16 deletions
diff --git a/drivers/gpu/drm/exynos/exynos_drm_ipp.c b/drivers/gpu/drm/exynos/exynos_drm_ipp.c index c1f12301224..0bda96454a0 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_ipp.c +++ b/drivers/gpu/drm/exynos/exynos_drm_ipp.c | |||
@@ -27,7 +27,7 @@ | |||
27 | #include "exynos_drm_iommu.h" | 27 | #include "exynos_drm_iommu.h" |
28 | 28 | ||
29 | /* | 29 | /* |
30 | * IPP is stand for Image Post Processing and | 30 | * IPP stands for Image Post Processing and |
31 | * supports image scaler/rotator and input/output DMA operations. | 31 | * supports image scaler/rotator and input/output DMA operations. |
32 | * using FIMC, GSC, Rotator, so on. | 32 | * using FIMC, GSC, Rotator, so on. |
33 | * IPP is integration device driver of same attribute h/w | 33 | * IPP is integration device driver of same attribute h/w |
@@ -1292,7 +1292,7 @@ static int ipp_start_property(struct exynos_drm_ippdrv *ippdrv, | |||
1292 | DRM_DEBUG_KMS("%s:prop_id[%d]\n", __func__, property->prop_id); | 1292 | DRM_DEBUG_KMS("%s:prop_id[%d]\n", __func__, property->prop_id); |
1293 | 1293 | ||
1294 | /* store command info in ippdrv */ | 1294 | /* store command info in ippdrv */ |
1295 | ippdrv->cmd = c_node; | 1295 | ippdrv->c_node = c_node; |
1296 | 1296 | ||
1297 | if (!ipp_check_mem_list(c_node)) { | 1297 | if (!ipp_check_mem_list(c_node)) { |
1298 | DRM_DEBUG_KMS("%s:empty memory.\n", __func__); | 1298 | DRM_DEBUG_KMS("%s:empty memory.\n", __func__); |
@@ -1303,7 +1303,7 @@ static int ipp_start_property(struct exynos_drm_ippdrv *ippdrv, | |||
1303 | ret = ipp_set_property(ippdrv, property); | 1303 | ret = ipp_set_property(ippdrv, property); |
1304 | if (ret) { | 1304 | if (ret) { |
1305 | DRM_ERROR("failed to set property.\n"); | 1305 | DRM_ERROR("failed to set property.\n"); |
1306 | ippdrv->cmd = NULL; | 1306 | ippdrv->c_node = NULL; |
1307 | return ret; | 1307 | return ret; |
1308 | } | 1308 | } |
1309 | 1309 | ||
@@ -1487,11 +1487,6 @@ void ipp_sched_cmd(struct work_struct *work) | |||
1487 | mutex_lock(&c_node->cmd_lock); | 1487 | mutex_lock(&c_node->cmd_lock); |
1488 | 1488 | ||
1489 | property = &c_node->property; | 1489 | property = &c_node->property; |
1490 | if (!property) { | ||
1491 | DRM_ERROR("failed to get property:prop_id[%d]\n", | ||
1492 | c_node->property.prop_id); | ||
1493 | goto err_unlock; | ||
1494 | } | ||
1495 | 1490 | ||
1496 | switch (cmd_work->ctrl) { | 1491 | switch (cmd_work->ctrl) { |
1497 | case IPP_CTRL_PLAY: | 1492 | case IPP_CTRL_PLAY: |
@@ -1704,7 +1699,7 @@ void ipp_sched_event(struct work_struct *work) | |||
1704 | return; | 1699 | return; |
1705 | } | 1700 | } |
1706 | 1701 | ||
1707 | c_node = ippdrv->cmd; | 1702 | c_node = ippdrv->c_node; |
1708 | if (!c_node) { | 1703 | if (!c_node) { |
1709 | DRM_ERROR("failed to get command node.\n"); | 1704 | DRM_ERROR("failed to get command node.\n"); |
1710 | return; | 1705 | return; |
@@ -1895,7 +1890,7 @@ static int ipp_probe(struct platform_device *pdev) | |||
1895 | struct exynos_drm_subdrv *subdrv; | 1890 | struct exynos_drm_subdrv *subdrv; |
1896 | int ret; | 1891 | int ret; |
1897 | 1892 | ||
1898 | ctx = kzalloc(sizeof(*ctx), GFP_KERNEL); | 1893 | ctx = devm_kzalloc(&pdev->dev, sizeof(*ctx), GFP_KERNEL); |
1899 | if (!ctx) | 1894 | if (!ctx) |
1900 | return -ENOMEM; | 1895 | return -ENOMEM; |
1901 | 1896 | ||
@@ -1916,8 +1911,7 @@ static int ipp_probe(struct platform_device *pdev) | |||
1916 | ctx->event_workq = create_singlethread_workqueue("ipp_event"); | 1911 | ctx->event_workq = create_singlethread_workqueue("ipp_event"); |
1917 | if (!ctx->event_workq) { | 1912 | if (!ctx->event_workq) { |
1918 | dev_err(dev, "failed to create event workqueue\n"); | 1913 | dev_err(dev, "failed to create event workqueue\n"); |
1919 | ret = -EINVAL; | 1914 | return -EINVAL; |
1920 | goto err_clear; | ||
1921 | } | 1915 | } |
1922 | 1916 | ||
1923 | /* | 1917 | /* |
@@ -1958,8 +1952,6 @@ err_cmd_workq: | |||
1958 | destroy_workqueue(ctx->cmd_workq); | 1952 | destroy_workqueue(ctx->cmd_workq); |
1959 | err_event_workq: | 1953 | err_event_workq: |
1960 | destroy_workqueue(ctx->event_workq); | 1954 | destroy_workqueue(ctx->event_workq); |
1961 | err_clear: | ||
1962 | kfree(ctx); | ||
1963 | return ret; | 1955 | return ret; |
1964 | } | 1956 | } |
1965 | 1957 | ||
@@ -1985,8 +1977,6 @@ static int ipp_remove(struct platform_device *pdev) | |||
1985 | destroy_workqueue(ctx->cmd_workq); | 1977 | destroy_workqueue(ctx->cmd_workq); |
1986 | destroy_workqueue(ctx->event_workq); | 1978 | destroy_workqueue(ctx->event_workq); |
1987 | 1979 | ||
1988 | kfree(ctx); | ||
1989 | |||
1990 | return 0; | 1980 | return 0; |
1991 | } | 1981 | } |
1992 | 1982 | ||