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: remoteproc: qcom, sm8550-pas: Add Glymur ADSP Date: Wed, 11 Mar 2026 13:43:35 +1000 Message-ID: In-Reply-To: <20260310033617.3108675-2-sibi.sankar@oss.qualcomm.com> References: <20260310033617.3108675-1-sibi.sankar@oss.qualcomm.com> <20260310033617.3108675-2-sibi.sankar@oss.qualcomm.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Looks correct.** Adds `qcom,glymur-adsp-pas` as a compatible that falls back to `qcom,sm8550-adsp-pas`, alongside `qcom,kaanapali-adsp-pas` and `qcom,sm8750-adsp-pas`. Also adds it to the `allOf` `contains` block that constrains interrupts to 6 and memory-region to 2, which matches the DTS in patch 4 (6 interrupts including "shutdown-ack", 2 memory regions). No issues found. --- Generated by Claude Code Patch Reviewer