Re: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate

2016-12-08 Thread Mark Thompson
On 08/12/16 01:13, Zhao Yakui wrote: > On 12/07/2016 01:48 PM, Xiang, Haihao wrote: >> Now we have the same way for frame rate for each codec in i965_encoder.c, it >> would be better to use the result directly > > Hi, Mark > > Can you refresh your patch based on Haihao's suggestion? > No

Re: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate

2016-12-07 Thread Zhao Yakui
-boun...@lists.freedesktop.org] On Behalf Of Mark Thompson Sent: Tuesday, December 6, 2016 2:02 AM To: libva@lists.freedesktop.org Subject: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate Signed-off-by: Mark Thompson --- Tested on Kaby Lake. Someone who has access to the manuals s

Re: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate

2016-12-07 Thread Sean V Kelley
Message- > >From: Libva [mailto:libva-boun...@lists.freedesktop.org] On Behalf Of > Mark > >Thompson > >Sent: Tuesday, December 6, 2016 2:02 AM > >To: libva@lists.freedesktop.org > >Subject: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate > > > &

Re: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate

2016-12-06 Thread Xiang, Haihao
2:02 AM >To: libva@lists.freedesktop.org >Subject: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate > >Signed-off-by: Mark Thompson >--- >Tested on Kaby Lake. Someone who has access to the manuals should make >sure that the framerate numerator/denominator actually

Re: [Libva] [PATCH 3/3] VP9 encode: support fractional framerate

2016-12-05 Thread Zhao Yakui
On 12/06/2016 02:02 AM, Mark Thompson wrote: Signed-off-by: Mark Thompson Good work. This patch looks good to me. Add: Reviewed-by: Zhao Yakui Thanks --- Tested on Kaby Lake. Someone who has access to the manuals should make sure that the framerate numerator/denominator actually works t

[Libva] [PATCH 3/3] VP9 encode: support fractional framerate

2016-12-05 Thread Mark Thompson
Signed-off-by: Mark Thompson --- Tested on Kaby Lake. Someone who has access to the manuals should make sure that the framerate numerator/denominator actually works there as I am guessing it does. Removes the frame_rate field in struct gen9_vp9_brc_curbe_param, because there is no longer a se