Claude review: drm/rockchip: vop2: Set correct output format for RK3576 YUV422

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

Patch Review

The nested `if/switch` in `vop2_crtc_atomic_enable`:

```c
else if (vcstate->output_mode == ROCKCHIP_OUT_MODE_YUV422 &&
         vop2->version == VOP_VERSION_RK3576)
    switch (vcstate->output_type) {
    ...
    default:
        drm_err(vop2->drm, "Unknown DRM_MODE_CONNECTOR %d\n",
            vcstate->output_type);
        vop2_unlock(vop2);
        return;
    }
```

The early return in the default case after `vop2_unlock()` is correct for error handling but leaves the CRTC in a partially-enabled state. This is a hardware enable path, so returning mid-way could leave things inconsistent. An `else` after the if-else chain isn't braced, which combines with the switch to make the control flow hard to follow. This could use braces around the switch for clarity.

---
Generated by Claude Code Patch Reviewer