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: convert users of of_drm_find_bridge(), part 4 Date: Sun, 12 Apr 2026 11:12:49 +1000 Message-ID: In-Reply-To: <20260409-drm-bridge-alloc-getput-drm_of_find_bridge-4-v5-0-d7381c07788a@bootlin.com> References: <20260409-drm-bridge-alloc-getput-drm_of_find_bridge-4-v5-0-d7381c07788a@bootlin.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: drm/bridge: convert users of of_drm_find_bridge(), part 4 Author: Luca Ceresoli Patches: 3 Reviewed: 2026-04-12T11:12:49.754762 --- This is a clean, well-structured 2-patch series converting the two Renesas DU encoder drivers (`rz-du` and `rcar-du`) from the deprecated `of_drm_find_bridge()` to `of_drm_find_and_get_bridge()`, as part of the ongoing DRM bridge refcounting work. The series is at v5 and has already collected Reviewed-by tags from four reviewers (Biju Das, Linus Walleij, Laurent Pinchart, Louis Chauvet). The approach is sound: use `__free(drm_bridge_put)` on the local `bridge` variable for automatic cleanup, take an extra reference via `drm_bridge_get()` in the panel-bridge path to unify the two branches, and use `no_free_ptr()` to suppress cleanup for error pointers. The v5 simplification with `no_free_ptr` is a nice improvement over explicit goto-based cleanup. Both patches are correct. The refcounting logic is balanced across all paths. No issues found that would block merging. --- Generated by Claude Code Patch Reviewer