From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: dt-bindings: display: bridge: Document Renesas RZ/G3L LVDS encoder Date: Thu, 23 Apr 2026 08:20:38 +1000 Message-ID: In-Reply-To: <20260421172910.218497-3-biju.das.jz@bp.renesas.com> References: <20260421172910.218497-1-biju.das.jz@bp.renesas.com> <20260421172910.218497-3-biju.das.jz@bp.renesas.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review This binding looks well-structured and complete. A few notes: **Description grammar nit** (line 299 of mbox): ```yaml description: | This binding describe the LVDS encoder embedded in the Renesas RZ/G3L ``` Should be "This binding **describes**". **Example nests LVDS node inside syscon** - The example shows the LVDS child node at `reg = <0x10 0x8>` inside a `lvds-cmn@108a0000` syscon parent. This correctly matches the driver's use of `syscon_node_to_regmap(dev->of_node->parent)`. The binding is clear that this is a child of a syscon MFD. **Reset names** - The binding defines 4 resets: `lvdrst`, `prst`, `rst`, `arst`. The names are short but somewhat cryptic. The descriptions help (LVDS_RESET_N, MIPI_DSI_PRESET_N, MIPI_DSI_CMN_RSTB, MIPI_DSI_ARESET_N). This is acceptable. **No issues blocking merge.** Already has Acked-by from Conor Dooley. --- --- Generated by Claude Code Patch Reviewer