Claude review: drm/bridge: analogix_dp: Convert status check functions to bool

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

Patch Review

**Correctness — OK.** The key semantic question is whether the inversion is handled correctly:

- **Original:** `return 0` means "clock/stream IS on", callers test `if (func() == 0)` to enter the success path.
- **New:** `return true` means "clock/stream IS on", callers test `if (func())` to enter the success path.

Both call sites are correct after the change:

```c
// analogix_dp_core.c, first loop (slave video stream clock):
-		if (analogix_dp_is_slave_video_stream_clock_on(dp) == 0)
+		if (analogix_dp_is_slave_video_stream_clock_on(dp))
 			break;
```

```c
// analogix_dp_core.c, second loop (video stream on):
-		if (analogix_dp_is_video_stream_on(dp) == 0) {
+		if (analogix_dp_is_video_stream_on(dp)) {
 			done_count++;
```

Both loops preserve identical behavior — the `else if (done_count)` branch in the second loop is also unaffected since the boolean false case maps exactly to the old non-zero case.

**Scope — OK.** Both functions are static-linkage internal to the analogix_dp driver. There are no other callers anywhere in the tree. The header, implementation, and call sites are all updated consistently.

**No issues found.** This is a clean, correct cleanup patch.

---
Generated by Claude Code Patch Reviewer