[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6d88effc-2aed-4f92-89a7-17d48f43ccd3@linaro.org>
Date: Wed, 5 Mar 2025 15:22:42 +0100
From: neil.armstrong@...aro.org
To: Dikshita Agarwal <quic_dikshita@...cinc.com>, quic_vgarodia@...cinc.com,
quic_abhinavk@...cinc.com, mchehab@...nel.org
Cc: hverkuil@...all.nl, linux-media@...r.kernel.org,
linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH 00/12] Add support for HEVC and VP9 codecs in decoder
Hi,
On 05/03/2025 11:43, Dikshita Agarwal wrote:
> Hi all,
>
> This patch series adds initial support for the HEVC(H.265) and VP9
> codecs in iris decoder. The objective of this work is to extend the
> decoder's capabilities to handle HEVC and VP9 codec streams,
> including necessary format handling and buffer management.
> In addition, the series also includes a set of fixes to address issues
> identified during testing of these additional codecs.
>
> I'm sharing this series as an RFC because compliance and conformance
> testing are still in progress.
> While initial functional tests show positive results, I would
> appreciate early feedback on the design, implementation, and fixes
> before moving to a formal submission.
>
> I plan to submit a formal patch series after completing all compliance
> checks. Meanwhile, any feedback or suggestion to improve this work are
> very welcome.
>
> Thanks,
> Dikshita
>
> Dikshita Agarwal (12):
> media: iris: Add HEVC and VP9 formats for decoder
> media: iris: Add platform capabilities for HEVC and VP9 decoders
> media: iris: Set mandatory properties for HEVC and VP9 decoders.
> media: iris: Add internal buffer calculation for HEVC and VP9 decoders
> media: iris: Skip destroying internal buffer if not dequeued
> media: iris: Update CAPTURE format info based on OUTPUT format
> media: iris: Add handling for corrupt and drop frames
> media: iris: Avoid updating frame size to firmware during reconfig
> media: iris: Avoid sending LAST flag multiple times
> media: iris: Send V4L2_BUF_FLAG_ERROR for buffers with 0 filled length
> media: iris: Fix handling of eos buffer during drain
> media: iris: Add handling for no show frames
I should be better to move patches 1, 2 & 3 at the end, after the patches
adding support for specific hecv & h265 features, and please check that
none of the patches breaks h264 at any time to keep bisectability.
Neil
Neil
>
> .../media/platform/qcom/iris/iris_buffer.c | 22 +-
> drivers/media/platform/qcom/iris/iris_ctrls.c | 28 +-
> .../platform/qcom/iris/iris_hfi_common.h | 1 +
> .../qcom/iris/iris_hfi_gen1_command.c | 38 +-
> .../qcom/iris/iris_hfi_gen1_defines.h | 4 +
> .../qcom/iris/iris_hfi_gen1_response.c | 11 +
> .../qcom/iris/iris_hfi_gen2_command.c | 129 +++++-
> .../qcom/iris/iris_hfi_gen2_defines.h | 5 +
> .../qcom/iris/iris_hfi_gen2_response.c | 56 ++-
> .../media/platform/qcom/iris/iris_instance.h | 6 +
> .../platform/qcom/iris/iris_platform_common.h | 25 +-
> .../platform/qcom/iris/iris_platform_sm8250.c | 4 +-
> .../platform/qcom/iris/iris_platform_sm8550.c | 141 ++++++-
> drivers/media/platform/qcom/iris/iris_vb2.c | 3 +-
> drivers/media/platform/qcom/iris/iris_vdec.c | 80 +++-
> drivers/media/platform/qcom/iris/iris_vdec.h | 11 +
> drivers/media/platform/qcom/iris/iris_vidc.c | 3 -
> .../platform/qcom/iris/iris_vpu_buffer.c | 397 +++++++++++++++++-
> .../platform/qcom/iris/iris_vpu_buffer.h | 46 +-
> 19 files changed, 931 insertions(+), 79 deletions(-)
>
Powered by blists - more mailing lists