Claude review: drm/tidss: Fix missing drm_bridge_attach() call

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

Patch Review

**One issue:** The subject line says "Fix missing drm_bridge_attach() call" but the actual fix adds a `drm_bridge_add()` call. The subject should read **"Fix missing drm_bridge_add() call"**. The body text is correct.

**Missing error check:** `devm_drm_bridge_add()` returns `int` (0 on success, negative errno on failure). The patch does not check the return value:

```c
+	devm_drm_bridge_add(tidss->dev, &t_enc->bridge);
```

This should be:

```c
	ret = devm_drm_bridge_add(tidss->dev, &t_enc->bridge);
	if (ret)
		return ret;
```

The `ret` variable is already declared and available in scope at `tidss_encoder.c:91`.

The placement of the call (after `devm_drm_bridge_alloc()` and encoder init, but before `drm_bridge_attach()`) is correct — the bridge must be added to the global list before it can be attached.

The Fixes tag, Cc: stable annotation, and the thorough explanation of why the fix is only needed for v6.17+ are all well done.

---
Generated by Claude Code Patch Reviewer