From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: arm64: dts: rockchip: Add eDP node for RK3576 Date: Sun, 22 Mar 2026 04:26:15 +1000 Message-ID: In-Reply-To: <20260319104031.1986946-7-damon.ding@rock-chips.com> References: <20260319104031.1986946-1-damon.ding@rock-chips.com> <20260319104031.1986946-7-damon.ding@rock-chips.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 DTS node looks well-formed. It follows the pattern of the RK3588 eDP nodes with appropriate RK3576-specific clock/reset/interrupt references. One observation: The RK3588 eDP nodes do not have a `power-domains` property in the upstream DTS (only clocks, resets, phys, etc.), but this RK3576 node adds `power-domains = <&power RK3576_PD_VO0>`. This is likely correct for RK3576 but worth confirming it's needed. Reviewed-by tag from Nicolas Frattaroli is present. Looks good. --- Generated by Claude Code Patch Reviewer