Claude review: drm/panel: jdi-fhd-r63452: remove panel on DSI attach failure

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

Patch Review

**Correctness:** The fix is correct. Looking at the probe function:

```c
drm_panel_add(&ctx->panel);          // line 203

ret = mipi_dsi_attach(dsi);          // line 205
if (ret < 0) {
    dev_err(dev, "Failed to attach to DSI host: %d\n", ret);
    return ret;                      // BUG: panel left in global list
}
```

`devm_drm_panel_alloc()` registers a devm action for `drm_panel_put()` but does NOT handle `drm_panel_remove()` — the `drm_panel_add()`/`drm_panel_remove()` lifecycle is the driver's responsibility. The `.remove()` callback at line 214 calls `drm_panel_remove()`, but that path is never taken when probe fails. So the added `drm_panel_remove()` call is the correct fix.

**Commit message:** The commit message clearly explains the problem and solution. However:

- **Missing `Fixes:` tag.** This is a bugfix and should reference the commit that introduced the bug. This is important for stable-kernel backporting. The commit that introduced the driver (or that converted it to `devm_drm_panel_alloc` without adding error-path cleanup) should be cited.

**Style:** The patch is minimal and well-placed — exactly one line in the right location.

**Nit:** The blank line between "If mipi_dsi_attach() fails, the driver returns without calling" and "drm_panel_remove()" in the commit message body (lines 101-102 of the mbox) appears to be an unintentional line break splitting a sentence across paragraphs. It should read as a continuous paragraph.

---
Generated by Claude Code Patch Reviewer