From: Sumit Garg sumit.garg@oss.qualcomm.com
As per testing the SCM backend just ignores it while OP-TEE makes use of it to for proper book keeping purpose.
Reviewed-by: Mukesh Ojha mukesh.ojha@oss.qualcomm.com Tested-by: Mukesh Ojha mukesh.ojha@oss.qualcomm.com # Lemans Reviewed-by: Vikash Garodia vikash.garodia@oss.qualcomm.com Reviewed-by: Konrad Dybcio konrad.dybcio@oss.qualcomm.com Signed-off-by: Sumit Garg sumit.garg@oss.qualcomm.com --- Changes in v10: - Resent standalone, as all the earlier patches of the series are merged. - Link to v9: https://lore.kernel.org/all/20260702115835.167602-11-sumit.garg@kernel.org/ --- drivers/media/platform/qcom/iris/iris_firmware.c | 2 +- drivers/media/platform/qcom/venus/firmware.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/qcom/iris/iris_firmware.c b/drivers/media/platform/qcom/iris/iris_firmware.c index 21f57d37da18..ed2bd0e83797 100644 --- a/drivers/media/platform/qcom/iris/iris_firmware.c +++ b/drivers/media/platform/qcom/iris/iris_firmware.c @@ -262,5 +262,5 @@ int iris_fw_unload(struct iris_core *core)
int iris_set_hw_state(struct iris_core *core, bool resume) { - return qcom_pas_set_remote_state(resume, 0); + return qcom_pas_set_remote_state(resume, IRIS_PAS_ID); } diff --git a/drivers/media/platform/qcom/venus/firmware.c b/drivers/media/platform/qcom/venus/firmware.c index 3a38ff985822..3c0727ea137d 100644 --- a/drivers/media/platform/qcom/venus/firmware.c +++ b/drivers/media/platform/qcom/venus/firmware.c @@ -59,7 +59,7 @@ int venus_set_hw_state(struct venus_core *core, bool resume) int ret;
if (core->use_tz) { - ret = qcom_pas_set_remote_state(resume, 0); + ret = qcom_pas_set_remote_state(resume, VENUS_PAS_ID); if (resume && ret == -EINVAL) ret = 0; return ret;
--- base-commit: d135ab453a7400a34930bdda1ef4bdc8acfac80e change-id: 20261027-pas-id-media-7cc70fc5716f
Best regards,
On 10/27/2026 5:32 PM, Sumit Garg wrote:
From: Sumit Garg sumit.garg@oss.qualcomm.com
As per testing the SCM backend just ignores it while OP-TEE makes use of it to for proper book keeping purpose.
Reviewed-by: Mukesh Ojha mukesh.ojha@oss.qualcomm.com Tested-by: Mukesh Ojha mukesh.ojha@oss.qualcomm.com # Lemans Reviewed-by: Vikash Garodia vikash.garodia@oss.qualcomm.com Reviewed-by: Konrad Dybcio konrad.dybcio@oss.qualcomm.com Signed-off-by: Sumit Garg sumit.garg@oss.qualcomm.com
Changes in v10:
- Resent standalone, as all the earlier patches of the series are merged.
- Link to v9: https://lore.kernel.org/all/20260702115835.167602-11-sumit.garg@kernel.org/
drivers/media/platform/qcom/iris/iris_firmware.c | 2 +- drivers/media/platform/qcom/venus/firmware.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/qcom/iris/iris_firmware.c b/drivers/media/platform/qcom/iris/iris_firmware.c index 21f57d37da18..ed2bd0e83797 100644 --- a/drivers/media/platform/qcom/iris/iris_firmware.c +++ b/drivers/media/platform/qcom/iris/iris_firmware.c @@ -262,5 +262,5 @@ int iris_fw_unload(struct iris_core *core) int iris_set_hw_state(struct iris_core *core, bool resume) {
- return qcom_pas_set_remote_state(resume, 0);
- return qcom_pas_set_remote_state(resume, IRIS_PAS_ID);
} diff --git a/drivers/media/platform/qcom/venus/firmware.c b/drivers/media/platform/qcom/venus/firmware.c index 3a38ff985822..3c0727ea137d 100644 --- a/drivers/media/platform/qcom/venus/firmware.c +++ b/drivers/media/platform/qcom/venus/firmware.c @@ -59,7 +59,7 @@ int venus_set_hw_state(struct venus_core *core, bool resume) int ret; if (core->use_tz) {
ret = qcom_pas_set_remote_state(resume, 0);
if (resume && ret == -EINVAL) ret = 0; return ret;ret = qcom_pas_set_remote_state(resume, VENUS_PAS_ID);
base-commit: d135ab453a7400a34930bdda1ef4bdc8acfac80e change-id: 20261027-pas-id-media-7cc70fc5716f
Best regards,
Reviewed-by: Gourav Kumar gourav.kumar@oss.qualcomm.com
op-tee@lists.trustedfirmware.org