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 <ckoenig.leichtzumer...@gmail.com> Sent: September 26, 2018 4:05 AM To: Zhang, Boyuan <boyuan.zh...@amd.com>; 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 <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
