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

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

Patch Review

**Status: Minor issue — missing return value check.**

The fix correctly adds `devm_drm_bridge_add()` before the `drm_bridge_attach()` call, which resolves the "Missing drm_bridge_add() before attach" warning. The placement at line 108 (after setting `possible_crtcs`, before attaching) is correct.

The commit message is excellent — it carefully documents why the Fixes tag references one commit but a stable backport to pre-v6.17 would need different code, and why the bug hasn't caused real issues despite being present since v6.6.

**Issue: `devm_drm_bridge_add()` returns `int` and can fail** (it calls `devm_add_action_or_reset()` internally), but the return value is not checked:

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

Other callers in the tree check this return value (e.g., `msm/hdmi/hdmi_bridge.c`). This should be:

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

While the failure mode (OOM from `devm_add_action_or_reset`) is unlikely, it's good practice to propagate the error, especially since the function already has `ret` declared and returns errors from other calls on the same path.

---
Generated by Claude Code Patch Reviewer