Re: [Libva] [PATCH 0/2] add user specified tiling/stride support, clean up assert

2014-03-18 Thread Zhao, Halley
Thanks for your comments. The 2nd patch is so big that it is blocked by mail man, so I sent to some stake holders for review. Your VA_INTEL_DEBUG=asserts sounds good, we can consider it. -Original Message- From: Gwenole Beauchesne [mailto:gb.de...@gmail.com] Sent: Wednesday, March 19,

Re: [Libva] [PATCH] VPP: Fix the typo error of "VV16"

2014-03-18 Thread Zhao Yakui
On Tue, 2014-03-18 at 23:01 -0600, Gwenole Beauchesne wrote: > 2014-03-18 9:35 GMT+01:00 Zhao Yakui : > > Thank Gwenole for finding this typo error which is caused by > > the commit 2b5fad11a5c12d3c6ffbef15c02449a3b4e90b98. > > OK, but please also schedule another patch where you have > programmat

Re: [Libva] [PATCH] VPP: Fix the typo error of "VV16"

2014-03-18 Thread Gwenole Beauchesne
2014-03-18 9:35 GMT+01:00 Zhao Yakui : > Thank Gwenole for finding this typo error which is caused by > the commit 2b5fad11a5c12d3c6ffbef15c02449a3b4e90b98. OK, but please also schedule another patch where you have programmatically (script, command) replaced any instance of VA_FOURCC('A','B','C','

Re: [Libva] [PATCH 0/2] add user specified tiling/stride support, clean up assert

2014-03-18 Thread Gwenole Beauchesne
Hi, 2014-03-14 10:19 GMT+01:00 Zhao, Halley : > when I implements feature for user specified tiling/stride support, > haihao mentioned that I'd better 'return fail' instead of assert(). > so, I tried to clean up the assert in i965_drv_video.c as well. The assert_ret() patch did not appear on the

Re: [Libva] [ANNOUNCE] libva-1.3.0.pre1

2014-03-18 Thread Gwenole Beauchesne
Hi, 2014-03-17 5:40 GMT+01:00 Xiang, Haihao : > libva-1.3.0 is planned for release soon. Please report any critical > issue with Oh, well, 1.2.2 was fine actually. If I had mentioned 0.35.0 in another patch, that was because of the planned interface addition. This one could have remained at 0.3

Re: [Libva] [PATCH 1/2] va: User specified tiling and stride support.

2014-03-18 Thread Zhao Yakui
On Mon, 2014-03-17 at 00:09 -0600, Zhao, Halley wrote: > Thanks Yakui。 > LGTM. this will be pushed. thanks. Yakui > > -Original Message- > From: Zhao, Yakui > Sent: Monday, March 17, 2014 10:25 AM > To: Zhao, Halley > Cc: libva@lists.freedesktop.org > Subject: Re: [Libva] [PATCH 1/2

Re: [Libva] [PATCH] VPP: Fix the typo error of "VV16"

2014-03-18 Thread Zhao Yakui
On Tue, 2014-03-18 at 02:35 -0600, Zhao, Yakui wrote: > Thank Gwenole for finding this typo error which is caused by > the commit 2b5fad11a5c12d3c6ffbef15c02449a3b4e90b98. this will be pushed. thanks. Yakui > > Signed-off-by: Zhao Yakui > --- > src/gen8_post_processing.c | 2 +- > src/i965_

[Libva] [PATCH] VPP: Fix the typo error of "VV16"

2014-03-18 Thread Zhao Yakui
Thank Gwenole for finding this typo error which is caused by the commit 2b5fad11a5c12d3c6ffbef15c02449a3b4e90b98. Signed-off-by: Zhao Yakui --- src/gen8_post_processing.c | 2 +- src/i965_post_processing.c | 8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/gen8_post

Re: [Libva] [PATCH Libva-intel-driver 4/4] Add the csc conversion from YV16 to NV12

2014-03-18 Thread Zhao Yakui
On Mon, 2014-03-17 at 23:06 -0600, Gwenole Beauchesne wrote: > Hi, > > 2014-03-18 1:44 GMT+01:00 Wu, Zhiwen : > > Looks good to me, tested on baytraili. > > Interesting, how? The patch doesn't do what it tells to do in several places: > > >>-Original Message- > >>From: Libva [mailto:libv