diff options
author | Harry Wentland <harry.wentland@amd.com> | 2017-09-27 10:53:50 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-09-28 16:46:15 -0400 |
commit | 2004f45ef83f07f43f5da6ede780b08068c7583d (patch) | |
tree | 011ac2087c8a8c9272a4c4c5eaacd69d44c2319a /drivers/gpu/drm/amd/display/modules/freesync/freesync.c | |
parent | 82b400a62f2fd42b87f91a298c5641d0ead99251 (diff) |
drm/amd/display: Use kernel alloc/free
Abstractions are frowned upon.
cocci script:
virtual context
virtual patch
virtual org
virtual report
@@
expression ptr;
@@
- dm_alloc(ptr)
+ kzalloc(ptr, GFP_KERNEL)
@@
expression ptr, size;
@@
- dm_realloc(ptr, size)
+ krealloc(ptr, size, GFP_KERNEL)
@@
expression ptr;
@@
- dm_free(ptr)
+ kfree(ptr)
v2: use GFP_KERNEL, not GFP_ATOMIC. add cocci script
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/modules/freesync/freesync.c')
-rw-r--r-- | drivers/gpu/drm/amd/display/modules/freesync/freesync.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c index 52350d0e68d0..4d7db4aa28e0 100644 --- a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c +++ b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c | |||
@@ -143,7 +143,7 @@ static bool check_dc_support(const struct dc *dc) | |||
143 | struct mod_freesync *mod_freesync_create(struct dc *dc) | 143 | struct mod_freesync *mod_freesync_create(struct dc *dc) |
144 | { | 144 | { |
145 | struct core_freesync *core_freesync = | 145 | struct core_freesync *core_freesync = |
146 | dm_alloc(sizeof(struct core_freesync)); | 146 | kzalloc(sizeof(struct core_freesync), GFP_KERNEL); |
147 | 147 | ||
148 | 148 | ||
149 | struct persistent_data_flag flag; | 149 | struct persistent_data_flag flag; |
@@ -153,8 +153,8 @@ struct mod_freesync *mod_freesync_create(struct dc *dc) | |||
153 | if (core_freesync == NULL) | 153 | if (core_freesync == NULL) |
154 | goto fail_alloc_context; | 154 | goto fail_alloc_context; |
155 | 155 | ||
156 | core_freesync->map = dm_alloc(sizeof(struct freesync_entity) * | 156 | core_freesync->map = kzalloc(sizeof(struct freesync_entity) * MOD_FREESYNC_MAX_CONCURRENT_STREAMS, |
157 | MOD_FREESYNC_MAX_CONCURRENT_STREAMS); | 157 | GFP_KERNEL); |
158 | 158 | ||
159 | if (core_freesync->map == NULL) | 159 | if (core_freesync->map == NULL) |
160 | goto fail_alloc_map; | 160 | goto fail_alloc_map; |
@@ -197,10 +197,10 @@ struct mod_freesync *mod_freesync_create(struct dc *dc) | |||
197 | return &core_freesync->public; | 197 | return &core_freesync->public; |
198 | 198 | ||
199 | fail_construct: | 199 | fail_construct: |
200 | dm_free(core_freesync->map); | 200 | kfree(core_freesync->map); |
201 | 201 | ||
202 | fail_alloc_map: | 202 | fail_alloc_map: |
203 | dm_free(core_freesync); | 203 | kfree(core_freesync); |
204 | 204 | ||
205 | fail_alloc_context: | 205 | fail_alloc_context: |
206 | return NULL; | 206 | return NULL; |
@@ -217,9 +217,9 @@ void mod_freesync_destroy(struct mod_freesync *mod_freesync) | |||
217 | if (core_freesync->map[i].stream) | 217 | if (core_freesync->map[i].stream) |
218 | dc_stream_release(core_freesync->map[i].stream); | 218 | dc_stream_release(core_freesync->map[i].stream); |
219 | 219 | ||
220 | dm_free(core_freesync->map); | 220 | kfree(core_freesync->map); |
221 | 221 | ||
222 | dm_free(core_freesync); | 222 | kfree(core_freesync); |
223 | } | 223 | } |
224 | } | 224 | } |
225 | 225 | ||