From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: accel/neutron: Add mailbox support Date: Fri, 27 Feb 2026 12:10:08 +1000 Message-ID: In-Reply-To: <20260226-neutron-v1-6-46eccb3bb50a@nxp.com> References: <20260226-neutron-v1-0-46eccb3bb50a@nxp.com> <20260226-neutron-v1-6-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 This patch is clean and simple. The mailbox interface with 8 register-backed mailboxes, command send, state read, and reset operations looks fine. One minor note: `neutron_mbox_send_cmd` writes args with `writel_relaxed` but the final command ID with `writel`. The ordering guarantee from the non-relaxed `writel` ensures all args are visible before the command, which is correct. --- Generated by Claude Code Patch Reviewer