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

Fix the wrong h264 profiles order. Previously, the constrained baseline was
added in between baseline and main profiles, which breaked the logic in
radeon/vce when converting from pipe_video_profile to profile_idc

I think it would be better to use a switch/case in radeon/vce or even better make a helper function which converts between PIPE_VIDEO_PROFILE_MPEG4_AVC_* and the profile_idc from the specification.

Christian.


Signed-off-by: Boyuan Zhang <boyuan.zh...@amd.com>
---
  src/gallium/include/pipe/p_video_enums.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/include/pipe/p_video_enums.h 
b/src/gallium/include/pipe/p_video_enums.h
index b5b8b06228..260f47ea8a 100644
--- a/src/gallium/include/pipe/p_video_enums.h
+++ b/src/gallium/include/pipe/p_video_enums.h
@@ -55,8 +55,8 @@ enum pipe_video_profile
     PIPE_VIDEO_PROFILE_VC1_SIMPLE,
     PIPE_VIDEO_PROFILE_VC1_MAIN,
     PIPE_VIDEO_PROFILE_VC1_ADVANCED,
-   PIPE_VIDEO_PROFILE_MPEG4_AVC_BASELINE,
     PIPE_VIDEO_PROFILE_MPEG4_AVC_CONSTRAINED_BASELINE,
+   PIPE_VIDEO_PROFILE_MPEG4_AVC_BASELINE,
     PIPE_VIDEO_PROFILE_MPEG4_AVC_MAIN,
     PIPE_VIDEO_PROFILE_MPEG4_AVC_EXTENDED,
     PIPE_VIDEO_PROFILE_MPEG4_AVC_HIGH,

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

Reply via email to