[PATCH v5] media: venus : hfi: add venus image info into smem

2021-04-12 Thread Dikshita Agarwal
Fill fw version info into smem to be printed as part of soc info. Signed-off-by: Dikshita Agarwal Reviewed-by: Stephen Boyd --- drivers/media/platform/Kconfig | 2 +- drivers/media/platform/qcom/venus/hfi_msgs.c | 20 ++-- 2 files changed, 19 insertions(+), 3

[PATCH] media: venus: Enable low power setting for encoder

2021-04-12 Thread Dikshita Agarwal
Set the FW to run in low power for encoder to accommodate more session without losing much on quality. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 6 ++ drivers/media/platform/qcom/venus/helpers.c| 2 + drivers/media/platform/qcom/venus

[PATCH v4] media: venus : hfi: add venus image info into smem

2021-04-09 Thread Dikshita Agarwal
Fill fw version info into smem to be printed as part of soc info. Signed-off-by: Dikshita Agarwal Reported-by: kernel test robot change since v3: added dependency on QCOM_SMEM (Stephen) --- drivers/media/platform/Kconfig | 2 +- drivers/media/platform/qcom/venus/hfi_msgs.c

Re: [PATCH v2] media: venus : hfi: add venus image info into smem

2021-04-06 Thread dikshita
Hi Bjorn, Thanks for your review comments. I have addressed all in the latest patch v3. couldn't think of a shorter name for variables without losing the readability so kept as it is. Thanks, Dikshita On 2021-03-30 09:29, Bjorn Andersson wrote: On Fri 26 Mar 01:33 CDT 2021, Dikshita Agarwal

[PATCH v3] media: venus : hfi: add venus image info into smem

2021-04-06 Thread Dikshita Agarwal
Fill fw version info into smem to be printed as part of soc info. Signed-off-by: Dikshita Agarwal changes since v2: - adressed all review comments. --- drivers/media/platform/qcom/venus/hfi_msgs.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git

[PATCH v2] media: venus : hfi: add venus image info into smem

2021-03-26 Thread Dikshita Agarwal
Fill fw version info into smem to be printed as part of soc info. Signed-off-by: Dikshita Agarwal Changes since v1: adressed comments from stephen. removed unwanted code. --- drivers/media/platform/qcom/venus/hfi_msgs.c | 21 +++-- 1 file changed, 19 insertions(+), 2

