On Sat, Dec 7, 2013 at 8:11 AM, Martin Peres <[email protected]> wrote: > From: Martin Peres <[email protected]> > > This patch is a follow-up from Ilia Mirkin's enable H.264 patch which > solves the problem that prevented MPEG-4 videos to play correctly. > > Tested on an nva3.
I might reword this as """ VP3/VP4 now work on all the codecs they are supposed to, remove old restriction. Tested on NVA3 """ Or something like that... > Signed-off-by: Martin Peres <[email protected]> > Tested-by: Martin Peres <[email protected]> > Cc: "10.0" <[email protected]> > --- > src/gallium/drivers/nouveau/nouveau_vp3_video.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/src/gallium/drivers/nouveau/nouveau_vp3_video.c > b/src/gallium/drivers/nouveau/nouveau_vp3_video.c > index 2f4196c..0843b78 100644 > --- a/src/gallium/drivers/nouveau/nouveau_vp3_video.c > +++ b/src/gallium/drivers/nouveau/nouveau_vp3_video.c > @@ -362,11 +362,6 @@ nouveau_vp3_screen_get_video_param(struct pipe_screen > *pscreen, > enum pipe_video_format codec = u_reduce_video_profile(profile); > switch (param) { > case PIPE_VIDEO_CAP_SUPPORTED: > - /* For now, mpeg4 doesn't work on pre-nvc0. */ > - if (chipset < 0xc0) > - return codec == PIPE_VIDEO_FORMAT_MPEG12 || > - codec == PIPE_VIDEO_FORMAT_VC1 || > - codec == PIPE_VIDEO_FORMAT_MPEG4_AVC; > /* In the general case, this should work, once the pre-nvc0 problems > are > * resolved. */ Update this comment to reflect reality. Or just remove it. With that change, Reviewed-by: Ilia Mirkin <[email protected]> Or I can fold this into my h264 change if you don't want to send a v2, up to you. > return profile >= PIPE_VIDEO_PROFILE_MPEG1 && ( > -- > 1.8.4.2 > > _______________________________________________ > Nouveau mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/nouveau _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
