Hello Kamil,
Sorry for the delayed reply. I'm afraid this doesnt fix the whole issue.
Still there is some problem with S_CTRL when used with custom controls.
when my application call s_ctrl with
V4L2_CID_MPEG_MFC51_VIDEO_DECODER_H264_DISPLAY_DELAY
the call finally lands on V4L2_CID_MPEG_VIDEO_DECODER_MPEG4_DEBLOCK_FILTER in
driver (The set value is also lost).
I think this is because of corruption of the master or cluster in
v4l2-ioctl.c/ctrl.c and pointing to the standard controls master.
instead of the custom control master.
Kindly, let me know your opinion or any clues on this.
Thanks & Regards
Naveen Krishna
------- Original Message -------
Sender : Kamil Debski<k.deb...@samsung.com> Software Engineer/Poland R&D
Center-Linux Platform (SSD)/Samsung Electronics
Date : Jun 15, 2012 13:51 (GMT+05:00)
Title : [PATCH] s5p-mfc: Fix setting controls
Fixed s_ctrl function when setting the following controls:
- V4L2_CID_MPEG_VIDEO_DECODER_MPEG4_DEBLOCK_FILTER
- V4L2_CID_MPEG_MFC51_VIDEO_DECODER_H264_DISPLAY_DELAY
Signed-off-by: Kamil Debski <k.deb...@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
---
drivers/media/video/s5p-mfc/s5p_mfc_dec.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/video/s5p-mfc/s5p_mfc_dec.c
b/drivers/media/video/s5p-mfc/s5p_mfc_dec.c
index e1ebc76..eaab13e 100644
--- a/drivers/media/video/s5p-mfc/s5p_mfc_dec.c
+++ b/drivers/media/video/s5p-mfc/s5p_mfc_dec.c
@@ -639,13 +639,13 @@ static int s5p_mfc_dec_s_ctrl(struct v4l2_ctrl *ctrl)
switch (ctrl->id) {
case V4L2_CID_MPEG_MFC51_VIDEO_DECODER_H264_DISPLAY_DELAY:
- ctx->loop_filter_mpeg4 = ctrl->val;
+ ctx->display_delay = ctrl->val;
break;
case V4L2_CID_MPEG_MFC51_VIDEO_DECODER_H264_DISPLAY_DELAY_ENABLE:
ctx->display_delay_enable = ctrl->val;
break;
case V4L2_CID_MPEG_VIDEO_DECODER_MPEG4_DEBLOCK_FILTER:
- ctx->display_delay = ctrl->val;
+ ctx->loop_filter_mpeg4 = ctrl->val;
break;
case V4L2_CID_MPEG_VIDEO_DECODER_SLICE_INTERFACE:
ctx->slice_interface = ctrl->val;
--
1.7.0.4
<p> </p><p> </p>Thanks & Best Regards,
Naveen Krishna Ch
SE @
Samsung-B.LABN‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·¥Š{±™çbj)í…æèw* jg¬±¨ ¶‰šŽŠÝ¢j/�êäz¹Þ–Šà2ŠÞ™¨èÚ&¢)ß¡«a¶Úþø ®G«�éh®æj:+v‰¨Šwè†Ù¥
