Re: [Libva] [Libva-intel-driver][PATCH] Set default framerate to 30fps if user doesn't set the corresponding sequence parameter

2016-10-04 Thread Sean V Kelley
On Wed, 2016-09-28 at 09:04 +0800, Zhao Yakui wrote: > On 09/27/2016 10:49 PM, Xiang, Haihao wrote: > > > > User can update framerate using VAEncMiscParameterTypeFrameRate > > buffer later > > It looks good to me. > > Add: Reviewed-by: Zhao Yakui Sorry Yakui, didn't see your reply until after

Re: [Libva] [Libva-intel-driver][PATCH] Set default framerate to 30fps if user doesn't set the corresponding sequence parameter

2016-10-04 Thread Sean V Kelley
On Tue, 2016-09-27 at 22:49 +0800, Xiang, Haihao wrote: > User can update framerate using VAEncMiscParameterTypeFrameRate > buffer later I am okay with this.  lgtm, applied. Thanks, Sean > > Signed-off-by: Xiang, Haihao > --- >  src/i965_encoder.c | 7 ++- >  1 file changed, 6 insertions(

Re: [Libva] [PATCH intel-driver] test: add create surface test

2016-10-04 Thread Sean V Kelley
On Mon, 2016-10-03 at 13:21 -0700, U. Artie Eoff wrote: > Add test to verify correct result for create surfaces with > supported and unsupported pixel formats. > > Currently the IYUV pixel format case fails... see > https://bugs.freedesktop.org/show_bug.cgi?id=98033 I've confirmed the failure wi

Re: [Libva] libyami 1.0.0 release

2016-10-04 Thread Sean V Kelley
ArchLinux packages have been updated. Note that these are currently in AUR. https://aur.archlinux.org/packages/libyami/ https://aur.archlinux.org/packages/libyami-utils/ If you would like to see this get promoted out of AUR, I encourage you to register with AUR on ArchLinux.org and VOTE! Thank