Claude review: drm/panel: feiyang-fy07024di26a30d: return display-on error

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

Patch Review

**Correctness: Good.**

The original code:
```c
mipi_dsi_dcs_set_display_on(ctx->dsi);

return 0;
```
discards the return value from `mipi_dsi_dcs_set_display_on()`, unconditionally returning success. The replacement:
```c
return mipi_dsi_dcs_set_display_on(ctx->dsi);
```
correctly propagates any error to the caller.

**Consistency with the rest of the driver: Good.** The sibling function `feiyang_disable()` at line 110 already does exactly this pattern with `mipi_dsi_dcs_set_display_off()`:
```c
static int feiyang_disable(struct drm_panel *panel)
{
	struct feiyang *ctx = panel_to_feiyang(panel);

	return mipi_dsi_dcs_set_display_off(ctx->dsi);
}
```
So this patch makes `_enable` consistent with `_disable`.

**Minor observation (not blocking):** The `feiyang_unprepare()` function (line 113) takes a different approach — it checks the return value of `mipi_dsi_dcs_set_display_off()` and logs an error with `dev_err()` before continuing. One could argue that logging the error in `feiyang_enable()` before returning it would aid debugging, but the `_disable` function doesn't log either, so the patch as written is consistent with the existing pattern for the enable/disable pair. This is a style choice and not something that should block acceptance.

**Reviewed-by worthy.** No issues found.

---
Generated by Claude Code Patch Reviewer