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@xxxxxxx> > > Suggested-by: Christian König <christian.koenig@xxxxxxx> > --- [...] > 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@xxxxxxxxxxxxxx> _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
