Claude review: drm/v3d: Use devm_reset_control_get_optional_exclusive()

Claude Code Review Bot
2026-03-13T03:47:07Z

Patch Review

Straightforward cleanup using the proper `devm_reset_control_get_optional_exclusive()` API. The use of `dev_err_probe()` for error reporting is idiomatic. The separation of the error path (`IS_ERR`) from the absent-reset path (`!v3d->reset`) is cleaner than the previous approach:

```c
+ v3d->reset = devm_reset_control_get_optional_exclusive(dev, NULL);
if (IS_ERR(v3d->reset)) {
- ret = PTR_ERR(v3d->reset);
-
- if (ret == -EPROBE_DEFER)
- goto clk_disable;
-
- v3d->reset = NULL;
+ ret = dev_err_probe(dev, PTR_ERR(v3d->reset),
+ "Failed to get reset control\n");
+ goto clk_disable;
+ }
+
+ if (!v3d->reset) {
```

Has two Reviewed-by tags. No issues.