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: microchip, sam9x75-lvds: document SAMA7D65 binding Date: Fri, 27 Feb 2026 13:48:29 +1000 Message-ID: In-Reply-To: <20260225085430.480052-3-manikandan.m@microchip.com> References: <20260225085430.480052-1-manikandan.m@microchip.com> <20260225085430.480052-3-manikandan.m@microchip.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 patch looks correct. It changes the LVDS binding from a single `const` to a `oneOf` with either the bare `sam9x75-lvds` or a `sama7d65-lvds` + `sam9x75-lvds` fallback pair: ```yaml + oneOf: + - const: microchip,sam9x75-lvds + - items: + - const: microchip,sama7d65-lvds + - const: microchip,sam9x75-lvds ``` This is the standard and correct DT binding pattern for a new SoC-specific compatible that falls back to an existing driver compatible. No issues here. **Ordering note:** Convention is to place binding patches before DTS patches in a series. This should be patch 1, before the DTS additions. --- Generated by Claude Code Patch Reviewer