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: renesas: rz-du: mipi_dsi: Fix return path on error Date: Thu, 23 Apr 2026 07:31:41 +1000 Message-ID: In-Reply-To: <20260422145146.1638998-1-chris.brandt@renesas.com> References: <20260422145146.1638998-1-chris.brandt@renesas.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm: renesas: rz-du: mipi_dsi: Fix return path on error Author: Chris Brandt Patches: 2 Reviewed: 2026-04-23T07:31:41.505756 --- This is a single-patch fix for a resource leak bug in the RZ/G2L MIPI DSI d= river's probe function. The original code calls `dma_alloc_coherent` after = `mipi_dsi_host_register` and `pm_runtime_enable`, but on failure returns `-= ENOMEM` directly =E2=80=94 skipping the cleanup of the host registration an= d pm_runtime. The fix takes a clean approach: switch to the device-managed = `dmam_alloc_coherent` and move the allocation earlier in probe, before any = resources that would need unwinding. This is a well-structured v2 addressin= g reviewer feedback. **Verdict: Looks good to merge.** --- Generated by Claude Code Patch Reviewer