[Libva] [Libva-intel-driver][PATCH v2] Remove unused fields in struct encode_state

It also fixes a wrong assert() in gen8_mfc.c v2: Correct condition code when removing used fields Signed-off-by: Xiang, Haihao --- src/gen8_mfc.c | 2 +- src/i965_drv_video.c | 29 +++-- src/i965_drv_video.h | 6 -- 3 files changed, 4 insertions(+), 33 deleti

[Libva] [Libva-intel-driver][PATCH] Set cost to 0 for CHROMA INTRA mode

Commit 1cd6795 causes quality drop for U/V plane. Reported-by: Wang, Fei W Signed-off-by: Xiang, Haihao Tested-by: Wang, Fei W --- src/gen6_mfc_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gen6_mfc_common.c b/src/gen6_mfc_common.c index b0342da..9f041d8 1006

[Libva] [Libva-intel-driver][PATCH] Remove unused fields in struct encode_state

It also fixes a wrong assert() in gen8_mfc.c Signed-off-by: Xiang, Haihao --- src/gen8_mfc.c | 2 +- src/i965_drv_video.c | 29 +++-- src/i965_drv_video.h | 6 -- 3 files changed, 4 insertions(+), 33 deletions(-) diff --git a/src/gen8_mfc.c b/src/gen8_mfc.c i

[Libva] [PATCH v2][libva-intel-driver] i965_encoder: remove double check for VAStatus result

after creating underlying surface there's a double check on the VAStatus result. Replace it with ASSERT_RET. Signed-off-by: Daniel Charles --- src/i965_encoder.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/i965_encoder.c b/src/i965_encoder.c index beea811..d909846 100

Re: [Libva] [PATCH][libva-intel-driver] i965_encoder: remove double check for VAStatus result

On Mon, 2016-08-15 at 10:49 -0700, Daniel Charles wrote: > after creating underlying surface there's a double check on the > VAStatus result. Removed assert as it was part of the code when > the function didn't return vaStatus > > Signed-off-by: Daniel Charles > --- >  src/i965_encoder.c | 1 - >

[Libva] [PATCH][libva-intel-driver] i965_encoder: remove double check for VAStatus result

after creating underlying surface there's a double check on the VAStatus result. Removed assert as it was part of the code when the function didn't return vaStatus Signed-off-by: Daniel Charles --- src/i965_encoder.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/i965_encoder.c b/src/i96

Re: [Libva] [PATCH] i965_device_info: add CBR and VBR rate control flags for vdenc

On 12.08.2016 20:19, Sean V Kelley wrote: On Fri, 2016-08-12 at 16:20 +0300, Sreerenj wrote: On 12.08.2016 14:20, Sreerenj wrote: On 11.08.2016 19:47, Sean V Kelley wrote: On Thu, 2016-08-11 at 14:55 +0300, sreerenj.balachand...@intel.co m wrote: From: Sreerenj Balachandran Low power e

[Libva] [PATCH] Encoding: Add ROI example

From: Zhao Yakui v1: add --roi-test for test only for ROI. default only one region(0,0,120,120) has been test. v2: add ROI region size check to ensure region width/height < frame_width/4 or frame_height/4 if fram width/height < 120 add ROI attrib check Signed-off-by: Zhao Yakui Signed-off-by:

Re: [Libva] Driver stuck on picture encode

Hi Lionel, The driver should give up if it can't reach the size. I can't reproduce your issue. Could you provide a test case? It would be better if you can file a bug to track your issue. Thanks Haihao > Hi, > > Playing around with the h264 encoder on Haswell, I've noticed that I > can > get

Re: [Libva] [PATCH][libva-intel-driver] vp9encoder: encoder to handle properly CSC on input surface

Thanks for your patch, applied. > VP9 encoder was not checking for the yuv surface fourcc provided as > the input. > If the format is I420, the driver creates an underlying surface where > the input > is converted to NV12. > > The underlying temporary surface was not used properly by the > vme_p

Re: [Libva] [PATCH V4: 00/11]Add ROI feature on Gen7/Gen7.5/Gen8/Gen9 and scaling list features.

Applied this series of patches and updated the dependency on VA-API. Thanks Haihao > this add ROI feature on  Gen7/Gen7.5/Gen8/Gen9 and scaling list > features. > > v3: > do style change accordingly; > remove the frame qp delta related patch as interface will be > redefined. > remove the warnin