From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/v3d: Use devm_reset_control_get_optional_exclusive() Date: Wed, 01 Apr 2026 07:45:10 +1000 Message-ID: In-Reply-To: <20260331-v3d-power-management-v9-1-f52ff87bfd36@igalia.com> References: <20260331-v3d-power-management-v9-0-f52ff87bfd36@igalia.com> <20260331-v3d-power-management-v9-1-f52ff87bfd36@igalia.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Status: Looks good.** Clean, straightforward conversion from `devm_reset_control_get_exclusive()`= to `devm_reset_control_get_optional_exclusive()`. The old code had an ad-h= oc optional pattern (check for `-EPROBE_DEFER`, set to NULL otherwise); the= new code uses the proper API that returns NULL when the reset is absent an= d only returns errors for real failures. The switch to `dev_err_probe()` is a nice touch =E2=80=94 it handles `-EPRO= BE_DEFER` logging automatically. No issues. Already has R-b from Melissa, Philipp, and Florian. --- --- Generated by Claude Code Patch Reviewer