From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: MAINTAINERS: Add maintainer entry for Qualcomm PAS TZ service Date: Tue, 05 May 2026 08:22:46 +1000 Message-ID: In-Reply-To: <20260504130603.1474043-17-sumit.garg@kernel.org> References: <20260504130603.1474043-1-sumit.garg@kernel.org> <20260504130603.1474043-17-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 Adds Sumit Garg as maintainer for the four new PAS files. Alphabetically ordered in the right location. No issues. --- ## Summary | Severity | Patch | Issue | |----------|-------|-------| | **Bug** | 14 (ath12k) | Printf format `%d: %d` has two specifiers but only one argument. Missing `ret`. | | Medium | 2 (qcom_pas) | `qcom_pas_ops_register()` has a TOCTOU race -- two callers could both store. Consider `cmpxchg`. | | Low | 4 (qcom_pas_tee) | First `tee_client_invoke_func` call in `get_rsc_table` with NULL shm deserves a comment. | | Low | 4 (qcom_pas_tee) | `!CPU_BIG_ENDIAN` Kconfig dependency lacks explanatory comment. | The ath12k printf bug is the only must-fix. The registration race is worth addressing but is unlikely to trigger in practice. Otherwise this is a clean and well-organized series. --- Generated by Claude Code Patch Reviewer