On 12/22/16, 5:08 PM, "Libva on behalf of Mark Thompson"
wrote:
Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
Lgtm,
Sean
Signed-off-by: Mark Thompson
---
Changes for the series:
* Derive numerator and denominator from the correct parts of the
: [Libva] [PATCH 1/4] i965_encoder: consistently represent framerate
>as a fraction
>
>Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
>
>Signed-off-by: Mark Thompson
>---
>Changes for the series:
>* Derive numerator and denominator from the correct p
Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
Signed-off-by: Mark Thompson
---
Changes for the series:
* Derive numerator and denominator from the correct parts of the framerate
parameter (see also earlier patch fixing the comment in va.h).
* Make VP8 HRD buffer size deriva
> On 21/12/16 04:21, Xiang, Haihao wrote:
> >
> > > Update references in both H.264 encoders (gen6_mfc and
> > > gen9_vdenc).
> > >
> > > Signed-off-by: Mark Thompson
> > > ---
> > > New version.
> > >
> > > Changes for the whole series:
> > > * Use a single field for framerate (adding a new s
On 21/12/16 04:21, Xiang, Haihao wrote:
>
>> Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
>>
>> Signed-off-by: Mark Thompson
>> ---
>> New version.
>>
>> Changes for the whole series:
>> * Use a single field for framerate (adding a new struct to represent
>> it), as recomme
> Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
>
> Signed-off-by: Mark Thompson
> ---
> New version.
>
> Changes for the whole series:
> * Use a single field for framerate (adding a new struct to represent
> it), as recommended by Haihao.
> * Fix some missed cases where b
Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
Signed-off-by: Mark Thompson
---
New version.
Changes for the whole series:
* Use a single field for framerate (adding a new struct to represent it), as
recommended by Haihao.
* Fix some missed cases where bitrate was read from
> On 15/12/16 08:54, Xiang, Haihao wrote:
> >
> > Hi Mark,
> >
> > Thanks for your patch, is there any benefit to use a fraction?
> > using
> > 100 as framerate_den works well in the driver.
>
> Everything the driver interacts with directly, on both sides, uses a
> fraction:
>
> * In VAAPI, VA
I'm fine with this change and it adds consistency.
lgtm
Thanks,
Sean
On Thu, Dec 15, 2016 at 4:26 AM, Mark Thompson wrote:
> On 15/12/16 08:54, Xiang, Haihao wrote:
> >
> > Hi Mark,
> >
> > Thanks for your patch, is there any benefit to use a fraction? using
> > 100 as framerate_den works wel
On 15/12/16 08:54, Xiang, Haihao wrote:
>
> Hi Mark,
>
> Thanks for your patch, is there any benefit to use a fraction? using
> 100 as framerate_den works well in the driver.
Everything the driver interacts with directly, on both sides, uses a fraction:
* In VAAPI, VAEncMiscParameterFrameRate c
Hi Mark,
Thanks for your patch, is there any benefit to use a fraction? using
100 as framerate_den works well in the driver.
Regards
Haihao
> Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
>
> Signed-off-by: Mark Thompson
> ---
> src/gen6_mfc_common.c | 11 ++
>
Update references in both H.264 encoders (gen6_mfc and gen9_vdenc).
Signed-off-by: Mark Thompson
---
src/gen6_mfc_common.c | 11 ++
src/gen9_vdenc.c | 29 ++--
src/gen9_vdenc.h | 3 ++-
src/i965_encoder.c| 61 +---
12 matches
Mail list logo