Am Dienstag, den 05.06.2018, 13:02 -0400 schrieb Andrey Grodzovsky:
Everything in the flush code path (i.e. waiting for SW queue to become empty) names with *_flush() and everything in the release code path names *_fini()
This patch also effect the amdgpu and etnaviv drivers which use those functions.
Signed-off-by: Andrey Grodzovsky andrey.grodzovsky@amd.com Suggested-by: Christian König christian.koenig@amd.com
[...]
diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c b/drivers/gpu/drm/etnaviv/etnaviv_drv.c index 23e73c2..3dff4d0 100644 --- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c +++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c @@ -140,7 +140,7 @@ static void etnaviv_postclose(struct drm_device *dev, struct drm_file *file)
gpu->lastctx = NULL; mutex_unlock(&gpu->lock);
drm_sched_entity_fini(&gpu->sched,
drm_sched_entity_destroy(&gpu->sched,&ctx->sched_entity[i]);
Style nit: this disaligns the second row of parameters to the opening parenthesis where it was previously aligned. I would prefer if the second line is also changed to keep the alignment.
Acked-by: Lucas Stach l.stach@pengutronix.de