Am 09.12.2017 um 00:33 schrieb boyuan.zh...@amd.com:
From: Boyuan Zhang <boyuan.zh...@amd.com>

Remove is_idr flag, and use idr_pic_flag provided by vaapi directly

Signed-off-by: Boyuan Zhang <boyuan.zh...@amd.com>

Reviewed-by: Christian König <christian.koe...@amd.com> for this one and patch #3.

But where is patch #1 in this series?

Regards,
Christian.

---
  src/gallium/state_trackers/va/picture.c | 8 +++-----
  1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/gallium/state_trackers/va/picture.c 
b/src/gallium/state_trackers/va/picture.c
index 55ca16e..8951573 100644
--- a/src/gallium/state_trackers/va/picture.c
+++ b/src/gallium/state_trackers/va/picture.c
@@ -432,7 +432,6 @@ handleVAEncPictureParameterBufferType(vlVaDriver *drv, 
vlVaContext *context, vlV
     h264 = buf->data;
     context->desc.h264enc.frame_num = h264->frame_num;
     context->desc.h264enc.not_referenced = false;
-   context->desc.h264enc.is_idr = (h264->pic_fields.bits.idr_pic_flag == 1);
     context->desc.h264enc.pic_order_cnt = h264->CurrPic.TopFieldOrderCnt;
     if (context->desc.h264enc.gop_cnt == 0)
        context->desc.h264enc.i_remain = context->gop_coeff;
@@ -451,7 +450,7 @@ handleVAEncPictureParameterBufferType(vlVaDriver *drv, 
vlVaContext *context, vlV
                       UINT_TO_PTR(h264->CurrPic.picture_id),
                       UINT_TO_PTR(h264->frame_num));
- if (context->desc.h264enc.is_idr)
+   if (h264->pic_fields.bits.idr_pic_flag == 1)
        context->desc.h264enc.picture_type = PIPE_H264_ENC_PICTURE_TYPE_IDR;
     else
        context->desc.h264enc.picture_type = PIPE_H264_ENC_PICTURE_TYPE_P;
@@ -493,10 +492,9 @@ handleVAEncSliceParameterBufferType(vlVaDriver *drv, 
vlVaContext *context, vlVaB
     else if (h264->slice_type == 0)
        context->desc.h264enc.picture_type = PIPE_H264_ENC_PICTURE_TYPE_P;
     else if (h264->slice_type == 2) {
-      if (context->desc.h264enc.is_idr){
-         context->desc.h264enc.picture_type = PIPE_H264_ENC_PICTURE_TYPE_IDR;
+      if (context->desc.h264enc.picture_type == PIPE_H264_ENC_PICTURE_TYPE_IDR)
           context->desc.h264enc.idr_pic_id++;
-          } else
+          else
           context->desc.h264enc.picture_type = PIPE_H264_ENC_PICTURE_TYPE_I;
     } else
        context->desc.h264enc.picture_type = PIPE_H264_ENC_PICTURE_TYPE_SKIP;

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

Reply via email to