mm-video-v4l2: venc: Fix 601 to 709 color conversion issue

The color conversion flag - V4L2_MSM_BUF_FLAG_YUV_601_709_CLAMP,
needs to be set for every ETB and not just the first time.

CRs-Fixed: 958431
Change-Id: I82b3362ca8000cae1acd2c11452ade7a6f90d7d4
This commit is contained in:
Chinmay Sawarkar 2016-12-09 16:05:41 -08:00 committed by Gerrit - the friendly Code Review server
parent 826746ffc8
commit d5d7b14458

View file

@ -3771,16 +3771,22 @@ bool venc_dev::venc_empty_buf(void *buffer, void *pmem_data_buf, unsigned index,
DEBUG_PRINT_ERROR("ERROR: venc_etb: handle is NULL"); DEBUG_PRINT_ERROR("ERROR: venc_etb: handle is NULL");
return false; return false;
} }
int usage = 0;
usage = MetaBufferUtil::getIntAt(hnd, 0, MetaBufferUtil::INT_USAGE);
usage = usage > 0 ? usage : 0;
if ((usage & private_handle_t::PRIV_FLAGS_ITU_R_601_FR)
&& (is_csc_enabled)) {
buf.flags |= V4L2_MSM_BUF_FLAG_YUV_601_709_CLAMP;
}
if (!streaming[OUTPUT_PORT] && !(m_sVenc_cfg.inputformat == V4L2_PIX_FMT_RGB32 || if (!streaming[OUTPUT_PORT] && !(m_sVenc_cfg.inputformat == V4L2_PIX_FMT_RGB32 ||
m_sVenc_cfg.inputformat == V4L2_PIX_FMT_RGBA8888_UBWC)) { m_sVenc_cfg.inputformat == V4L2_PIX_FMT_RGBA8888_UBWC)) {
int usage = 0;
struct v4l2_format fmt; struct v4l2_format fmt;
OMX_COLOR_FORMATTYPE color_format = (OMX_COLOR_FORMATTYPE)QOMX_COLOR_FORMATYUV420PackedSemiPlanar32m; OMX_COLOR_FORMATTYPE color_format = (OMX_COLOR_FORMATTYPE)QOMX_COLOR_FORMATYUV420PackedSemiPlanar32m;
color_format = (OMX_COLOR_FORMATTYPE)MetaBufferUtil::getIntAt(hnd, 0, MetaBufferUtil::INT_COLORFORMAT); color_format = (OMX_COLOR_FORMATTYPE)MetaBufferUtil::getIntAt(hnd, 0, MetaBufferUtil::INT_COLORFORMAT);
usage = MetaBufferUtil::getIntAt(hnd, 0, MetaBufferUtil::INT_USAGE);
usage = usage > 0 ? usage : 0;
memset(&fmt, 0, sizeof(fmt)); memset(&fmt, 0, sizeof(fmt));
if (usage & private_handle_t::PRIV_FLAGS_ITU_R_709 || if (usage & private_handle_t::PRIV_FLAGS_ITU_R_709 ||
@ -3790,10 +3796,17 @@ bool venc_dev::venc_empty_buf(void *buffer, void *pmem_data_buf, unsigned index,
} }
if (usage & private_handle_t::PRIV_FLAGS_ITU_R_601_FR) { if (usage & private_handle_t::PRIV_FLAGS_ITU_R_601_FR) {
if (is_csc_enabled) { if (is_csc_enabled) {
buf.flags = V4L2_MSM_BUF_FLAG_YUV_601_709_CLAMP; struct v4l2_control control;
fmt.fmt.pix_mp.colorspace = V4L2_COLORSPACE_REC709; control.id = V4L2_CID_MPEG_VIDC_VIDEO_VPE_CSC;
venc_set_colorspace(MSM_VIDC_BT709_5, 1, control.value = V4L2_CID_MPEG_VIDC_VIDEO_VPE_CSC_ENABLE;
MSM_VIDC_TRANSFER_BT709_5, MSM_VIDC_MATRIX_BT_709_5); if (ioctl(m_nDriver_fd, VIDIOC_S_CTRL, &control)) {
DEBUG_PRINT_ERROR("venc_empty_buf: Failed to set VPE CSC for 601_to_709");
} else {
DEBUG_PRINT_INFO("venc_empty_buf: Will convert 601-FR to 709");
fmt.fmt.pix_mp.colorspace = V4L2_COLORSPACE_REC709;
venc_set_colorspace(MSM_VIDC_BT709_5, 1,
MSM_VIDC_TRANSFER_BT709_5, MSM_VIDC_MATRIX_BT_709_5);
}
} else { } else {
venc_set_colorspace(MSM_VIDC_BT601_6_525, 1, venc_set_colorspace(MSM_VIDC_BT601_6_525, 1,
MSM_VIDC_TRANSFER_601_6_525, MSM_VIDC_MATRIX_601_6_525); MSM_VIDC_TRANSFER_601_6_525, MSM_VIDC_MATRIX_601_6_525);
@ -3808,10 +3821,6 @@ bool venc_dev::venc_empty_buf(void *buffer, void *pmem_data_buf, unsigned index,
m_sVenc_cfg.inputformat = V4L2_PIX_FMT_NV12_UBWC; m_sVenc_cfg.inputformat = V4L2_PIX_FMT_NV12_UBWC;
} }
if (usage & private_handle_t::PRIV_FLAGS_ITU_R_709) {
buf.flags = V4L2_MSM_BUF_FLAG_YUV_601_709_CLAMP;
}
if (color_format > 0 && !venc_set_color_format(color_format)) { if (color_format > 0 && !venc_set_color_format(color_format)) {
DEBUG_PRINT_ERROR("Failed setting color format in Camerasource %lx", m_sVenc_cfg.inputformat); DEBUG_PRINT_ERROR("Failed setting color format in Camerasource %lx", m_sVenc_cfg.inputformat);
return false; return false;
@ -4129,6 +4138,7 @@ bool venc_dev::venc_empty_batch(OMX_BUFFERHEADERTYPE *bufhdr, unsigned index)
} }
for (int i = 0; i < numBufs; ++i) { for (int i = 0; i < numBufs; ++i) {
int v4l2Id = v4l2Ids[i]; int v4l2Id = v4l2Ids[i];
int usage = 0;
memset(&buf, 0, sizeof(buf)); memset(&buf, 0, sizeof(buf));
memset(&plane, 0, sizeof(plane)); memset(&plane, 0, sizeof(plane));
@ -4145,6 +4155,14 @@ bool venc_dev::venc_empty_batch(OMX_BUFFERHEADERTYPE *bufhdr, unsigned index)
buf.m.planes = plane; buf.m.planes = plane;
buf.length = num_input_planes; buf.length = num_input_planes;
usage = MetaBufferUtil::getIntAt(hnd, i, MetaBufferUtil::INT_USAGE);
usage = usage > 0 ? usage : 0;
if ((usage & private_handle_t::PRIV_FLAGS_ITU_R_601_FR)
&& (is_csc_enabled)) {
buf.flags |= V4L2_MSM_BUF_FLAG_YUV_601_709_CLAMP;
}
extra_idx = EXTRADATA_IDX(num_input_planes); extra_idx = EXTRADATA_IDX(num_input_planes);
if (extra_idx && (extra_idx < VIDEO_MAX_PLANES)) { if (extra_idx && (extra_idx < VIDEO_MAX_PLANES)) {