diff options
| author |   Christian König <christian.koenig@amd.com> | 2018-11-14 14:24:27 +0100 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2018-12-05 11:01:11 +0100 |
| commit | 0b258ed1a219a9776e8f6967eb34837ae0332e64 (patch) | |
| tree | ea88b7eb50ff2bad95db20360c600b33d7627343 /drivers/gpu/drm/v3d | |
| parent | drm/tinydrm: Use kmemdup rather than duplicating its implementation in repaper_spi_transfer() (diff) | |
| download | linux-dev-0b258ed1a219a9776e8f6967eb34837ae0332e64.tar.xz linux-dev-0b258ed1a219a9776e8f6967eb34837ae0332e64.zip | |
drm: revert "expand replace_fence to support timeline point v2"
This reverts commit 9a09a42369a4a37a959c051d8e1a1f948c1529a4.
The whole interface isn't thought through. Since this function can't
fail we actually can't allocate an object to store the sync point.
Sorry, I should have taken the lead on this from the very beginning and
reviewed it more thoughtfully. Going to propose a new interface as a
follow up change.
Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Link: https://patchwork.freedesktop.org/patch/265580/
Diffstat (limited to 'drivers/gpu/drm/v3d')
| -rw-r--r-- | drivers/gpu/drm/v3d/v3d_gem.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/v3d/v3d_gem.c b/drivers/gpu/drm/v3d/v3d_gem.c index f565b197cba9..cb99e53f7607 100644 --- a/drivers/gpu/drm/v3d/v3d_gem.c +++ b/drivers/gpu/drm/v3d/v3d_gem.c @@ -609,8 +609,7 @@ v3d_submit_cl_ioctl(struct drm_device *dev, void *data, /* Update the return sync object for the */ sync_out = drm_syncobj_find(file_priv, args->out_sync); if (sync_out) { - drm_syncobj_replace_fence(sync_out, 0, - exec->render_done_fence); + drm_syncobj_replace_fence(sync_out, exec->render_done_fence); drm_syncobj_put(sync_out); } |