From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: ARM: dts: microchip: sama7d65: add LVDS controller Date: Fri, 27 Feb 2026 13:48:29 +1000 Message-ID: In-Reply-To: <20260225085430.480052-4-manikandan.m@microchip.com> References: <20260225085430.480052-1-manikandan.m@microchip.com> <20260225085430.480052-4-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 The LVDS controller node looks correct and properly uses the fallback compatible as documented in patch 2: ``` + lvdsc: lvds-controller@e1408000 { + compatible = "microchip,sama7d65-lvds", "microchip,sam9x75-lvds"; ``` The address `0xe1408000` is properly ordered in the DT between the LCD controller (`0xe1400000`) and the crypto block (`0xe1600000`). The properties (reg, interrupts, clocks, clock-names, status) match the binding schema requirements. This patch depends on patch 1 (it inserts after the xlcdc node added there) and patch 2 (it uses the newly-documented compatible). No issues. --- Generated by Claude Code Patch Reviewer