Claude review: drm/panel: tdo-tl070wsh30: Use devm_drm_panel_add()

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

Patch Review

**Build failure — function does not exist**

The core change:
```c
-	drm_panel_add(&tdo_tl070wsh30->base);
-
-	return 0;
+	return devm_drm_panel_add(dev, &tdo_tl070wsh30->base);
```

`devm_drm_panel_add()` is not declared in `include/drm/drm_panel.h` and is not defined anywhere in `drivers/gpu/drm/drm_panel.c` or elsewhere. This will fail at compile time with an implicit function declaration error. The existing API provides:
- `drm_panel_add(struct drm_panel *)` — void, non-managed
- `drm_panel_remove(struct drm_panel *)` — the matching cleanup
- `__devm_drm_panel_alloc()` — managed *allocation*, not registration

The correct approach to achieve managed panel registration would be one of:
1. Use `devm_add_action_or_reset()` to register a callback that calls `drm_panel_remove()` on cleanup, creating an ad-hoc devm wrapper.
2. Propose a new `devm_drm_panel_add()` helper in drm_panel.c/drm_panel.h as a preparatory patch, then use it here.

**Remove-side cleanup is correct but moot**

The removal of `drm_panel_remove()` from `tdo_tl070wsh30_panel_remove()` and the now-unused local variable is the right cleanup *if* the devm registration existed, but without it this just creates a leak where the panel is never unregistered.

**Fixes tag correctness**: The identified bug (panel remains registered if `mipi_dsi_attach()` fails) is real and the Fixes tag `cf40c6600592` appears correct, but the fix itself is broken.

---
Generated by Claude Code Patch Reviewer