From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: Add support for Renesas RZ/G3L LVDS encoder Date: Wed, 27 May 2026 15:11:40 +1000 Message-ID: In-Reply-To: <20260526074746.20371-1-biju.das.jz@bp.renesas.com> References: <20260526074746.20371-1-biju.das.jz@bp.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: Add support for Renesas RZ/G3L LVDS encoder Author: Biju Patches: 3 Reviewed: 2026-05-27T15:11:40.495706 --- This is a clean, well-structured 2-patch series adding LVDS encoder support= for the Renesas RZ/G3L SoC. The driver follows established patterns from t= he existing `rzg2l_mipi_dsi.c` driver in the same directory and uses modern= DRM bridge APIs (`devm_drm_bridge_alloc`, `devm_drm_bridge_add`). The DT b= inding is straightforward and already has Acked-by from a DT maintainer. The series is in good shape overall for v3. There are a few minor issues wo= rth flagging =E2=80=94 one potential refcount concern with storing `devm_dr= m_of_get_bridge()` result directly into `bridge.next_bridge`, an unnecessar= y include, and some register header cleanup opportunities =E2=80=94 but not= hing that blocks merging. --- --- Generated by Claude Code Patch Reviewer