Re: [PATCH v2 1/3] firmware: qcom: pas: Add optional video memory protection op

[Konrad Dybcio]

On 10/27/26 1:39 PM, Sumit Garg wrote:
> From: Sumit Garg <sumit.garg@xxxxxxxxxxxxxxxx>
> 
> qcom_scm_mem_protect_video_var() is only serviced by the SCM backend.
> Other PAS backends, e.g. OP-TEE, own secure memory protection and don't
> implement such a call.
> 
> Add an optional mem_protect_video_var() PAS op along with a generic
> qcom_pas_mem_protect_video_var() wrapper so that clients stay agnostic
> of the active backend. The SCM backend maps it to the existing SCM call
> while backends leaving it unset turn it into a no-op.
> 
> Signed-off-by: Sumit Garg <sumit.garg@xxxxxxxxxxxxxxxx>
> ---

I agree it's imperfect but it fits better

Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>

Konrad