Claude review: drm/panel: visionox-vtdr6130: switch to devm panel calls and drop remove

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

Patch Review

This patch converts `drm_panel_add()` + `mipi_dsi_attach()` to their devm-managed equivalents and removes the now-unnecessary `remove()` callback.

**Probe changes:**
```c
-	drm_panel_add(&ctx->panel);
-
-	ret = mipi_dsi_attach(dsi);
-	if (ret < 0) {
-		dev_err(dev, "Failed to attach to DSI host: %d\n", ret);
-		drm_panel_remove(&ctx->panel);
+	ret = devm_drm_panel_add(dev, &ctx->panel);
+	if (ret)
 		return ret;
-	}
-
-	return 0;
+
+	return devm_mipi_dsi_attach(dev, dsi);
```

Clean conversion. The devm ordering is correct — `devm_drm_panel_add()` is called first, so on teardown the DSI detach (registered second) will happen before the panel removal. This matches the manual teardown order in the old `remove()` callback. The error path simplification (dropping `drm_panel_remove` on attach failure) is correct since devm handles it.

**Remove callback deletion:**
```c
-static void visionox_vtdr6130_remove(struct mipi_dsi_device *dsi)
-{
-	...
-}
```
and
```c
 static struct mipi_dsi_driver visionox_vtdr6130_driver = {
 	.probe = visionox_vtdr6130_probe,
-	.remove = visionox_vtdr6130_remove,
```

Both `devm_drm_panel_add()` and `devm_mipi_dsi_attach()` are available in the tree (confirmed in headers). This is a standard cleanup seen across many panel drivers.

**Verdict:** Correct, clean conversion. No issues.

---
Generated by Claude Code Patch Reviewer