Claude review: media: renesas: vsp1: Split vsp1_du_setup_lif()

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

Patch Review

Splits the dual-purpose `vsp1_du_setup_lif()` (enable when cfg != NULL, disable when cfg == NULL) into two explicit functions: `vsp1_du_enable()` and `vsp1_du_disable()`. A temporary inline compatibility wrapper `vsp1_du_setup_lif()` is added to `include/media/vsp1.h` for existing DRM callers.

The split is clean — the enable path keeps the existing code minus the `if (!cfg)` block, and the disable path is the extracted block. The wrapper correctly dispatches:

```c
static inline int vsp1_du_setup_lif(struct device *dev, unsigned int pipe_index,
                                    const struct vsp1_du_lif_config *cfg)
{
    if (cfg)
        return vsp1_du_enable(dev, pipe_index, cfg);
    else
        return vsp1_du_disable(dev, pipe_index);
}
```

**Minor nit:** The commit message has a typo — "disaple" should be "disable":
> a vsp1_du_disable() function to disaple it.

Otherwise correct.

---
Generated by Claude Code Patch Reviewer