Re: [Mesa-dev] [PATCH] st/va: Support fractional framerate in misc parameter

2017-03-01 Thread Mark Thompson 
On 27/01/17 22:03, Mark Thompson wrote:
> Signed-off-by: Mark Thompson 
> ---
> See  - this is a 
> recent addition, but is backwards-compatible.
> 
> Depends on Andy's patch 
>  
> for actual support for fractional framerate.
> 
>  src/gallium/state_trackers/va/picture.c | 8 +++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/src/gallium/state_trackers/va/picture.c 
> b/src/gallium/state_trackers/va/picture.c
> index a024437bff..8073ad8906 100644
> --- a/src/gallium/state_trackers/va/picture.c
> +++ b/src/gallium/state_trackers/va/picture.c
> @@ -357,7 +357,13 @@ static VAStatus
>  handleVAEncMiscParameterTypeFrameRate(vlVaContext *context, 
> VAEncMiscParameterBuffer *misc)
>  {
> VAEncMiscParameterFrameRate *fr = (VAEncMiscParameterFrameRate 
> *)misc->data;
> -   context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate;
> +   if (fr->framerate & 0x) {
> +  context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate   & 
> 0x;
> +  context->desc.h264enc.rate_ctrl.frame_rate_den = fr->framerate >> 16 & 
> 0x;
> +   } else {
> +  context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate;
> +  context->desc.h264enc.rate_ctrl.frame_rate_den = 1;
> +   }
> return VA_STATUS_SUCCESS;
>  }
>  
> 

Ping this patch and 
.

Thanks,

- Mark
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] st/va: Support fractional framerate in misc parameter

2017-01-30 Thread Andy Furniss 

Mark Thompson wrote:

Signed-off-by: Mark Thompson 
---
See  - this is a recent 
addition, but is backwards-compatible.

Depends on Andy's patch 
 for 
actual support for fractional framerate.


Seems good testing so far, in the sense that it doesn't regress anything.

Added CCs.



  src/gallium/state_trackers/va/picture.c | 8 +++-
  1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/va/picture.c 
b/src/gallium/state_trackers/va/picture.c
index a024437bff..8073ad8906 100644
--- a/src/gallium/state_trackers/va/picture.c
+++ b/src/gallium/state_trackers/va/picture.c
@@ -357,7 +357,13 @@ static VAStatus
  handleVAEncMiscParameterTypeFrameRate(vlVaContext *context, 
VAEncMiscParameterBuffer *misc)
  {
 VAEncMiscParameterFrameRate *fr = (VAEncMiscParameterFrameRate 
*)misc->data;
-   context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate;
+   if (fr->framerate & 0x) {
+  context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate   & 
0x;
+  context->desc.h264enc.rate_ctrl.frame_rate_den = fr->framerate >> 16 & 
0x;
+   } else {
+  context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate;
+  context->desc.h264enc.rate_ctrl.frame_rate_den = 1;
+   }
 return VA_STATUS_SUCCESS;
  }




___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] st/va: Support fractional framerate in misc parameter

2017-01-27 Thread Mark Thompson 
Signed-off-by: Mark Thompson 
---
See  - this is a recent 
addition, but is backwards-compatible.

Depends on Andy's patch 
 for 
actual support for fractional framerate.

 src/gallium/state_trackers/va/picture.c | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/va/picture.c 
b/src/gallium/state_trackers/va/picture.c
index a024437bff..8073ad8906 100644
--- a/src/gallium/state_trackers/va/picture.c
+++ b/src/gallium/state_trackers/va/picture.c
@@ -357,7 +357,13 @@ static VAStatus
 handleVAEncMiscParameterTypeFrameRate(vlVaContext *context, 
VAEncMiscParameterBuffer *misc)
 {
VAEncMiscParameterFrameRate *fr = (VAEncMiscParameterFrameRate *)misc->data;
-   context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate;
+   if (fr->framerate & 0x) {
+  context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate   & 
0x;
+  context->desc.h264enc.rate_ctrl.frame_rate_den = fr->framerate >> 16 & 
0x;
+   } else {
+  context->desc.h264enc.rate_ctrl.frame_rate_num = fr->framerate;
+  context->desc.h264enc.rate_ctrl.frame_rate_den = 1;
+   }
return VA_STATUS_SUCCESS;
 }
 
-- 
2.11.0

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev