Commit 2628009d authored by Philipp Stanner's avatar Philipp Stanner Committed by Danilo Krummrich
Browse files

drm/nouveau: Simplify nouveau_fence_done() 



nouveau_fence_done() contains an if branch that checks whether a
nouveau_fence has either of the two existing nouveau_fence backend ops,
which will always evaluate to true.

Remove the surplus check.

Signed-off-by: default avatarPhilipp Stanner <phasta@kernel.org>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Link: https://lore.kernel.org/r/20250424130254.42046-5-phasta@kernel.org


Signed-off-by: default avatarDanilo Krummrich <dakr@kernel.org>
parent 2c0ddff2

drivers/gpu/drm/nouveau/nouveau_fence.c

+11 −13
Original line number Diff line number Diff line
@@ -249,8 +249,6 @@ nouveau_fence_emit(struct nouveau_fence *fence) 
bool

nouveau_fence_done(struct nouveau_fence *fence)

{

	if (fence->base.ops == &nouveau_fence_ops_legacy ||

	    fence->base.ops == &nouveau_fence_ops_uevent) {

	struct nouveau_fence_chan *fctx = nouveau_fctx(fence);

	struct nouveau_channel *chan;

	unsigned long flags;
@@ -263,7 +261,7 @@ nouveau_fence_done(struct nouveau_fence *fence) 
	if (chan)

		nouveau_fence_update(chan, fctx);

	spin_unlock_irqrestore(&fctx->lock, flags);

	}



	return dma_fence_is_signaled(&fence->base);

}