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/bridge: sii9234: use dev_err_probe where applicable Date: Thu, 04 Jun 2026 12:19:46 +1000 Message-ID: In-Reply-To: <20260602-exynos4-sii9234-driver-v4-2-9b2fc38e104a@grimler.se> References: <20260602-exynos4-sii9234-driver-v4-0-9b2fc38e104a@grimler.se> <20260602-exynos4-sii9234-driver-v4-2-9b2fc38e104a@grimler.se> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Verdict: Good to go.** Converts three error paths to use `dev_err_probe()`: 1. GPIO reset acquisition -- was using `IS_ERR` + `dev_err` + return, now uses `dev_err_probe` which correctly handles deferred probe suppression. 2. Regulator bulk get -- was manually checking for `-EPROBE_DEFER` to suppress the message, `dev_err_probe` handles this automatically. 3. IRQ handler request -- was using `dev_err` unconditionally, `dev_err_probe` is better since `devm_request_threaded_irq` can return `-EPROBE_DEFER`. All three conversions are correct. Net reduction of 5 lines. Reviewed-by from Dmitry is appropriate. --- Generated by Claude Code Patch Reviewer