On 6/26/26 3:34 PM, Sumit Garg wrote: > From: Sumit Garg <[email protected]> > > 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 <[email protected]> > Tested-by: Mukesh Ojha <[email protected]> # Lemans > Reviewed-by: Vikash Garodia <[email protected]> > Signed-off-by: Sumit Garg <[email protected]> > --- > 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 ea9654dd679e..d2e7ba4f37e3 100644 > --- a/drivers/media/platform/qcom/iris/iris_firmware.c > +++ b/drivers/media/platform/qcom/iris/iris_firmware.c > @@ -110,5 +110,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);
This should not be in the middle of a mildly related series.. The PAS IDs should be centralized into a single header. And the name of the driver shouldn't be part of the define. I would guesstimate that on the secure side it's probably called VPU or VIDEO Konrad
