diff options
author | Eric Anholt <eric@anholt.net> | 2017-04-12 15:12:00 -0400 |
---|---|---|
committer | Rob Clark <robdclark@gmail.com> | 2017-05-27 13:48:26 -0400 |
commit | 3c30cc41a87880db86c343b8c9cc8bc2d3d36055 (patch) | |
tree | 7227f99689f4827d2e63ed45be2dd6b637d5e7bb | |
parent | 43523eba79bda8f5b4c27f8ffe20ea078d20113a (diff) |
drm/msm: Reuse dma_fence_release.
If we follow the typical pattern of the base class being the first
member, we can use the default dma_fence_free function.
Signed-off-by: Eric Anholt <eric@anholt.net>
Cc: Rob Clark <robdclark@gmail.com>
Cc: linux-arm-msm@vger.kernel.org
Cc: freedreno@lists.freedesktop.org
Reviewed-by: Rob Clark <robdclark@gmail.com>
Signed-off-by: Rob Clark <robdclark@gmail.com>
-rw-r--r-- | drivers/gpu/drm/msm/msm_fence.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/drivers/gpu/drm/msm/msm_fence.c b/drivers/gpu/drm/msm/msm_fence.c index 3f299c537b77..a2f89bac9c16 100644 --- a/drivers/gpu/drm/msm/msm_fence.c +++ b/drivers/gpu/drm/msm/msm_fence.c | |||
@@ -99,8 +99,8 @@ void msm_update_fence(struct msm_fence_context *fctx, uint32_t fence) | |||
99 | } | 99 | } |
100 | 100 | ||
101 | struct msm_fence { | 101 | struct msm_fence { |
102 | struct msm_fence_context *fctx; | ||
103 | struct dma_fence base; | 102 | struct dma_fence base; |
103 | struct msm_fence_context *fctx; | ||
104 | }; | 104 | }; |
105 | 105 | ||
106 | static inline struct msm_fence *to_msm_fence(struct dma_fence *fence) | 106 | static inline struct msm_fence *to_msm_fence(struct dma_fence *fence) |
@@ -130,19 +130,13 @@ static bool msm_fence_signaled(struct dma_fence *fence) | |||
130 | return fence_completed(f->fctx, f->base.seqno); | 130 | return fence_completed(f->fctx, f->base.seqno); |
131 | } | 131 | } |
132 | 132 | ||
133 | static void msm_fence_release(struct dma_fence *fence) | ||
134 | { | ||
135 | struct msm_fence *f = to_msm_fence(fence); | ||
136 | kfree_rcu(f, base.rcu); | ||
137 | } | ||
138 | |||
139 | static const struct dma_fence_ops msm_fence_ops = { | 133 | static const struct dma_fence_ops msm_fence_ops = { |
140 | .get_driver_name = msm_fence_get_driver_name, | 134 | .get_driver_name = msm_fence_get_driver_name, |
141 | .get_timeline_name = msm_fence_get_timeline_name, | 135 | .get_timeline_name = msm_fence_get_timeline_name, |
142 | .enable_signaling = msm_fence_enable_signaling, | 136 | .enable_signaling = msm_fence_enable_signaling, |
143 | .signaled = msm_fence_signaled, | 137 | .signaled = msm_fence_signaled, |
144 | .wait = dma_fence_default_wait, | 138 | .wait = dma_fence_default_wait, |
145 | .release = msm_fence_release, | 139 | .release = dma_fence_free, |
146 | }; | 140 | }; |
147 | 141 | ||
148 | struct dma_fence * | 142 | struct dma_fence * |