From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: media: qcom: Pass proper PAS ID to set_remote_state API Date: Tue, 05 May 2026 08:22:46 +1000 Message-ID: In-Reply-To: <20260504130603.1474043-13-sumit.garg@kernel.org> References: <20260504130603.1474043-1-sumit.garg@kernel.org> <20260504130603.1474043-13-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 Changes the PAS ID argument from `0` to the actual PAS ID for both iris and venus: ```c - return qcom_pas_set_remote_state(resume, 0); + return qcom_pas_set_remote_state(resume, core->iris_platform_data->pas_id); ``` The commit message explains the SCM backend ignores the value while OP-TEE uses it. This is a behavioral change for the SCM path but is safe since the backend ignores it. Good that this is in its own patch for bisectability. No issues. --- --- Generated by Claude Code Patch Reviewer