Claude review: drm/bridge: synopsys: dw-dp: fix bool function returning negative error code

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

Patch Review

**Bug analysis:** Correct. The function signature is:
```c
static bool dw_dp_video_need_vsc_sdp(struct dw_dp *dp)
```
The old code had:
```c
state = dw_dp_get_bridge_state(dp);
if (!state)
    return -EINVAL;
```
`-EINVAL` is `-22`, which when cast to `bool` becomes `true`. This would cause callers at lines 1163 and 1388 to incorrectly set `DP_MSA_MISC_COLOR_VSC_SDP` or take the VSC SDP code path when the bridge state couldn't be retrieved — the opposite of the intended "bail out" behavior.

**Fix assessment:** The change to `return false` is correct. When `dw_dp_get_bridge_state()` fails, the safe default is to not require a VSC SDP. Both callers treat `true` as "do extra VSC SDP work", so `false` correctly skips that path on error.

**Minor nits (non-blocking):**
- The Fixes tag formatting has a minor style issue: `Fixes: 86eecc3a9c2e("drm/bridge:...` should have a space before the parenthesis per kernel convention: `Fixes: 86eecc3a9c2e ("drm/bridge:...`
- The commit message is missing a period at the end of the description paragraph ("the opposite of the intended behavior" → "the opposite of the intended behavior.").

**Reviewed-by worthy:** Yes — this is a clear, minimal, and correct bug fix.

---
Generated by Claude Code Patch Reviewer