Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-05 Thread Andy Furniss
Christian König wrote: Am 04.04.2017 um 18:40 schrieb Andy Furniss: Well it's a tricky situation with cabac which would be a shame to loose. Yeah, completely agree. Currently I guess it's a bit strange advertising main/high - but then others do without having mbaff support As noted be

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-05 Thread Christian König
Am 04.04.2017 um 18:40 schrieb Andy Furniss: Well it's a tricky situation with cabac which would be a shame to loose. Yeah, completely agree. Currently I guess it's a bit strange advertising main/high - but then others do without having mbaff support As noted before mbaff isn't so much

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread Andy Furniss
Andy Furniss wrote: Well it's a tricky situation with cabac which would be a shame to loose. Currently I guess it's a bit strange advertising main/high - but then others do without having mbaff support As it stands with cabac on I can make (what I think is) a perfectly legal stream with ff

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread Andy Furniss
Andy Furniss wrote: Well it's a tricky situation with cabac which would be a shame to loose. Currently I guess it's a bit strange advertising main/high - but then others do without having mbaff support As it stands with cabac on I can make (what I think is) a perfectly legal stream with ff

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread Emil Velikov
On 4 April 2017 at 17:13, Andreas Boll wrote: > 2017-04-04 17:58 GMT+02:00 Christian König : >> Am 04.04.2017 um 17:38 schrieb boyuan.zh...@amd.com: >>> >>> From: Boyuan Zhang >>> >>> Signed-off-by: Boyuan Zhang >> >> >> Reviewed-by: Christian König >> > > Cc: stable? > Good idea, but we really

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread Andy Furniss
Well it's a tricky situation with cabac which would be a shame to loose. Currently I guess it's a bit strange advertising main/high - but then others do without having mbaff support As it stands with cabac on I can make (what I think is) a perfectly legal stream with ffmpeg or gstreamer (ig

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread Andreas Boll
2017-04-04 17:58 GMT+02:00 Christian König : > Am 04.04.2017 um 17:38 schrieb boyuan.zh...@amd.com: >> >> From: Boyuan Zhang >> >> Signed-off-by: Boyuan Zhang > > > Reviewed-by: Christian König > Cc: stable? >> --- >> src/gallium/drivers/radeon/radeon_video.c | 2 +- >> 1 file changed, 1 in

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread Christian König
Am 04.04.2017 um 17:38 schrieb boyuan.zh...@amd.com: From: Boyuan Zhang Signed-off-by: Boyuan Zhang Reviewed-by: Christian König --- src/gallium/drivers/radeon/radeon_video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/radeon_video.c

[Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

2017-04-04 Thread boyuan.zhang
From: Boyuan Zhang Signed-off-by: Boyuan Zhang --- src/gallium/drivers/radeon/radeon_video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/radeon_video.c b/src/gallium/drivers/radeon/radeon_video.c index 605a2c7..02e8dcf 100644 --- a/src/gallium