Claude review: drm/atomic: Call complete_signaling only if prepare_signaling is done

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

> -	unsigned int i, j, num_fences;
> +	unsigned int i, j, num_fences = 0;

> -	complete_signaling(dev, state, fence_state, num_fences, !ret);
> +	if (num_fences)
> +		complete_signaling(dev, state, fence_state, num_fences, !ret);

This change is correct in isolation. Before this series, `complete_signaling` was always called after `prepare_signaling`. With the state allocation moved earlier (patch 4), early `goto out` paths can reach the cleanup before `prepare_signaling` runs. Initializing `num_fences = 0` and guarding the call is the right approach.

I verified that `complete_signaling` cleans up both fence state and crtc events. When `prepare_signaling` was never called, there are no events or fences to clean up, so skipping the call when `num_fences == 0` is safe. The events and fences are only created together in `prepare_signaling`, and `num_fences` is incremented for each one.

No issues found in this patch.

---
Generated by Claude Code Patch Reviewer