Re: [Mesa-dev] [PATCH] vl: reorder H264 profiles
Yes, I was thinking either using a switch helper function or doing this 1 line change since there will be no more profile to be added. But no problem, I agree that using switch helper function is a safer way. New patch just sent. Regards, Boyuan -Original Message- From: Christian König Sent: September 26, 2018 4:05 AM To: Zhang, Boyuan ; mesa-dev@lists.freedesktop.org Subject: Re: [PATCH] vl: reorder H264 profiles Am 26.09.2018 um 00:11 schrieb boyuan.zh...@amd.com: > From: Boyuan Zhang > > 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 > --- > 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
Re: [Mesa-dev] [PATCH] vl: reorder H264 profiles
Am 26.09.2018 um 00:11 schrieb boyuan.zh...@amd.com: From: Boyuan Zhang 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 --- 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
[Mesa-dev] [PATCH] vl: reorder H264 profiles
From: Boyuan Zhang 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 Signed-off-by: Boyuan Zhang --- 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, -- 2.17.1 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev