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.grodzov...@amd.com> > > Suggested-by: Christian König <christian.koe...@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.st...@pengutronix.de> _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
