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: add support for Triple 10-BIT Date: Sun, 12 Apr 2026 14:40:54 +1000 Message-ID: In-Reply-To: <20260406073930.24662-1-clamor95@gmail.com> References: <20260406073930.24662-1-clamor95@gmail.com> <20260406073930.24662-1-clamor95@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Author:** David Heidelberg, submitted by Svyatoslav Ryhel Adds `idt,v103` to the lvds-codec.yaml binding schema under the encoder enu= m: ```yaml - idt,v103 # For the Triple 10-BIT LVDS Transmitter ``` **Review:** - **Alphabetical ordering** is correct: `doestek,dtc34lm85am` =E2=86=92 `id= t,v103` =E2=86=92 `onnn,fin3385`. - **Vendor prefix** `idt` is registered in vendor-prefixes.yaml (line 766). - **No driver change needed** =E2=80=94 this entry falls under the `oneOf` = clause that requires a second compatible `lvds-encoder`, so the generic lvd= s-encoder/lvds-codec driver handles it via the fallback compatible. This is= the expected pattern. - **Nit:** Subject says "dt-bindigs" =E2=80=94 should be "dt-bindings". Min= or typo, not a blocker. - Has Reviewed-by from Robert Foss and Acked-by from Rob Herring. No issues. --- --- Generated by Claude Code Patch Reviewer