Claude review: accel/ethosu propagate devm_drm_dev_alloc() error code

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

Patch Review

**Correctness: Good.**

The change from:
```c
if (IS_ERR(ethosudev))
    return -ENOMEM;
```
to:
```c
if (IS_ERR(ethosudev))
    return PTR_ERR(ethosudev);
```

is correct. `devm_drm_dev_alloc()` is a wrapper around `__devm_drm_dev_alloc()` which can fail for multiple reasons, so hardcoding `-ENOMEM` loses information. `PTR_ERR()` is the standard pattern for extracting the error code from an `ERR_PTR` value.

**Minor nits:**

- The added blank line between `return PTR_ERR(ethosudev);` and `platform_set_drvdata(...)` is fine stylistically but is a whitespace-only change beyond the fix itself. Not a problem.

- The subject line is missing a colon after the subsystem prefix — it reads `accel/ethosu propagate` rather than `accel/ethosu: propagate`. This doesn't affect the code but is a minor deviation from standard kernel commit message conventions.

**Verdict:** Reviewed-by worthy. A straightforward and correct fix.

---
Generated by Claude Code Patch Reviewer