Re: [PATCH v4 4/8] media: qcom: iris: split HFI session ops from core ops

From: Dikshita Agarwal

Date: Fri Mar 13 2026 - 03:15:08 EST




On 3/13/2026 9:00 AM, Dmitry Baryshkov wrote:
> Calling HFI instance-specific ops should not require double indirection
> through the core ops. Split instance-specific ops to a separate struct,
> keep a pointer to it in struct iris_inst and set it directly in the
> get_instance function.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> ---
> drivers/media/platform/qcom/iris/iris_buffer.c | 4 +-
> drivers/media/platform/qcom/iris/iris_common.c | 8 ++--
> drivers/media/platform/qcom/iris/iris_ctrls.c | 46 +++++++++++-----------
> drivers/media/platform/qcom/iris/iris_hfi_common.h | 3 ++
> .../platform/qcom/iris/iris_hfi_gen1_command.c | 23 ++++++++---
> .../platform/qcom/iris/iris_hfi_gen2_command.c | 17 +++++---
> drivers/media/platform/qcom/iris/iris_instance.h | 4 ++
> drivers/media/platform/qcom/iris/iris_vb2.c | 2 +-
> drivers/media/platform/qcom/iris/iris_vdec.c | 6 +--
> drivers/media/platform/qcom/iris/iris_venc.c | 4 +-
> drivers/media/platform/qcom/iris/iris_vidc.c | 2 +-
> 11 files changed, 72 insertions(+), 47 deletions(-)
>

Reviewed-by: Dikshita Agarwal <dikshita.agarwal@xxxxxxxxxxxxxxxx>

Thanks,
Dikshita