From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/msm: Switch to generic PAS TZ APIs Date: Tue, 28 Apr 2026 14:49:24 +1000 Message-ID: In-Reply-To: <20260427095603.1157963-11-sumit.garg@kernel.org> References: <20260427095603.1157963-1-sumit.garg@kernel.org> <20260427095603.1157963-11-sumit.garg@kernel.org> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Clean migration. The error message change from "SCM is not available" to "Qcom PAS is not available" is appropriate. Note `adreno_gpu.c` still includes `qcom_scm.h` as well -- this is correct because it uses `qcom_scm_set_remote_state` elsewhere... wait, actually no. Looking more carefully, `a5xx_gpu.c` replaces the only `qcom_scm_set_remote_state` call with `qcom_pas_set_remote_state`. The remaining `#include ` in `adreno_gpu.c` may still be needed for other SCM calls (like `qcom_scm_gpu_init_regs`). Fine. --- Generated by Claude Code Patch Reviewer