[PATCH v10 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-03-24 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal Reviewed-by: Hans Verkuil --- .../userspace

[PATCH v10 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-03-24 Thread Dikshita Agarwal
Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 2 ++ drivers/media/platform/qcom/venus/venc.c

[PATCH v10 0/2] Add encoder ctrls for long term reference

2021-03-24 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. Dikshita Agarwal (2): media: v4l2-ctrl: add controls for long term reference. venus: venc: Add support for Long Term Reference (LTR) controls .../userspace-api/media/v4l/ext-ctrls

[PATCH] media: venus : hfi: add venus image info into smem

2021-03-23 Thread Dikshita Agarwal
fill fw version info into smem to be printed as part of soc info. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/hfi_msgs.c | 36 ++-- 1 file changed, 34 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/qcom/venus/hfi_msgs.c b

[PATCH v9 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-03-23 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal Reviewed-by: Hans Verkuil --- .../userspace

[PATCH v9 0/2] Add encoder ctrls for long term reference

2021-03-23 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. Dikshita Agarwal (2): media: v4l2-ctrl: add controls for long term reference. venus: venc: Add support for Long Term Reference (LTR) controls Changes since v8: - Update 2/2 patch

[PATCH v9 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-03-23 Thread Dikshita Agarwal
Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc_ctrls.c | 55 +- 1 file changed, 54

[PATCH] media: venus: Fix internal buffer size calculations for v6.

2021-03-17 Thread Dikshita Agarwal
decoder VPP command buffer. - Update VP9_NUM_FRAME_INFO_BUF to 32. Signed-off-by: Dikshita Agarwal --- .../media/platform/qcom/venus/hfi_plat_bufs_v6.c | 27 ++ 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/drivers/media/platform/qcom/venus/hfi_plat_bufs_v6.c

[PATCH v8 0/2] Add encoder ctrls for long term reference

2021-03-11 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. Dikshita Agarwal (2): media: v4l2-ctrl: add controls for long term reference. venus: venc: Add support for Long Term Reference (LTR) controls Changes since v7: - addressed comments

[PATCH v8 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-03-11 Thread Dikshita Agarwal
Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc_ctrls.c | 49 +- 1 file changed, 48

[PATCH v8 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-03-11 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

Re: [PATCH v7 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-03-11 Thread dikshita
Hi Hans, Thanks for your comments. I Will update the documentation based on your suggestion in the next patch. Thanks, Dikshita On 2021-03-05 14:08, Hans Verkuil wrote: Hi Dikshita, On 03/03/2021 12:09, Dikshita Agarwal wrote: Long Term Reference (LTR) frames are the frames

[PATCH v7 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-03-03 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

[PATCH v7 0/2] Add encoder ctrls for long term reference

2021-03-03 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. Dikshita Agarwal (2): media: v4l2-ctrl: add controls for long term reference. venus: venc: Add support for Long Term Reference (LTR) controls Changes since v6: - removed references

[PATCH v7 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-03-03 Thread Dikshita Agarwal
Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc_ctrls.c | 49 +- 1 file changed, 48

Re: [PATCH v6 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-02-03 Thread dikshita
On 2021-02-01 16:50, Hans Verkuil wrote: On 25/01/2021 06:51, Dikshita Agarwal wrote: Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable

Re: [PATCH v6 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-02-03 Thread dikshita
On 2021-02-01 16:40, Hans Verkuil wrote: On 25/01/2021 06:51, Dikshita Agarwal wrote: Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable

[PATCH v6 1/2] media: v4l2-ctrl: add controls for long term reference.

2021-01-24 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

[PATCH v6 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-01-24 Thread Dikshita Agarwal
Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc_ctrls.c | 49 +- 1 file changed, 48

[PATCH v6 0/2] Add encoder ctrls for long term reference

2021-01-24 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. Dikshita Agarwal (2): media: v4l2-ctrl: add controls for long term reference. venus: venc: Add support for Long Term Reference (LTR) controls .../userspace-api/media/v4l/ext-ctrls

[PATCH v6 0/2] Add encoder ctrls for long term reference

2021-01-24 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. Dikshita Agarwal (2): media: v4l2-ctrl: add controls for long term reference. venus: venc: Add support for Long Term Reference (LTR) controls .../userspace-api/media/v4l/ext-ctrls

Re: [PATCH v5 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-01-12 Thread dikshita
Hi Hans, On 2021-01-12 14:34, Hans Verkuil wrote: On 04/01/2021 06:09, Dikshita Agarwal wrote: Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES This patch does not add any Request API

[PATCH RESEND v3] venus: venc: set inband mode property to FW.

2021-01-07 Thread Dikshita Agarwal
set HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER to FW to support inband sequence header mode. Signed-off-by: Dikshita Agarwal Changes since v2: - fixed Null pointer dereference (Stanimir, Fritz) - added set property call at correct place. --- drivers/media/platform/qcom/venus/venc.c

[PATCH v3] venus: venc: set inband mode property to FW.

2021-01-07 Thread Dikshita Agarwal
set HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER to FW to support inband sequence header mode. Signed-off-by: Dikshita Agarwal Changes since v2: - fixed Null pointer dereference (Stanimir, Fritz) - added set property call at correct place. --- drivers/media/platform/qcom/venus/venc.c

[PATCH v4 0/2] Add base layer priority id control

2021-01-03 Thread Dikshita Agarwal
This series adds base layer priority id control for encoder and support for the same in venus driver. Changes since v3: - Rebased the changes on latest media tree. - Addressed the comments. - Added driver side implementation for new control. Dikshita Agarwal (2): media: v4l2-ctrl: Add base

[PATCH v4 2/2] venus: venc : Add support for priority ID control.

2021-01-03 Thread Dikshita Agarwal
Add support for base layer priority ID control in encoder. This is a preparation patch to support v6. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 2 ++ drivers/media/platform/qcom/venus/venc_ctrls.c | 9 - 2 files changed, 10 insertions(+), 1

[PATCH v4 1/2] media: v4l2-ctrl: Add base layer priority id control.

2021-01-03 Thread Dikshita Agarwal
This control indicates the priority id to be applied to base layer. Signed-off-by: Dikshita Agarwal --- Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst | 9 + drivers/media/v4l2-core/v4l2-ctrls.c | 1 + include/uapi/linux/v4l2-controls.h

[PATCH v5 1/2] media: v4l2-ctrl: add control for long term reference.

2021-01-03 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

[PATCH v5 2/2] venus: venc: Add support for Long Term Reference (LTR) controls

2021-01-03 Thread Dikshita Agarwal
Add support for below LTR controls in encoder: - V4L2_CID_MPEG_VIDEO_LTR_COUNT - V4L2_CID_MPEG_VIDEO_FRAME_LTR_INDEX - V4L2_CID_MPEG_VIDEO_USE_LTR_FRAMES Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc_ctrls.c | 49 +- 1 file changed, 48

[PATCH v5 0/2] Add encoder ctrls for long term reference

2021-01-03 Thread Dikshita Agarwal
This series add the encoder controls for long term reference (LTR) and support for the same in venus driver. changes since v4: - Rebased the changes on latest media tree - Addressed the comments - Added driver side implementation for new controls. Dikshita Agarwal (2): media: v4l2-ctrl: add

[PATCH v4 0/3] Add new controls for QP and layer bitrate

2020-12-24 Thread Dikshita Agarwal
This series adds frame specific min/max qp controls for hevc and layer wise bitrate control for h264. Chnages since v2: - Rebased the changes on latest media tree - Added driver side implementation for new controls. Dikshita Agarwal (3): media: v4l2-ctrl: Add frame-specific min/max qp

[PATCH v4 1/3] media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc

2020-12-24 Thread Dikshita Agarwal
- Adds min/max qp controls for B frame for h264. - Adds min/max qp controls for I/P/B frames for hevc similar to h264. - Update valid range of min/max qp for hevc to accommodate 10 bit. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst| 52

[PATCH v4 3/3] venus: venc: Add support for frame-specific min/max qp controls

2020-12-24 Thread Dikshita Agarwal
Add support for frame type specific min and max qp controls for encoder. This is a preparation patch to support v6. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 18 drivers/media/platform/qcom/venus/venc.c | 21 +++-- drivers/media/platform

[PATCH v4 2/3] media: v4l2-ctrl: Add layer wise bitrate controls for h264

2020-12-24 Thread Dikshita Agarwal
Adds bitrate control for all coding layers for h264 same as hevc. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst | 20 drivers/media/v4l2-core/v4l2-ctrls.c | 7 +++ include/uapi/linux/v4l2-controls.h

[PATCH] venus: venc: set IDR period to FW only for H264 & HEVC

2020-12-15 Thread Dikshita Agarwal
HFI_PROPERTY_CONFIG_VENC_IDR_PERIOD is supported for only H264 & HEVC codec. There is no need to set it for VP8 since all key frames are treated as IDR frames for VP8. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc.c | 23 +-- 1 file changed

[PATCH v2] venus: venc: set inband mode property to FW.

2020-12-15 Thread Dikshita Agarwal
set HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER to FW to support inband sequence header mode. Signed-off-by: Dikshita Agarwal Changes since v1: - added codec check - fixed the mask while initializing the control --- drivers/media/platform/qcom/venus/venc_ctrls.c | 15

Re: [PATCH 1/2] venus: venc: update output format based on capture format.

2020-12-15 Thread dikshita
Hi All, Kindly ignore this patch. https://patchwork.linuxtv.org/project/linux-media/list/?series=3790 replaces this. Thanks, Dikshita On 2020-09-24 23:01, Dikshita Agarwal wrote: update output format parameters when s_fmt is called on capture port as recommended in encoder spec. Signed-off

[PATCH v5] venus: core: add support to dump FW region

2020-12-15 Thread Dikshita Agarwal
Add support to dump video FW region during FW crash using devcoredump helpers. Signed-off-by: Dikshita Agarwal Reviewed-by: Stephen Boyd Reviewed-by: Stanimir Varbanov Major changes since v1: - update the name of function (Stephen) - store start address and size in resource structure during

[PATCH] venus: venc: set inband mode property to FW.

2020-12-14 Thread Dikshita Agarwal
set HFI_PROPERTY_CONFIG_VENC_SYNC_FRAME_SEQUENCE_HEADER to FW to support inband sequence header mode. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc_ctrls.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform

[PATCH v4] venus: core: add support to dump FW region

2020-12-14 Thread Dikshita Agarwal
Add support to dump video FW region during FW crash using devcoredump helpers. Major changes since v1: - update the name of function (Stephen) - store start address and size in resource structure during probe and reuse while dumping (Stephen, Stanimir) Signed-off-by: Dikshita Agarwal Reviewed

Re: [PATCH v4] media: v4l2-ctrl: add control for long term reference.

2020-12-08 Thread dikshita
Hi Hans, On 2020-12-02 19:18, Hans Verkuil wrote: On 01/12/2020 09:13, Dikshita Agarwal wrote: Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls

Re: [PATCH v3 3/3] venus: venc: Add support for frame-specific min/max qp controls

2020-12-08 Thread dikshita
On 2020-12-02 19:06, Hans Verkuil wrote: On 18/11/2020 07:54, Dikshita Agarwal wrote: Add support for frame type specific min and max qp controls for encoder. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 18 + drivers/media/platform/qcom/venus

[PATCH v3] venus: core: add support to dump FW region

2020-12-06 Thread dikshita
From: Dikshita Agarwal Add support to dump video FW region during FW crash using devcoredump helpers. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.c | 31 drivers/media/platform/qcom/venus/core.h | 2 ++ drivers/media

[RESEND v2] venus: core: add support to dump FW region

2020-12-03 Thread dikshita
From: Dikshita Agarwal Add support to dump video FW region during FW crash using devcoredump helpers. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.c | 30 drivers/media/platform/qcom/venus/core.h | 2 ++ drivers/media

[PATCH v2] venus: core: add support to dump FW region

2020-12-03 Thread dikshita
From: Dikshita Agarwal Add support to dump video FW region during FW crash using devcoredump helpers. Signed-off-by: Dikshita Agarwal Change-Id: Ic055579b99ffd10fe5c346f95c33aac6a97b21f2 --- drivers/media/platform/qcom/venus/core.c | 30 drivers/media/platform

Re: [PATCH 1/3] venus: venc: Init the session only once in queue_setup

2020-12-01 Thread dikshita
Hi Stan, On 2020-11-20 05:40, Stanimir Varbanov wrote: Init the hfi session only once in queue_setup and also cover that with inst->lock. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/venus/venc.c | 98 ++-- 1 file changed, 73 insertions(+), 25

[PATCH v4] media: v4l2-ctrl: add control for long term reference.

2020-12-01 Thread Dikshita Agarwal
Long Term Reference (LTR) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

[PATCH] venus: core: add support to dump FW region

2020-11-29 Thread Dikshita Agarwal
Add support to dump video FW region during FW crash using devcoredump helpers. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.c | 47 1 file changed, 47 insertions(+) diff --git a/drivers/media/platform/qcom/venus/core.c b/drivers

[PATCH v3] media: v4l2-ctrl: Add base layer priority id control.

2020-11-23 Thread Dikshita Agarwal
This control indicates the priority id to be applied to base layer. Signed-off-by: Dikshita Agarwal --- Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst | 9 + drivers/media/v4l2-core/v4l2-ctrls.c | 1 + include/uapi/linux/v4l2-controls.h

[PATCH v3 1/3] media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc

2020-11-17 Thread Dikshita Agarwal
- Adds min/max qp controls for B frame for h264. - Adds min/max qp controls for I/P/B frames for hevc similar to h264. - Update valid range of mim/max qp for hevc to accommodate 10 bit. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst| 52

[PATCH v3 3/3] venus: venc: Add support for frame-specific min/max qp controls

2020-11-17 Thread Dikshita Agarwal
Add support for frame type specific min and max qp controls for encoder. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 18 + drivers/media/platform/qcom/venus/venc.c | 21 --- drivers/media/platform/qcom/venus/venc_ctrls.c | 51

[PATCH v3 0/3] Add new controls for QP and layer bitrate

2020-11-17 Thread Dikshita Agarwal
This series adds frame specific min/max qp controls for hevc and layer wise bitrate control for h264. Chnages since v2: - addressed comments. - added driver side implementation for new controls. Dikshita Agarwal (3): media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc media

[PATCH v3 2/3] media: v4l2-ctrl: Add layer wise bitrate controls for h264

2020-11-17 Thread Dikshita Agarwal
Adds bitrate control for all coding layers for h264 same as hevc. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst | 20 drivers/media/v4l2-core/v4l2-ctrls.c | 7 +++ include/uapi/linux/v4l2-controls.h

[PATCH v2] venus: venc: fix handlig of S_SELECTION and G_SELECTION

2020-11-04 Thread Dikshita Agarwal
- return correct width and height for G_SELECTION - update capture port wxh with rectangle wxh. - add support for HFI_PROPERTY_PARAM_UNCOMPRESSED_PLANE_ACTUAL_INFO to set stride info and chroma offset to FW. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/helpers.c

Re: [PATCH 2/2] venus: venc: fix handlig of S_SELECTION and G_SELECTION

2020-11-03 Thread dikshita
Hi Tomasz, Proper handling of CROP is added with new patch https://lkml.org/lkml/2020/11/3/811 Thanks, Dikshita On 2020-10-27 20:32, vgaro...@codeaurora.org wrote: On 2020-10-23 04:38, Tomasz Figa wrote: On Thu, Oct 22, 2020 at 6:37 AM wrote: Hi Tomasz, On 2020-10-13 19:09, Tomasz Figa

[PATCH] venus: venc: fix handlig of S_SELECTION and G_SELECTION

2020-11-03 Thread Dikshita Agarwal
- return correct width and height for G_SELECTION - update capture port wxh with rectangle wxh. - add support for HFI_PROPERTY_PARAM_UNCOMPRESSED_PLANE_ACTUAL_INFO to set stride info and chroma offset to FW. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/helpers.c

Re: [PATCH] venus: venc: add handling for VIDIOC_ENCODER_CMD

2020-10-22 Thread dikshita
expect one more buffer to be queued on src queue which needs to be handled by queueing an empty buffer with EOS flag. Without EOS, we can't handle B frames. Thanks, Dikshita On 2020-10-20 14:27, Alexandre Courbot wrote: Hi Dikshita, On Mon, Oct 19, 2020 at 11:29 PM Dikshita Agarwal wrote: Add

[PATCH] venus: venc: add handling for VIDIOC_ENCODER_CMD

2020-10-19 Thread Dikshita Agarwal
Add handling for below commands in encoder: 1. V4L2_ENC_CMD_STOP 2. V4L2_ENC_CMD_START Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.h | 9 + drivers/media/platform/qcom/venus/venc.c | 64 +++- 2 files changed, 72 insertions(+), 1

[PATCH v2] media: v4l2-ctrl: Add base layer priority id control.

2020-10-15 Thread Dikshita Agarwal
This control indicates the priority id to be applied to base layer. Signed-off-by: Dikshita Agarwal --- Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst | 9 + drivers/media/v4l2-core/v4l2-ctrls.c | 1 + include/uapi/linux/v4l2-controls.h

[PATCH v3] media: v4l2-ctrl: add control for long term reference.

2020-10-15 Thread Dikshita Agarwal
LTR (Long Term Reference) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

[PATCH] media: v4l2-ctrl: Add base layer priority id control.

2020-10-08 Thread Dikshita Agarwal
This control indicates the priority id to be applied to base layer. Signed-off-by: Dikshita Agarwal --- Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst | 9 + drivers/media/v4l2-core/v4l2-ctrls.c | 1 + include/uapi/linux/v4l2-controls.h

[PATCH 0/2] Venus: fix handling of crop and s_fmt

2020-09-24 Thread Dikshita Agarwal
Dikshita Agarwal (1): venus: venc: update output format based on capture format. Stanimir Varbanov (1): venus: venc: fix handlig of S_SELECTION and G_SELECTION drivers/media/platform/qcom/venus/venc.c | 48 ++-- 1 file changed, 27 insertions(+), 21 deletions

[PATCH 2/2] venus: venc: fix handlig of S_SELECTION and G_SELECTION

2020-09-24 Thread Dikshita Agarwal
From: Stanimir Varbanov - return correct width and height for G_SELECTION - if requested rectangle wxh doesn't match with capture port wxh adjust the rectangle to supported wxh. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc.c | 20 1 file

[PATCH 1/2] venus: venc: update output format based on capture format.

2020-09-24 Thread Dikshita Agarwal
update output format parameters when s_fmt is called on capture port as recommended in encoder spec. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/venc.c | 28 +++- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/drivers/media

[PATCH v2 2/2] media: v4l2-ctrl: Add layer wise bitrate controls for h264

2020-09-21 Thread Dikshita Agarwal
Adds bitrate control for all coding layers for h264 same as hevc. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst | 20 drivers/media/v4l2-core/v4l2-ctrls.c | 7 +++ include/uapi/linux/v4l2-controls.h

[PATCH v2 0/2] Add new controls for QP and layer bitrate

2020-09-21 Thread Dikshita Agarwal
This series adds frame specific min/max qp controls for hevc and layer wise bitrate control for h264. Chnage since v1: corrected email. Dikshita Agarwal (2): media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc media: v4l2-ctrl: Add layer wise bitrate controls for h264

[PATCH v2 1/2] media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc

2020-09-21 Thread Dikshita Agarwal
- Adds min/max qp controls for B frame for h264. - Adds min/max qp controls for I/P/B frames for hevc similar to h264. - Update valid range of mim/max qp for hevc to accommodate 10 bit. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst| 54

[PATCH v2 0/2] Add new controls for QP and layer bitrate

2020-09-21 Thread Dikshita Agarwal
This series adds frame specific min/max qp controls for hevc and layer wise bitrate control for h264. change since v1: corrected email. Dikshita Agarwal (2): media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc media: v4l2-ctrl: Add layer wise bitrate controls for h264

Re: [PATCH 0/2] Add new controls for QP and layer bitrate

2020-09-21 Thread dikshita
Hi All, Kindly ignore this patch series. Will be posting a new one. Thanks, Dikshita On 2020-09-21 18:11, Dikshita Agarwal wrote: This series adds frame specific min/max qp controls for hevc and layer wise bitrate control for h264. Dikshita Agarwal (2): media: v4l2-ctrl: Add frame-specific

[PATCH 1/2] media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc

2020-09-21 Thread Dikshita Agarwal
From: Dikshita Agarwal - Adds min/max qp controls for B frame for h264. - Adds min/max qp controls for I/P/B frames for hevc similar to h264. - Update valid range of mim/max qp for hevc to accommodate 10 bit. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst

[PATCH 2/2] media: v4l2-ctrl: Add layer wise bitrate controls for h264

2020-09-21 Thread Dikshita Agarwal
From: Dikshita Agarwal Adds bitrate control for all coding layers for h264 same as hevc. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls-codec.rst | 20 drivers/media/v4l2-core/v4l2-ctrls.c | 7 +++ include/uapi/linux

[PATCH 0/2] Add new controls for QP and layer bitrate

2020-09-21 Thread Dikshita Agarwal
This series adds frame specific min/max qp controls for hevc and layer wise bitrate control for h264. Dikshita Agarwal (2): media: v4l2-ctrl: Add frame-specific min/max qp controls for hevc media: v4l2-ctrl: Add layer wise bitrate controls for h264 .../userspace-api/media/v4l/ext-ctrls

Re: [RFC] METADATA design using V4l2 Request API

2020-09-14 Thread dikshita
not getting updated properly through __v4l2_ctrl_s_ctrl_compound(). Is there any issue when using volatile flag with compound control? Thanks, Dikshita On 2020-08-03 17:02, Hans Verkuil wrote: On 16/06/2020 15:00, diksh...@codeaurora.org wrote: Hi Nicolas, Hans, Thanks for your

Re: [PATCH v2] media: v4l2-ctrl: add control for long term reference.

2020-08-27 Thread dikshita
Hi Hans, Thanks for your comments. On 2020-08-25 15:34, Hans Verkuil wrote: On 14/08/2020 07:29, Dikshita Agarwal wrote: LTR (Long Term Reference) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames

[PATCH v2] media: v4l2-ctrl: add control for long term reference.

2020-08-13 Thread Dikshita Agarwal
LTR (Long Term Reference) frames are the frames that are encoded sometime in the past and stored in the DPB buffer list to be used as reference to encode future frames. This change adds controls to enable this feature. Signed-off-by: Dikshita Agarwal --- .../userspace-api/media/v4l/ext-ctrls

Re: [RFC PATCH 0/1] Add LTR controls

2020-08-06 Thread dikshita
On 2020-07-22 19:56, Hans Verkuil wrote: On 20/07/2020 16:33, diksh...@codeaurora.org wrote: On 2020-07-16 14:25, Hans Verkuil wrote: On 16/06/2020 20:41, diksh...@codeaurora.org wrote: Hi Hans, Nicolas, Thanks for your comments. On 2020-06-12 14:41, Hans Verkuil wrote: Hi Dikshita

Re: [RFC PATCH 0/1] Add LTR controls

2020-07-20 Thread dikshita
On 2020-07-16 14:25, Hans Verkuil wrote: On 16/06/2020 20:41, diksh...@codeaurora.org wrote: Hi Hans, Nicolas, Thanks for your comments. On 2020-06-12 14:41, Hans Verkuil wrote: Hi Dikshita, Nicolas, On 11/06/2020 16:22, Nicolas Dufresne wrote: Le jeudi 11 juin 2020 à 15:55 +0530, Dikshita

[PATCH v3] venus: move platform specific data to platform file

2020-07-14 Thread Dikshita Agarwal
From: Dikshita Agarwal Move all data specific to platform into a separate file. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/Makefile | 3 +- drivers/media/platform/qcom/venus/core.c | 20 ++- drivers/media/platform/qcom/venus/core.h | 12

[PATCH v2] venus: move platform specific data to platform file

2020-07-14 Thread Dikshita Agarwal
From: Dikshita Agarwal Move all data specific to platform into a separate file. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/Makefile | 3 +- drivers/media/platform/qcom/venus/core.c | 20 ++-- drivers/media/platform/qcom/venus/core.h | 12

Re: [RFC PATCH 0/1] Add LTR controls

2020-07-14 Thread dikshita
Hi, A gentle reminder for the review. Thanks, Dikshita On 2020-06-17 00:11, diksh...@codeaurora.org wrote: Hi Hans, Nicolas, Thanks for your comments. On 2020-06-12 14:41, Hans Verkuil wrote: Hi Dikshita, Nicolas, On 11/06/2020 16:22, Nicolas Dufresne wrote: Le jeudi 11 juin 2020 à 15:55

Re: [RFC] METADATA design using V4l2 Request API

2020-07-02 Thread dikshita
Hi, A gentle reminder for review. Thanks, Dikshita On 2020-06-23 18:17, diksh...@codeaurora.org wrote: Hi Hans, On 2020-06-16 18:30, diksh...@codeaurora.org wrote: Hi Nicolas, Hans, Thanks for your comments and sorry for the delayed response. On 2020-06-12 22:07, Nicolas Dufresne wrote

Re: [PATCH] venus: move buffer calculation to driver

2020-06-24 Thread dikshita
Hi Stanimir, A gentle reminder for the review. Thanks, Dikshita On 2020-06-09 17:24, Dikshita Agarwal wrote: Right now we get buffer requirements from fw. The call to get these requirements is a blocking call and takes a lot of time. This change calculates all the buffer requirement within

Re: [PATCH] venus: move platform specific data to platform file

2020-06-24 Thread dikshita
Hi Stanimir, A gentle reminder for the review. On 2020-05-29 12:37, Dikshita Agarwal wrote: Move all data specific to platform into a separate file. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/Makefile | 3 +- drivers/media/platform/qcom/venus/core.c

Re: [PATCH] venus: Add platform specific capabilities

2020-06-24 Thread dikshita
Hi Stanimir, A gentle reminder for the review. Thanks, Dikshita On 2020-05-29 12:48, Dikshita Agarwal wrote: Add platform specific capabilities and use them in place of firmware capabilities. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.c | 3

Re: [RFC] METADATA design using V4l2 Request API

2020-06-23 Thread dikshita
gt; > > > > > > > On 2020-05-26 16:27, Hans Verkuil wrote: > > > > > Hi Dikshita, > > > > > > > > > > My apologies for the delay, this was (mostly) due to various vacation > > > > > days. > > > > &g

Re: [PATCH v2, 2/2] media: v4l: Add Ro request api for capture queue

2020-06-23 Thread dikshita
Hi Yunfei Dong, Thanks for the patch. I believe there is one additional change required in vb2_core_queue_init() to allow ro request if the driver sets supports_ro_requests while queue_init. thanks, Dikshita On 2020-06-22 07:22, Yunfei Dong wrote: Add Read-only(Ro) request for capture queue

Re: [RFC PATCH 0/1] Add LTR controls

2020-06-16 Thread dikshita
Hi Hans, Nicolas, Thanks for your comments. On 2020-06-12 14:41, Hans Verkuil wrote: Hi Dikshita, Nicolas, On 11/06/2020 16:22, Nicolas Dufresne wrote: Le jeudi 11 juin 2020 à 15:55 +0530, Dikshita Agarwal a écrit : LTR (Long Term Reference) frames are the frames that are encoded sometime

Re: [RFC] METADATA design using V4l2 Request API

2020-06-16 Thread dikshita
sne a écrit : > > Le jeudi 28 mai 2020 à 13:24 +0200, Hans Verkuil a écrit : > > > On 28/05/2020 12:48, diksh...@codeaurora.org wrote: > > > > Hi Hans, > > > > > > > > Thanks for the review. > > > > > > > > On 2020-05-26 16:2

[RFC PATCH 0/1] Add LTR controls

2020-06-11 Thread Dikshita Agarwal
s consists of bits [0, 15]. A total of N LSB bits of this field are valid, where N is the maximum number of LTRs supported. All the other bits are invalid and should be rejected. The LSB corresponds to the LTR index 0. Bit N-1 from the LSB corresponds to the LTR index max LTR-1. Dikshi

[RFC PATCH 1/1] media: v4l2-ctrls: add control for ltr

2020-06-11 Thread Dikshita Agarwal
Add v4l2 controls for ltr Signed-off-by: Dikshita Agarwal --- drivers/media/v4l2-core/v4l2-ctrls.c | 6 ++ include/uapi/linux/v4l2-controls.h | 4 2 files changed, 10 insertions(+) diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c index

[PATCH] venus: move buffer calculation to driver

2020-06-09 Thread Dikshita Agarwal
Right now we get buffer requirements from fw. The call to get these requirements is a blocking call and takes a lot of time. This change calculates all the buffer requirement within driver and this removes the necessity for get buffer requirements call. Signed-off-by: Dikshita Agarwal

Re: [RFC] METADATA design using V4l2 Request API

2020-06-05 Thread dikshita
secure bitstream and extract them. Apart from this, there are other metadata like "histogram" which is not part of the bitstream and generated by hardware Thanks, Dikshita

[PATCH] venus: Add platform specific capabilities

2020-05-29 Thread Dikshita Agarwal
Add platform specific capabilities and use them in place of firmware capabilities. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/core.c | 3 + drivers/media/platform/qcom/venus/hfi_helper.h | 3 +- drivers/media/platform/qcom/venus/hfi_parser.c

[PATCH] venus: move platform specific data to platform file

2020-05-29 Thread Dikshita Agarwal
Move all data specific to platform into a separate file. Signed-off-by: Dikshita Agarwal --- drivers/media/platform/qcom/venus/Makefile | 3 +- drivers/media/platform/qcom/venus/core.c | 20 ++--- drivers/media/platform/qcom/venus/core.h | 10

  1   2   >