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: npu: Add bindings for NXP Neutron Date: Fri, 27 Feb 2026 12:10:07 +1000 Message-ID: In-Reply-To: <20260226-neutron-v1-3-46eccb3bb50a@nxp.com> References: <20260226-neutron-v1-0-46eccb3bb50a@nxp.com> <20260226-neutron-v1-3-46eccb3bb50a@nxp.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 binding looks reasonable. Minor note: the example has mixed-case hex addresses (`0x4AB10000`) vs lowercase in the actual DTS patch (`0x4ab10000`). DT convention is lowercase hex. The example should be consistent: ```yaml reg = <0x0 0x4ab00000 0x0 0x00000400>, <0x0 0x4AB10000 0x0 0x00010000>, <0x0 0x4AB08000 0x0 0x00008000>; ``` Should use lowercase `0x4ab10000`, `0x4ab08000`. --- Generated by Claude Code Patch Reviewer